Skip to content

Conversation

@jgraettinger
Copy link
Contributor

@jgraettinger jgraettinger commented Jul 23, 2025

See individual commits


This change is Reviewable

@jgraettinger jgraettinger force-pushed the johnny/update-rocksdb branch from 3532af0 to d35c635 Compare July 24, 2025 19:06
@jgraettinger jgraettinger force-pushed the johnny/update-rocksdb branch 4 times, most recently from b5db0e6 to b7a53ee Compare August 15, 2025 05:19
Code changes to accommodate API updates:
- Fixed tablewriter API: SetHeader() -> Header()
- Fixed gRPC balancer test mocks to work with v1.73+ requirements
  by using wrapper pattern instead of direct interface implementation
- Simplified test structures by removing unnecessary wrapper layer
- Fixed some incorrect usages of legacy `errors` package.
@jgraettinger jgraettinger force-pushed the johnny/update-rocksdb branch from b7a53ee to 2b4ac36 Compare October 6, 2025 23:38
@jgraettinger jgraettinger requested a review from mdibaiee October 6, 2025 23:39
@jgraettinger jgraettinger merged commit c458758 into master Oct 7, 2025
1 check passed
@jgraettinger jgraettinger deleted the johnny/update-rocksdb branch October 7, 2025 13:24
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.

3 participants