Skip to content

Conversation

@shubhamdhama
Copy link
Contributor

Enable the Blob service on the DRPC server in addition to gRPC. This is controlled by rpc.experimental_drpc.enabled (off by default).

This change is part of a series and is similar to: #146926

Note: This only registers the service; the client is not updated to use the DRPC client, so this service will not have any functional effect.

Epic: CRDB-48925
Release note: None

@shubhamdhama shubhamdhama requested a review from a team as a code owner June 18, 2025 09:34
@shubhamdhama shubhamdhama requested review from kev-cao and removed request for a team June 18, 2025 09:34
@blathers-crl
Copy link

blathers-crl bot commented Jun 18, 2025

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor

@cthumuluru-crdb cthumuluru-crdb left a comment

Choose a reason for hiding this comment

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

LGTM! Please add a ticket to both commit message and PR before committing this change.

Enable the `Blob` service on the DRPC server in addition to gRPC.
This is controlled by `rpc.experimental_drpc.enabled` (off by default).

This change is part of a series and is similar to: cockroachdb#146926

Note: This only registers the service; the client is not updated to use the
DRPC client, so this service will not have any functional effect.

Epic: CRDB-48925
Release note: None
@cthumuluru-crdb
Copy link
Contributor

bors r+

@craig
Copy link
Contributor

craig bot commented Jun 23, 2025

Build failed:

@cthumuluru-crdb
Copy link
Contributor

bors r+

@craig
Copy link
Contributor

craig bot commented Jun 23, 2025

@craig craig bot merged commit 64dc647 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.

3 participants