ryan-mbuashundip commented on code in PR #36708:
URL: https://github.com/apache/beam/pull/36708#discussion_r2491588456
##########
model/fn-execution/src/main/proto/org/apache/beam/model/fn_execution/v1/beam_fn_api.proto:
##########
@@ -415,6 +415,12 @@ message ProcessBundleRequest {
// beam:protocol:control_request_elements_embedding:v1 capability. See more
// at https://s.apache.org/beam-fn-api-control-data-embedding.
Elements elements = 3;
+
+ // If "true", SDK will skip state fetch
+ bool key_has_no_state = 4;
+
+ // If "true", SDK will skip state write
Review Comment:
> Bundles are not restricted to have a single key. In fact they very
frequently have a whole key range.
Is this true for streaming as well?
> Do these booleans mean that no key in the bundle has any state / has any
further bundles?
Given that state is a function of the operation/transform I believe this is
safe to assume.
Thanks for the example blurbs, I think they cover the intention of the bits
well. Given your clarification, `has_no_keyed_state` and `only_bundle_for_keys`
are better names for these bits.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
