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

Modifications to Block Stream based on recent work #421

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jsync-swirlds
Copy link
Member

Modify block service definition to allow repeated block items

  • Modified PublishStreamRequest to permit repeated block items.
  • Added a SubscribeStreamResponseSet wrapper
    • This works around that oneof does not permit repeated fields.
  • Modified SubscribeStreamResponseSet to have a repeated BlockItem list.

Make the gRPC service modular

  • Modify gRPC service design.
    • Replace single service with four specific services for modularity

<h3>Modify block service definition to allow repeated block items</h3>
* Modified `PublishStreamRequest` to permit repeated block items.
* Added a `SubscribeStreamResponseSet` wrapper
   * This works around that `oneof` does not permit `repeated` fields.
* Modified `SubscribeStreamResponseSet` to have a `repeated` `BlockItem` list.

<h3>Make the gRPC service modular</h3>
* Modify gRPC service design.
   * Replace single service with four specific services for modularity

Signed-off-by: Joseph Sinclair <joseph.sinclair@swirldslabs.com>
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.

2 participants