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.

Sign up for GitHub

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

Jump to bottom

Migrate to non-deprecated Apache Curator APIs #1886

Closed
moscicky opened this issue Aug 5, 2024 · 3 comments · Fixed by #1910
Closed

Migrate to non-deprecated Apache Curator APIs #1886

moscicky opened this issue Aug 5, 2024 · 3 comments · Fixed by #1910
Assignees
@Anarak404
Labels
hacktoberfest

Comments

@moscicky
Copy link
Collaborator

moscicky commented Aug 5, 2024

There are a couple of places which use deprecated Curator APIs, e.g. transactions and PathChildrenCache. Migrate them to new APIs
@pitagoras3 pitagoras3 mentioned this issue Aug 5, 2024
Merged
@szczygiel-m
Copy link
Contributor

Rewrite API related to transactions should be easy, and it can be done during hacktoberfest.

Code related with HierarchicalCache class is a little bit harder, and it can be done in a separate PR.

@Anarak404
Copy link
Contributor

Hi, can I work on this issue?

@szczygiel-m
Copy link
Contributor

Sure, assigned 😄 If You have any questions, feel free to ask

Anarak404 added a commit to Anarak404/hermes that referenced this issue Oct 16, 2024
@Anarak404
resolve allegro#1886 | rewrite deprecated curator API - transactions
27b942f
@Anarak404 Anarak404 mentioned this issue Oct 16, 2024
Merged
szczygiel-m added a commit to Anarak404/hermes that referenced this issue Nov 8, 2024
@szczygiel-m
Merge branch 'master' into feature/allegroGH-1886--curator-api
d6a44ed
szczygiel-m added a commit that referenced this issue Nov 8, 2024
@Anarak404 @szczygiel-m
resolve #1886 | rewrite deprecated curator API - transactions (#1910)
0952046 
Co-authored-by: Mateusz <76775507+szczygiel-m@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Anarak404 Anarak404

Labels
hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Rewrite deprecated curator API - transactions Anarak404/hermes
3 participants
@moscicky @Anarak404 @szczygiel-m