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

Flatten out protobuf wrappers #2

Open
jaganseshadri opened this issue Sep 21, 2019 · 0 comments
Open

Flatten out protobuf wrappers #2

jaganseshadri opened this issue Sep 21, 2019 · 0 comments

Comments

@jaganseshadri
Copy link
Owner

The protobuf wrapper class mimic the structure of the message defined inside the .proto files.
Pros:

  • Provides namespace organization synonymous with the protobuf definition.
    Cons:
  • Loses out on namespace abstraction that makes it easier for people who don't know either gRPC or protobuf or both to work with it, i.e. it's still laborious to know which message needs what attributes to be set.

Proposed solution:

  1. More documentation would definitely help.
  2. Flatten out the protobuf definition, i.e. bring out some nested attributes to the parent level.
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

No branches or pull requests

1 participant