programmerloverun commented on PR #10849: URL: https://github.com/apache/seatunnel/pull/10849#issuecomment-4570952036
@DanielLeens Thanks for the detailed follow-up review. Regarding Problem 1 (PR scope): on the latest head 6d7eb94, I reverted all unrelated seatunnel-engine-ui / seatunnel-common changes. The current PR diff against dev contains 37 files only, limited to: connector-google-bigtable module (source, sink, config, tests) plugin-mapping.properties / config/plugin_config seatunnel-connectors-v2/pom.xml and seatunnel-dist/pom.xml en/zh docs and changelog for Google Bigtable There are no remaining changes under seatunnel-engine-ui or seatunnel-common in the current diff. On the Bigtable connector path itself, I believe the earlier runtime/test blockers have been addressed as you noted. I will also sync the branch with the latest dev and re-run CI so we have a clean green signal. Could you please take another look at the current file list on 6d7eb94? Happy to adjust if anything still looks out of scope. -- 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]
