Skip to content

Conversation

PatrickRen
Copy link
Contributor

This PR introduces a new feature #254 for the new MySQL CDC source, that users could specify starting point of binlog reading, including earliest offset, specific offset and timestamp.

Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@PatrickRen Thanks for the contribution, Could you fix the compile issue firstly?

@PatrickRen PatrickRen force-pushed the mysql-starting-offset branch 3 times, most recently from a3b29ca to 1446ad8 Compare October 27, 2022 04:27
Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @PatrickRen for the contribution, I left some comments. Please update the connector docs as well.

Comment on lines 34 to 49
/** The binlog offset is described by a timestamp. */
TIMESTAMP,

/**
* A specific offset described by binlog file name and position, or GTID set if GTID is enabled
* on the cluster.
*/
SPECIFIC,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can given three examples for each kind in note.

@PatrickRen PatrickRen force-pushed the mysql-starting-offset branch from ef8b512 to 27ee38c Compare October 28, 2022 08:17
@PatrickRen PatrickRen force-pushed the mysql-starting-offset branch from 150c4f7 to 5d9878f Compare November 1, 2022 13:04
@PatrickRen PatrickRen force-pushed the mysql-starting-offset branch from 5d9878f to 76aa5ad Compare November 1, 2022 13:05
Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Big thanks @PatrickRen for the update, LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants