y1chi commented on a change in pull request #15917:
URL: https://github.com/apache/beam/pull/15917#discussion_r747726431
##########
File path: model/pipeline/src/main/proto/beam_runner_api.proto
##########
@@ -1579,6 +1585,12 @@ message StandardRunnerProtocols {
enum Enum {
// Indicates suport the MonitoringInfo short id protocol.
MONITORING_INFO_SHORT_IDS = 0 [(beam_urn) =
"beam:protocol:monitoring_info_short_ids:v1"];
+
+ // Indicates that this runner can process elements embedded in Fn API
+ // control plane ProcessBundleResponse. See more about the protocol at
+ // https://s.apache.org/beam-fn-api-control-data-embedding
Review comment:
Done.
##########
File path: model/pipeline/src/main/proto/beam_runner_api.proto
##########
@@ -1570,6 +1570,12 @@ message StandardProtocols {
// the entire SDK harness process, not specific to a bundle.
HARNESS_MONITORING_INFOS = 4
[(beam_urn) = "beam:protocol:harness_monitoring_infos:v1"];
+
+ // Indicates that this SDK can process elements embedded in Fn API
+ // control plane ProcessBundleRequest. See more about the protocol at
+ // https://s.apache.org/beam-fn-api-control-data-embedding
Review comment:
Done.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@beam.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
