acrites commented on code in PR #37691:
URL: https://github.com/apache/beam/pull/37691#discussion_r2963078095
##########
sdks/java/harness/src/main/java/org/apache/beam/fn/harness/state/BagUserState.java:
##########
@@ -54,6 +55,8 @@ public class BagUserState<T> {
private List<T> newValues;
private boolean isCleared;
private boolean isClosed;
+ private final Supplier<Boolean> hasNoState;
Review Comment:
I believe all of these `*UserState` classes are only used for a single
bundle, afterwhich they are closed and can't be used again. Hence we can
simplify this and only pass in Boolean's. It's only the Factory that needs the
Suppliers.
--
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]
