Zakelly commented on code in PR #23441:
URL: https://github.com/apache/flink/pull/23441#discussion_r1349473352
##########
flink-core/src/main/java/org/apache/flink/api/common/state/AppendingState.java:
##########
@@ -60,8 +60,11 @@ public interface AppendingState<IN, OUT> extends State {
*
* <p>If null is passed in, the state value will remain unchanged.
*
+ * <p>Except for list state, where null value is not allowed to be passed
in.
Review Comment:
Agreed. I believe it would be better to eliminate the description of the
null value at the moment, considering the implementations have already become
varied. We can consider refactoring the interfaces in the upcoming major
version. What do you think? @davidradl
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
