zhupitertop commented on PR #11413: URL: https://github.com/apache/seatunnel/pull/11413#issuecomment-5176895860
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), mainly because it includes: * Adding the HugeGraph Source Connector * Refactoring the existing HugeGraph Sink Connector with multi-mapping support * Adding related unit tests, E2E tests, and documentation I understand that a large PR may increase the review and maintenance burden. Before proceeding, I would like to know whether the SeaTunnel community prefers this type of change to be split into multiple smaller PRs (for example, separating the source connector, sink refactoring, and tests/docs), or whether it is acceptable to review and merge it as a single PR. For more details about the design and test results, I have prepared additional documentation here: https://yqsi6vck42g.feishu.cn/wiki/RoNTwu5C3iqpHJkDgLmc1MuGnUb The document includes the architecture design, implementation details, test scenarios, and performance results. 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]
