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
feat: added generic proto file for grpc service #2742
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few early things:
- Make sure to follow convention: We should adhere to Google convention.
- Comments should be consistent. Preferrably
//
oneof io_descriptor{ | ||
string text = 1; | ||
Array array = 2; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://developers.google.com/protocol-buffers/docs/proto3#reserved
add a reserved message field for BentoService? @ssheng @sauyon ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should reserve a range eventually, yes.
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com> Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com> Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com> Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com> Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com> Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com> Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com> Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com> Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com> Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Co-authored-by: Sadab Hafiz <sadabhafizny@gmail.com> Co-authored-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com> Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
What does this PR address?
The generic proto file contains a
BentoML
service with an RPC calledcall
. Fixedio_descriptor.proto
file with updated name ofArray
instead ofNumpyNdarray
.Before submitting:
guide on how to create a pull request.
make format
andmake lint
script have passed (instructions)?those accordingly? Here are documentation guidelines and tips on writting docs.
Who can help review?
The proto file and changes are based on our discussion. Would appreciate any feedback!
@aarnphm @sauyon @ssheng