danny0405 commented on PR #19304: URL: https://github.com/apache/hudi/pull/19304#issuecomment-5740434786
Hi, @yihua I make a new commit to adjust the logic a little bit, the major changes are: 1. move the recordComplexKeygenEncodingIfMissing from startCommit to rige before the actual ingestion workload is triggered, so that we avoid redundant check for long running job(flink and deltastreamer); 2. simplify the Flink validation logic: remove the encoding inference since there is rare users for hudi flink 1.0.x(most upgrade to 1.1.x directly), will add it when we got a real feedback. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
