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

Add ability to configure durableDelete property for Batch Client Policies #256

Conversation

RomanCht
Copy link
Contributor

@RomanCht RomanCht commented Sep 4, 2024

@amizurov
Copy link

amizurov commented Sep 5, 2024

@agrgr, @roimenashe, @reugn Hey guys, PTAL. It would be great if we could patch previous versions аs well. Thanks.

@reugn
Copy link
Member

reugn commented Sep 8, 2024

@RomanCht, @amizurov, thanks for the PR. We're looking into it.

@RomanCht
Copy link
Contributor Author

@agrgr, @roimenashe, @reugn Guys, sorry for being pushy, have you had a chance to check this out?

@agrgr agrgr added the enhancement New feature or request label Sep 23, 2024
@agrgr agrgr merged commit 05ded3b into aerospike-community:main Sep 24, 2024
3 of 4 checks passed
@RomanCht
Copy link
Contributor Author

Guys, thank you for merge!

We would like to have other minor versions to be patched with this change.
But this repository doesn't have separate branches for these versions (0.14.x, 0.13.x ... 0.8.x) so there's no where to make Pull requests to.

Could you please create those branches or is there any other way to patch these minor versions somehow?

@reugn
Copy link
Member

reugn commented Sep 26, 2024

@RomanCht what is your motivation to patch older versions instead of updating to the new one in all services? We plan to coordinate the release with you to make sure that it meets your needs.

@RomanCht
Copy link
Contributor Author

RomanCht commented Sep 27, 2024

@reugn hi, thank you!
We plan to switch on durable delete for a big number of services, which are using different combinations of aerospike client, spring and other internal dependencies. It'll be much easier to just increment needed patch version of the starter without the risk to break something or cause additional code migration complexity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants