davidradl commented on code in PR #27375:
URL: https://github.com/apache/flink/pull/27375#discussion_r2669189215
##########
flink-runtime/src/main/java/org/apache/flink/runtime/taskmanager/RuntimeEnvironment.java:
##########
@@ -408,4 +411,14 @@ public CheckpointStorageAccess
getCheckpointStorageAccess() {
public ChannelStateWriteRequestExecutorFactory
getChannelStateExecutorFactory() {
return channelStateExecutorFactory;
}
+
+ public void setChannelStateWriter(ChannelStateWriter channelStateWriter) {
+ checkState(this.channelStateWriter == null, "Can not set
channelStateWriter twice!");
Review Comment:
Do we know if we can get here with an existing one present - if so can we
check for that situation earlier? I am happy with the check to be safe - but
wanted to ask this question.
nit: Can not -> Cannot
--
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]
