[GitHub] [flink] pgaref commented on pull request #21923: FLINK-13871: Consolidate volatile status fields in StreamTask

2023-02-17 Thread via GitHub
pgaref commented on PR #21923: URL: https://github.com/apache/flink/pull/21923#issuecomment-1434911355 @flinkbot run azure -- 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. T

[GitHub] [flink] pgaref commented on pull request #21923: FLINK-13871: Consolidate volatile status fields in StreamTask

2023-02-17 Thread via GitHub
pgaref commented on PR #21923: URL: https://github.com/apache/flink/pull/21923#issuecomment-1434978767 @flinkbot run azure -- 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. T

[GitHub] [flink] pgaref commented on pull request #21923: FLINK-13871: Consolidate volatile status fields in StreamTask

2023-02-26 Thread via GitHub
pgaref commented on PR #21923: URL: https://github.com/apache/flink/pull/21923#issuecomment-1445719083 > Hi @pgaref , thanks a lot for the refactor, it's very useful for maintainability. I have left some comments, please help take a look in your free time, thanks~ > > Thanks @pnowojs

[GitHub] [flink] pgaref commented on pull request #21923: FLINK-13871: Consolidate volatile status fields in StreamTask

2023-02-27 Thread via GitHub
pgaref commented on PR #21923: URL: https://github.com/apache/flink/pull/21923#issuecomment-1447279707 Thanks everyone for the useful comments! PTAL on the latest changes, in short: - Checked all state transitions and documented them as part of https://github.com/apache/flink/pull/21