Kezhu Wang created FLINK-21132:
----------------------------------
Summary: BoundedOneInput.endInput is called when taking synchronous savepoint
Key: FLINK-21132
URL:
https://issues.apache.org/jira/browse/FLINK-21132 Project: Flink
Issue Type: Bug
Components: Runtime / Task
Affects Versions: 1.12.1, 1.11.3, 1.10.2, 1.10.3
Reporter: Kezhu Wang
[~elkhand](?) reported on project iceberg that {{BoundedOneInput.endInput}} was [called|
https://github.com/apache/iceberg/issues/2033#issuecomment-765864038] when [stopping job with savepoint|
https://github.com/apache/iceberg/issues/2033#issuecomment-765557995].
I think it is a bug of Flink and was introduced in FLINK-14230. The [changes|
https://github.com/apache/flink/pull/9854/files#diff-0c5fe245445b932fa83fdaf5c4802dbe671b73e8d76f39058c6eaaaffd9639faL577] rely on {{StreamTask.afterInvoke}} and {{OperatorChain.closeOperators}} will only be invoked after *end of input*. But that is not true long before after [FLIP-34: Terminate/Suspend Job with Savepoint|
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=103090212]. Task could enter state called [*finished*|
https://github.com/apache/flink/blob/3a8e06cd16480eacbbf0c10f36b8c79a6f741814/flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/tasks/StreamTask.java#L467] after synchronous savepoint, that is an expected job suspension and stopping.
[~sunhaibotb] [~pnowojski] [~roman_khachatryan] Could you help confirm this ?
For full context, see [apache/iceberg#2033|
https://github.com/apache/iceberg/issues/2033]. I have pushed branch [synchronous-savepoint-conflict-with-bounded-end-input-case|
https://github.com/kezhuw/flink/commits/synchronous-savepoint-conflict-with-bounded-end-input-case] in my repository. Test case {{SavepointITCase.testStopSavepointWithBoundedInput}} failed due to {{BoundedOneInput.endInput}} called.
I am also aware of [FLIP-147: Support Checkpoints After Tasks Finished|
https://cwiki.apache.org/confluence/x/mw-ZCQ], maybe the three should align on what *finished* means exactly. [~kkl0u] [~chesnay] [~gaoyunhaii]
--
This message was sent by Atlassian Jira
(v8.3.4#803005)