rakeshcusat commented on code in PR #35209:
URL: https://github.com/apache/beam/pull/35209#discussion_r2138550498
##########
sdks/python/apache_beam/transforms/userstate.py:
##########
@@ -63,7 +63,7 @@ def to_runner_api(
raise NotImplementedError
-class ReadModifyWriteStateSpec(StateSpec):
+class ReadModifyWriteStateSpec(StateSpec): #TODO: This state is missing from
the programming guide.
Review Comment:
That's right and they may need to upgrade it.
Can we just introduce ValueStateSpec and mark this ReadModifyWriteStateSpec
as deprecated? It will not break the older code but moving forward it will be
consistent with the other SDKs.
--
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]
