Skip to content
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

Handle create/drop sequence job type #950

Merged
merged 2 commits into from
Apr 24, 2020
Merged

Conversation

july2993
Copy link
Contributor

@july2993 july2993 commented Apr 8, 2020

What problem does this PR solve?

handle create/drop sequence job type
filter out the special DDL binlog for replicate the value of sequence now.

before this PR: will abort the replicate once create sequence at upstream.
after this PR: the create/drop sequence dll can be replicated, but the value of the sequence WILL NOT be replicated to downstream.

What is changed and how it works?

  1. handle create/drop sequence job type
  2. ref tables: add sequence binlog support tidb#14954
    TiDB will write a fake ddl binlog like: select setval(seq.sequence_name, 1000)
    when the value of sequence is changed for replicate the value of the sequence.
    we CAN NOT query the job from the tikv according the job id.
    just skip this kind of binlog now.

Check List

Tests

  • Integration test

Code changes

Side effects

Related changes

1. handle create/drop sequence job type
2. ref pingcap/tidb#14954
TiDB will write a fake ddl binlog like: select setval(`seq`.`sequence_name`, 1000)
when the value of sequence is changed for replicate the value of the sequence.
we CAN NOT query the job from the tikv according the job id.
just skip this kind of binlog now.
Copy link
Contributor

@WangXiangUSTC WangXiangUSTC left a comment

Choose a reason for hiding this comment

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

LGTM

@WangXiangUSTC
Copy link
Contributor

/run-integration-tests

@july2993
Copy link
Contributor Author

@GregoryIan PTAL

@IANTHEREAL
Copy link
Collaborator

This implementation of the DDL really irks me, completely out of spec, a DDL, but donn't have DDL job. This implementation forces the binlog or CDC to have special dispersed logic to filter this DDL @siddontang

@sre-bot
Copy link

sre-bot commented Apr 24, 2020

/run-all-tests

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

Successfully merging this pull request may close these issues.

4 participants