SEZ9 commented on issue #10425: URL: https://github.com/apache/seatunnel/issues/10425#issuecomment-5747418955
Thanks @goutamadwant for picking up the OpenMLDB slice and linking #12303 here — that keeps the umbrella easy to follow. A couple of things on the OpenMLDB row before the tracker body is refreshed: - In your proposed table, OpenMldb is listed as `Done` with #12303. From this thread alone I can't tell whether #12303 has actually merged into `dev` or is still under review. Could you confirm the current state of #12303? If it is still open, the row should read as in-progress with the PR linked, and we'll flip it to Done once it lands. - Please confirm the implementation follows the same pattern as your IoTDB/InfluxDB work: `tables_configs` (not the deprecated `table_list` path), one connector in the PR, and legacy single-table config still working. - If #12303 does not already reference this umbrella issue in its description, please add the link so the cross-reference shows up on both sides. Once you confirm the merge state, I'm happy to have the OpenMLDB row updated accordingly. <!-- streview-comment:1184 --> -- 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]
