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

Adopt new serializer protocols #29

Merged
merged 2 commits into from
Jan 17, 2025
Merged

Adopt new serializer protocols #29

merged 2 commits into from
Jan 17, 2025

Conversation

glbrntt
Copy link
Collaborator

@glbrntt glbrntt commented Jan 14, 2025

Motivation:

The core package made a few changes allowing for transport to define an associated bag-of-bytes types so that they can avoid copying to and from [UInt8]. This also came with changes to the serialization protocols.

Modifications:

  • Add a thin adapter type to bridge between gRPC and Protobuf contiguous bytes
  • Adopt new protocols

Result:

Builds again

Motivation:

The core package made a few changes allowing for transport to define an
associated bag-of-bytes types so that they can avoid copying to
and from `[UInt8]`. This also came with changes to the serialization
protocols.

Modifications:

- Add a thin adapter type to bridge between gRPC and Protobuf contiguous
  bytes
- Adopt new protocols

Result:

Builds again
@glbrntt glbrntt added the ⚠️ semver/major Breaks existing public API. label Jan 14, 2025
@glbrntt
Copy link
Collaborator Author

glbrntt commented Jan 14, 2025

This depends on and won't compile until grpc/grpc-swift#2155 is merged.

@glbrntt glbrntt marked this pull request as ready for review January 17, 2025 11:03
@glbrntt glbrntt enabled auto-merge (squash) January 17, 2025 11:16
@glbrntt glbrntt disabled auto-merge January 17, 2025 11:25
@glbrntt glbrntt merged commit 768985a into grpc:main Jan 17, 2025
20 of 21 checks passed
@glbrntt glbrntt deleted the new-serializer branch January 17, 2025 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️ semver/major Breaks existing public API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants