Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync 4.x with OSS 3.11 #2

Merged
merged 16 commits into from
Dec 21, 2022
Merged

Conversation

ayushisingh29
Copy link
Owner

No description provided.

mattl-netflix and others added 16 commits July 6, 2022 23:08
…d within a certain window. (#992)" (#997)

This reverts commit 57b26a4.
… we use BigInteger to store and compare them. (#1005)
…ad data files last. (#999)

* Reveal whether an sstable file is an incremental backup in SNS notification.

* Remove retry parameter from upload method. It is always 10 in practice and there is no way to change it without a code change.

* Move upload logic out of AbstractBackup into an interface called BackupHelper.

* Upload data files last.

* Don't compress anything when backupsToCompress is set to None.
…notification messages (#1006)

* Adding ability to add more keys in the message attributes for backup notification messages

* Formatting as per google java format

* Incorporating review comments

* Fixing failing tests in TestBackupNotificationMgr
… attributes. (#1010)

* Use an ImmutableSet rather than an ImmutableList of additional message attributes when sending SNS backup notifications.

* Use ImmutableSet rather than ImmutableList to store additional message attributes.
…en before then it is automatically compressed using SNAPPY before upload. (#1013)
@ayushisingh29 ayushisingh29 merged commit d3849d0 into ayushisingh29:ayushis/4.x Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants