-
Notifications
You must be signed in to change notification settings - Fork 8
Rename RPCs named as StreamX to GetXStream #84
Copy link
Copy link
Closed
Labels
part:protobufAffects the protocol buffer definition filesAffects the protocol buffer definition filespriority:❓We need to figure out how soon this should be addressedWe need to figure out how soon this should be addressedtype:enhancementNew feature or enhancement visitble to usersNew feature or enhancement visitble to users
Milestone
Metadata
Metadata
Assignees
Labels
part:protobufAffects the protocol buffer definition filesAffects the protocol buffer definition filespriority:❓We need to figure out how soon this should be addressedWe need to figure out how soon this should be addressedtype:enhancementNew feature or enhancement visitble to usersNew feature or enhancement visitble to users
What's needed?
streamas a verb refers to the act of publishing something. Therefore RPC names following the naming schemeStreamX, e.g.,StreamComponentData, are ambigous. E.g., a client invokingstream_component_datacould also indicate that the client intends to publish data, instead of consuming data from a stream.Proposed solution
A more appropriate name would be
GetXStream. E.g., the RPCStreamComponentDatacould be renamed toGetComponentDataStream, removing any ambiguity in the intention of the method.Edit:
The current consensus is to have
PublishXmethods to stream data, andSubscribeXmethods to consume data from streams.Use cases
No response
Alternatives and workarounds
No response
Additional context
No response