-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[oceanbase] Use global timestamp to start log client #2565
Conversation
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.
Thanks @whhe for the contribution, LGTM, feel free to merge
* use global timestamp to start log client * remove redundant fields * update log info
In oceanbase-ce:3.1.4, SELECT TS_VALUE FROM oceanbase.V$OB_TIMESTAMP_SERVICE return microsecond rather than nanosecond In oceanbase-ce:4.0.0.0, oceanbase.V$OB_TIMESTAMP_SERVICE or SYS.V$OB_TIMESTAMP_SERVICE does not exist Is this code only test on enterprise edition? |
Thanks for your reply. The target version is 4.2.1, and you are right that 3.x and 4.x use different precision in clock value. I will add a fallback process for it as soon as possible. |
* use global timestamp to start log client * remove redundant fields * update log info
This pr changed the startup order of snapshot and change events reading, and it will reduce the risk of change events buffer OOM.