SEZ9 commented on issue #11936: URL: https://github.com/apache/seatunnel/issues/11936#issuecomment-5409365139
Hi @Rangsh, thanks a lot for the support and for stepping back to avoid duplicated effort — much appreciated! To answer your comment directly: yes, your help would be very welcome, and here's where things stand: - **Done:** The core MVP logic for the BOS file source and sink is built and verified, and the implementation is up in PR #11952, reusing the existing file-base framework for the Hive-on-BOS use case. @DanielLeens has confirmed #11952 as the current implementation path for this issue. - **In progress:** I'm resolving the existing review comments, including the SDK dependency choice and the hard-coded string problems. Concrete asks for you: 1. Please review PR #11952 when you have time — extra eyes on the dependency choice and the thin-wrapper pattern over the file connector base would help. 2. Testing the BOS-specific configurations against the PR would be very valuable, since that's hard for me to cover exhaustively alone. 3. If you hit any Hive-on-BOS edge cases during testing, please add them directly to the PR discussion (as @DanielLeens suggested) so they stay tied to the code path. Looking forward to your feedback on the PR! <!-- streview-comment:534 --> -- 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]
