SEZ9 commented on issue #10753: URL: https://github.com/apache/seatunnel/issues/10753#issuecomment-6817961614
@SEPURI-SAI-KRISHNA thanks for the heads-up that #13166 is open for the JMS / AMQP (generic) row. Noted that it is slice 1 (the queue source), with the sink to follow in a separate PR. I have not reviewed it yet, so I am not stating anything about its state here; I will leave review comments on the PR itself. The tracker row can be recorded as claimed by you, with #13078 as the design note and #13166 as the implementation lane for the source slice. Two asks so the review goes smoothly: - In the PR description, please link #13078 and spell out which of the four open questions from the note (scope, naming, vendor-jar location, slice order) were settled and how, so reviewers do not have to reconstruct that from the design thread. - Please keep implementation and test discussion on #13166 and design discussion on #13078, so this tracker stays a status list. @goutamadwant thanks for opening the Mixpanel Source (#13093) and Mailchimp Source (#13094) PRs. I will take a look and leave any comments directly on those PRs; please post here only when their status changes so the tracker rows can be updated. <!-- streview-comment:2291 --> -- 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]
