gustavodemorais commented on PR #26673: URL: https://github.com/apache/flink/pull/26673#issuecomment-2967192241
> I'm fine with this change, but usually a PR should be covered by tests. Would be great to enable the MVP end to end running example first and then dive into details e.g. starting with append and switch to updating. My idea was to create smaller PRs to make them easier to review and make smaller iterations. Or else I'll not be able to continuously share the progress and just appear with one big PR eventually. I'll make sure we have enough tests to cover everything either way. What do you think it's the best approach? -- 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. To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
