zhupitertop commented on PR #11413: URL: https://github.com/apache/seatunnel/pull/11413#issuecomment-5023175991
Hi @davidzollo @imbajin, I would like to ask for your advice on the merge strategy for PR #11413. This PR is relatively large (~13k lines changed), because it includes: * Adding the HugeGraph Source Connector * Refactoring the existing HugeGraph Sink Connector with multi-mapping support * Adding related tests and documentation I understand that a large PR may increase the review and maintenance burden. Before moving forward, I would like to know whether the SeaTunnel community prefers this kind of change to be split into multiple smaller PRs (for example, source connector, sink refactoring, and tests/docs separately), or if it is acceptable to review and merge it as a single PR. I am happy to adjust the PR structure according to the project's preference. Thanks for your guidance! -- 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]
