[GitHub] [flink] XComp commented on pull request #22984: [FLINK-32570][runtime][stream] Deprecates @Public/@PublicEvolving API that uses Flink's Time classes

2023-07-13 Thread via GitHub
XComp commented on PR #22984: URL: https://github.com/apache/flink/pull/22984#issuecomment-1634211308 Yup, I agree. I feel like it makes sense to summarize everything in a FLIP. I created

[GitHub] [flink] XComp commented on pull request #22984: [FLINK-32570][runtime][stream] Deprecates @Public/@PublicEvolving API that uses Flink's Time classes

2023-07-12 Thread via GitHub
XComp commented on PR #22984: URL: https://github.com/apache/flink/pull/22984#issuecomment-1632417952 I assumed the [discussion mentioned in FLINK-14068](https://lists.apache.org/thread/76yywnwf3lk8qn4dby0vz7yoqx7f7pkj) and FLINK-14068 itself are good enough as a conclusion. But I can go