Skip to content

fix: keydb consistent hashing #6403

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
Merged
merged 1 commit into from
Sep 30, 2025
Merged

fix: keydb consistent hashing #6403

merged 1 commit into from
Sep 30, 2025

Conversation

fracasula
Copy link
Collaborator

@fracasula fracasula commented Sep 30, 2025 •
edited
Loading

Description

Updating keydb to use new client with consistent hashing.

Linear Ticket

Fixes PIPE-2437

Security

  • The code changed/added as part of this pull request won't create any security issues with how the software is being used.

@fracasula fracasula enabled auto-merge (squash) September 30, 2025 09:42
@codecov Codecov
Copy link

codecov bot commented Sep 30, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.58%. Comparing base (7628efa) to head (207cf61).
⚠️ Report is 1 commits behind head on release/1.60.x.

Additional details and impacted files 
@@                Coverage Diff                 @@
##           release/1.60.x    #6403      +/-   ##
==================================================
+ Coverage           77.54%   77.58%   +0.03%     
==================================================
  Files                 523      523              
  Lines               70327    70327              
==================================================
+ Hits                54534    54560      +26     
+ Misses              12945    12927      -18     
+ Partials             2848     2840       -8

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fracasula fracasula merged commit 3ee79b2 into release/1.60.x Sep 30, 2025
92 of 94 checks passed
@fracasula fracasula deleted the keydbConsistentHashing branch September 30, 2025 10:18
This was referenced Sep 30, 2025
Merged
Closed
Merged
mihir20 pushed a commit that referenced this pull request Sep 30, 2025
@devops-github-rudderstack
chore: prerelease 1.60.0-rc.3 (#6405)
6a86d03 
🤖 I have created a release *beep* *boop*
---


##
[1.60.0-rc.3](v1.59.0...v1.60.0-rc.3)
(2025-09-30)


### Features

* **router:** jobsdb pickup query batch size adapting to throttling
limit ([#6338](#6338))
([c38b971](c38b971))


### Bug Fixes

* add outgoing metrics to proxy flow
([#6355](#6355))
([23a5c81](23a5c81))
* aws session config region
([#6354](#6354))
([23a5c81](23a5c81))
* dedup gauge
([#6359](#6359))
([a562df4](a562df4))
* keydb consistent hashing
([#6403](#6403))
([3ee79b2](3ee79b2))
* keydb grpc config
([#6370](#6370))
([cf04743](cf04743))
* missing keydb client stats
([#6360](#6360))
([c42e5c8](c42e5c8))
* naming collision in redis throttling configuration
([#6365](#6365))
([4f87afb](4f87afb))
* set table type to external for glue
([#6386](#6386))
([9017582](9017582))
* ut mirroring tests
([#6341](#6341))
([51dd4f7](51dd4f7))
* **warehouse:** alter namespace col size
([#6379](#6379))
([fbba498](fbba498))
* **warehouse:** skip extract async job failing test
([#6378](#6378))
([89a4d1e](89a4d1e))


### Miscellaneous

* add authentication to reporting client
([#6384](#6384))
([502d2b0](502d2b0))
* add explicit permissions for workflows
([#6381](#6381))
([1d22623](1d22623))
* add readme for async destinaiton module
([#6356](#6356))
([cc80333](cc80333))
* configurable event name trimming for reporting
([#6394](#6394))
([7628efa](7628efa))
* remove deprecated throttling configuration keys
([#6377](#6377))
([8d667b7](8d667b7))
* upgrade build-scan-push-action to v1.8.0
([#6350](#6350))
([51dd4f7](51dd4f7))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
mihir20 pushed a commit that referenced this pull request Oct 1, 2025
@devops-github-rudderstack
chore: release 1.60.0 (#6390)
bc7879c 
🤖 I have created a release *beep* *boop*
---


##
[1.60.0](v1.59.0...v1.60.0)
(2025-09-30)


### Features

* **router:** jobsdb pickup query batch size adapting to throttling
limit ([#6338](#6338))
([c38b971](c38b971))


### Bug Fixes

* add outgoing metrics to proxy flow
([#6355](#6355))
([23a5c81](23a5c81))
* aws session config region
([#6354](#6354))
([23a5c81](23a5c81))
* dedup gauge
([#6359](#6359))
([a562df4](a562df4))
* keydb consistent hashing
([#6403](#6403))
([3ee79b2](3ee79b2))
* keydb grpc config
([#6370](#6370))
([cf04743](cf04743))
* missing keydb client stats
([#6360](#6360))
([c42e5c8](c42e5c8))
* naming collision in redis throttling configuration
([#6365](#6365))
([4f87afb](4f87afb))
* set table type to external for glue
([#6386](#6386))
([9017582](9017582))
* ut mirroring tests
([#6341](#6341))
([51dd4f7](51dd4f7))
* **warehouse:** alter namespace col size
([#6379](#6379))
([fbba498](fbba498))
* **warehouse:** skip extract async job failing test
([#6378](#6378))
([89a4d1e](89a4d1e))


### Miscellaneous

* add authentication to reporting client
([#6384](#6384))
([502d2b0](502d2b0))
* add explicit permissions for workflows
([#6381](#6381))
([1d22623](1d22623))
* add readme for async destinaiton module
([#6356](#6356))
([cc80333](cc80333))
* configurable event name trimming for reporting
([#6394](#6394))
([7628efa](7628efa))
* remove deprecated throttling configuration keys
([#6377](#6377))
([8d667b7](8d667b7))
* upgrade build-scan-push-action to v1.8.0
([#6350](#6350))
([51dd4f7](51dd4f7))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
mihir20 pushed a commit that referenced this pull request Oct 1, 2025
@fracasula @mihir20
fix: keydb consistent hashing (#6403)
472e529 
# Description

Updating keydb to use new client with consistent hashing.

## Linear Ticket

Fixes
[PIPE-2437](https://linear.app/rudderstack/issue/PIPE-2437/keydb-fix-flaky-test)

## Security

- [x] The code changed/added as part of this pull request won't create
any security issues with how the software is being used.
mihir20 pushed a commit that referenced this pull request Oct 1, 2025
@devops-github-rudderstack @mihir20
chore: release 1.60.0 (#6390)
c461070 
🤖 I have created a release *beep* *boop*
---


##
[1.60.0](v1.59.0...v1.60.0)
(2025-09-30)


### Features

* **router:** jobsdb pickup query batch size adapting to throttling
limit ([#6338](#6338))
([c38b971](c38b971))


### Bug Fixes

* add outgoing metrics to proxy flow
([#6355](#6355))
([23a5c81](23a5c81))
* aws session config region
([#6354](#6354))
([23a5c81](23a5c81))
* dedup gauge
([#6359](#6359))
([a562df4](a562df4))
* keydb consistent hashing
([#6403](#6403))
([3ee79b2](3ee79b2))
* keydb grpc config
([#6370](#6370))
([cf04743](cf04743))
* missing keydb client stats
([#6360](#6360))
([c42e5c8](c42e5c8))
* naming collision in redis throttling configuration
([#6365](#6365))
([4f87afb](4f87afb))
* set table type to external for glue
([#6386](#6386))
([9017582](9017582))
* ut mirroring tests
([#6341](#6341))
([51dd4f7](51dd4f7))
* **warehouse:** alter namespace col size
([#6379](#6379))
([fbba498](fbba498))
* **warehouse:** skip extract async job failing test
([#6378](#6378))
([89a4d1e](89a4d1e))


### Miscellaneous

* add authentication to reporting client
([#6384](#6384))
([502d2b0](502d2b0))
* add explicit permissions for workflows
([#6381](#6381))
([1d22623](1d22623))
* add readme for async destinaiton module
([#6356](#6356))
([cc80333](cc80333))
* configurable event name trimming for reporting
([#6394](#6394))
([7628efa](7628efa))
* remove deprecated throttling configuration keys
([#6377](#6377))
([8d667b7](8d667b7))
* upgrade build-scan-push-action to v1.8.0
([#6350](#6350))
([51dd4f7](51dd4f7))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ktgowtham ktgowtham ktgowtham approved these changes

@mihir20 mihir20 mihir20 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fracasula @ktgowtham @mihir20