-
Notifications
You must be signed in to change notification settings - Fork 608
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(cdc): support INCLUDE TIMESTAMP for MySQL, PG and MongoDB cdc table #16833
Conversation
…bs/risingwave into siyuan/cdc-metadata-columns
thanks @StrikeW As per the original user request confirmed by @lmatz,
I am thinking if the two "static" fields, |
Good point. As long as they are persisted as part of the source/table catalog, it could be supported similar to However, why can't the user just write |
I have a comment for other metadata. |
6ad4ff5
to
41c3a49
Compare
755b6a6
to
d4c15d3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The impl general LGTM. Please also update the release doc part.
I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
INCLUDE TIMESTAMP [AS]
for MySQL, PG and MongoDB cdc table to ingest upstream commit timestamp.1970-01-01 00:00:00+00:00
, aligned with Flink.Example:
close: #16359
related: #16654
fix: #16850
Checklist
./risedev check
(or alias,./risedev c
)Documentation
Release note
INCLUDE TIMESTAMP [AS]
for MySQL, PG and MongoDB cdc table to ingest upstream commit timestamp.1970-01-01 00:00:00+00:00