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.

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

Add to_proto to descriptor classes (#19971) #20223

Merged
merged 1 commit into from
Feb 4, 2025
Merged

Conversation

copybara-service[bot]
Copy link

Add to_proto to descriptor classes (#19971)

Some APIs (e.g. BigQuery Storage) use DescriptorProto for schema descriptions. But, in Ruby, there are no methods for that so far.

This PR implements the method for that(to_proto) to descriptor classes.

Fixes #12044.

Closes #19971

COPYBARA_INTEGRATE_REVIEW=#19971 from y-yagi:add_to_proto_to_descriptors 8d50ae2
FUTURE_COPYBARA_INTEGRATE_REVIEW=#19971 from y-yagi:add_to_proto_to_descriptors 8d50ae2

@copybara-service copybara-service bot force-pushed the test_723117256 branch 2 times, most recently from 15cf1f2 to a576b6e Compare February 4, 2025 18:26
Some APIs (e.g. [BigQuery Storage](https://github.com/googleapis/googleapis/blob/ab61c52324b39d1c8619a96ab19cc6c173b13f2c/google/cloud/bigquery/storage/v1/protobuf.proto#L39)) use DescriptorProto for schema descriptions. But,  in Ruby, there are no methods for that so far.

This PR implements the method for that(`to_proto`) to descriptor classes.

Fixes #12044.

Closes #19971

PiperOrigin-RevId: 723149905
@copybara-service copybara-service bot merged commit 74122eb into main Feb 4, 2025
@copybara-service copybara-service bot deleted the test_723117256 branch February 4, 2025 18:54
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.

Ruby: Convert Descriptor to DescriptorProto
1 participant