Skip to content

br: add compression type for br backup request#642

Merged
overvenus merged 2 commits intopingcap:masterfrom
glorv:backup-compression
Jul 6, 2020
Merged

br: add compression type for br backup request#642
overvenus merged 2 commits intopingcap:masterfrom
glorv:backup-compression

Conversation

@glorv
Copy link
Contributor

@glorv glorv commented Jul 2, 2020

add CompressionType for br BackupRequest, so br can choose compress algorithm explicitly.

@glorv glorv requested a review from MyonKeminta July 2, 2020 08:38
@CLAassistant
Copy link

CLAassistant commented Jul 2, 2020

CLA assistant check
All committers have signed the CLA.

@glorv glorv requested review from 3pointer and overvenus July 2, 2020 08:38
@glorv glorv force-pushed the backup-compression branch from 3dd6c45 to c2e6be1 Compare July 2, 2020 08:40
@glorv glorv force-pushed the backup-compression branch from c2e6be1 to a2a6737 Compare July 2, 2020 08:42
Copy link
Contributor

@3pointer 3pointer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@overvenus overvenus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rest LGTM

Comment on lines +120 to +126
// algorithm used for compress sst file
enum CompressionType {
UNKNOWN = 0;
LZ4 = 1;
SNAPPY = 2;
ZSTD = 3;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you define the enum outside of BackupRequest?

Copy link
Member

@overvenus overvenus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@overvenus overvenus merged commit 1e0910a into pingcap:master Jul 6, 2020
glorv added a commit to glorv/kvproto that referenced this pull request Jul 15, 2020
glorv added a commit to glorv/kvproto that referenced this pull request Jul 15, 2020
overvenus pushed a commit that referenced this pull request Jul 15, 2020
* backup: add compression type for br backup request (#642)

* specify protobuf-build version
daimashusheng pushed a commit to daimashusheng/kvproto that referenced this pull request Sep 2, 2021
@glorv glorv deleted the backup-compression branch March 12, 2024 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants