Github user fhueske commented on a diff in the pull request:
https://github.com/apache/flink/pull/5860#discussion_r183757920
--- Diff:
flink-connectors/flink-connector-filesystem/src/main/java/org/apache/flink/streaming/connectors/fs/bucketing/BucketingSink.java
---
@@ -536,6 +553,9 @@ private void openNewPartFile(Path bucketPath,
BucketState<T> bucketState) throws
partPath = new Path(bucketPath, partPrefix + "-" +
subtaskIndex + "-" + bucketState.partCounter);
}
+ // Record the creation time of the bucket
+ bucketState.firstWrittenToTime =
processingTimeService.getCurrentProcessingTime();
--- End diff --
I think the current behavior is also more consistent than starting the
timeout when the first record is written.
---
