Skip to content

*: consolidate InternalClient RPC client creation #148607

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

Merged
merged 1 commit into from
Jun 23, 2025

Conversation

cthumuluru-crdb
Copy link
Contributor

This commit consolidates InternalClient RPC client creation logic and replaces RPC clients with adapters. It is a continuation of the work done in #147606.

Epic: CRDB-48923
Fixes: #148353
Release note: none

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@cthumuluru-crdb cthumuluru-crdb changed the title *: consolidate InternalClient RPC client creation *: consolidate InternalClient RPC client creation Jun 20, 2025
@cthumuluru-crdb cthumuluru-crdb added the do-not-merge bors won't merge a PR with this label. label Jun 20, 2025
@cthumuluru-crdb cthumuluru-crdb marked this pull request as ready for review June 20, 2025 18:55
@cthumuluru-crdb cthumuluru-crdb requested review from a team as code owners June 20, 2025 18:55
"github.com/cockroachdb/cockroach/pkg/server"
"github.com/cockroachdb/cockroach/pkg/server/serverpb"
"github.com/cockroachdb/cockroach/pkg/ts/tspb"
"github.com/cockroachdb/errors"
"google.golang.org/grpc"
"storj.io/drpc"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hello, beautiful

This commit consolidates `InternalClient` RPC client creation logic and
replaces RPC clients with adapters. It is a continuation of the work
done in cockroachdb#147606.

Epic: CRDB-48923
Fixes: cockroachdb#148353
Release note: none
@cthumuluru-crdb cthumuluru-crdb removed the do-not-merge bors won't merge a PR with this label. label Jun 23, 2025
@cthumuluru-crdb
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Jun 23, 2025

@craig craig bot merged commit dda5160 into cockroachdb:master Jun 23, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

drpc: replace gRPC/DRPC clients with adapters
3 participants