kbendick commented on a change in pull request #1515:
URL: https://github.com/apache/iceberg/pull/1515#discussion_r495527462
##########
File path:
flink/src/main/java/org/apache/iceberg/flink/sink/IcebergStreamWriter.java
##########
@@ -57,6 +64,9 @@ public void open() {
// Initialize the task writer.
this.writer = taskWriterFactory.create();
+
+ // If we don't enable checkpoint, we will use processingTimeSerice to do
commit,
+ boolean isCheckpointEnabled = getRuntimeContext().isCheckpointingEnabled();
Review comment:
This appears to be unused (it's neither stored as a class field or used
elsewhere in this constructor). Perhaps this is a bug or simply a left over
artifact from some refactoring?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
