y1chi commented on a change in pull request #15917:
URL: https://github.com/apache/beam/pull/15917#discussion_r747726284
##########
File path: model/fn-execution/src/main/proto/beam_fn_api.proto
##########
@@ -327,6 +335,13 @@ message ProcessBundleResponse {
// payload field with the bytes specified here.
map<string, bytes> monitoring_data = 5;
+ // (optional) Output elements of the processed bundle. Either all or
+ // none of the bundle elements should be included in the
ProcessBundleResponse.
+ // This elements embedding is to achieve better Fn API efficiency for bundles
+ // that only produce small amount of outputs. See more 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
