-
Notifications
You must be signed in to change notification settings - Fork 163
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 streaming endpoints to executiondata api #1275
Conversation
10cdee2
to
4ecca7d
Compare
aa6c2e8
to
15818fa
Compare
@@ -6,6 +6,9 @@ option go_package = "github.com/onflow/flow/protobuf/go/flow/access"; | |||
option java_package = "org.onflow.protobuf.access"; | |||
|
|||
import "flow/entities/block_execution_data.proto"; | |||
import "flow/entities/block_header.proto"; |
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.
unused?
@@ -25,4 +34,34 @@ message GetExecutionDataByBlockIDRequest { | |||
|
|||
message GetExecutionDataByBlockIDResponse { | |||
entities.BlockExecutionData block_execution_data = 1; | |||
} |
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.
This file would need way more comments since it would likely be documentation entry point.
A good example of the documentation is google public apis: https://github.com/googleapis/googleapis/blob/master/google/spanner/v1/spanner.proto
bytes start_block_id = 1; | ||
uint64 start_block_height = 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.
You may want to describe precedence of arguments and the behaviour when both of them are zero
message EventFilter { | ||
repeated string event_type = 1; | ||
repeated string address = 2; | ||
repeated string contract = 3; |
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.
For people with a lack of context you may give a couple of examples of filters to highlight that they are OR'ed.
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.
Thanks for all the documentation!
3723: [Access] Add streaming API for BlockExecutionData r=peterargue a=peterargue Implement streaming gRPC APIs for `BlockExecutionData` and events. Protobuf: onflow/flow#1275 FLIP: onflow/flips#73 Co-authored-by: Peter Argue <89119817+peterargue@users.noreply.github.com>
3723: [Access] Add streaming API for BlockExecutionData r=peterargue a=peterargue Implement streaming gRPC APIs for `BlockExecutionData` and events. Protobuf: onflow/flow#1275 FLIP: onflow/flips#73 Co-authored-by: Peter Argue <89119817+peterargue@users.noreply.github.com>
3723: [Access] Add streaming API for BlockExecutionData r=peterargue a=peterargue Implement streaming gRPC APIs for `BlockExecutionData` and events. Protobuf: onflow/flow#1275 FLIP: onflow/flips#73 Co-authored-by: Peter Argue <89119817+peterargue@users.noreply.github.com>
3723: [Access] Add streaming API for BlockExecutionData r=peterargue a=peterargue Implement streaming gRPC APIs for `BlockExecutionData` and events. Protobuf: onflow/flow#1275 FLIP: onflow/flips#73 Co-authored-by: Peter Argue <89119817+peterargue@users.noreply.github.com>
3723: [Access] Add streaming API for BlockExecutionData r=peterargue a=peterargue Implement streaming gRPC APIs for `BlockExecutionData` and events. Protobuf: onflow/flow#1275 FLIP: onflow/flips#73 Co-authored-by: Peter Argue <89119817+peterargue@users.noreply.github.com>
Closes: #???
Description
For contributor use:
master
branchFiles changed
in the Github PR explorer