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.

Sign up for GitHub

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

Jump to bottom

br: batch ddl for binlog #31028

Merged
merged 577 commits into from
Dec 26, 2021
Merged

br: batch ddl for binlog #31028

merged 577 commits into from
Dec 26, 2021

Conversation

fengou1
Copy link
Contributor

@fengou1 fengou1 commented Dec 26, 2021

What problem does this PR solve?

this if feature development pr

Please create an issue first to describe the problem.

There MUST be one line starting with "Issue Number: " and
linking the relevant issues via the "close" or "ref".

For more info, check https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/contribute-code.html#referring-to-an-issue.

-->

Issue Number: close #xxx

Problem Summary:

What is changed and how it works?

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Side effects

  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Breaking backward compatibility

Documentation

  • Affects user behaviors
  • Contains syntax changes
  • Contains variable changes
  • Contains experimental features
  • Changes MySQL compatibility

Release note

None

hawkingrei and others added 30 commits November 23, 2021 13:37
@hawkingrei
executor: migrate test-infra to testify for executor/show_stats_test.…
805a887 
…go (pingcap#30000)
@wangggong
ddl/table_split_test.go: migrate test-infra to testify (pingcap#29960)
2f119e3
@lcwangchao
*: fix unstable test caused by TestRandomPanicAggConsume (pingcap#30035)
e030ff1
@clovis1122
ddl/ddl_test.go: refactor testSchemaInfo to return error pingcap#29964 (
efd6c5e 
pingcap#30008)
@hawkingrei
executor: remove useless parameter (pingcap#30043)
759aab5
@unconsolable
executor: fix error msg of granting non-table level privilege (pingca…
e453873 
…p#29321)
@zgcbj
server: fix bug https://asktug.com/t/topic/213082/11 (pingcap#29577)
0f86a53
@wjhuang2016
*: replace compareDatum by compare (pingcap#30048)
fb5eb1f
@wjhuang2016
executor, store: replace compareDatum by compare (pingcap#30044)
205d9ab
@hawkingrei
executor: migrate test-infra to testify for executor/write_test.go (p…
bba5fee 
…ingcap#29953)
@mmyj
planner: add sub plan info of shuffleReceiver when query explain anal…
7fdafb4 
…yze (pingcap#27992)
@wshwsh12
expression: fix wrong flen for CastAsString funtion (pingcap#29563)
8858592
@tisonkun
br: migrate test-infra to testify for redact (pingcap#29747)
cc8617e
@wjhuang2016
ddl: stable create expression index test (pingcap#29990)
d252fd1
@wjhuang2016
*: replace compareDatum by compare (pingcap#30060)
8b028b8
@qw4990
planner: fix the issue that binding cannot work when sql_select_limit…
18381f9 
… is enabled (pingcap#29789)
@JayLZhou
br : migrate test-infra to testify for lightning/backend/tidb/tidb_te…
f7ff198 
…st.go (pingcap#30042)
@lcwangchao
*: fix unstable test in placement gc cases (pingcap#30045)
9535f05
@glorv
lightning: let ignore columns be compatible with tidb backend (pingca…
c687915 
…p#27850)
@wjhuang2016
statistics: replace compareDatum by compare (pingcap#30052)
4051cf4
@tisonkun
br/pkg/mock: migrate test-infra to testify (pingcap#30034)
71e64bf
@sylzd
ddl: forbit alter table cache in system db (pingcap#29998)
b57c1e1
@mornyx
topsql: distinguish the row and index operation type (pingcap#29044)
b11171a
@tangenta
ddl, util/codec: fix add index OOM and prevent panic in logging (ping…
d04a128 
…cap#29925)
@sleepymole
br/pkg/utils: migrate tests to testify (pingcap#30032)
791f59d
@Defined2014
expression, sessionctx: support rand_seed1 and rand_seed2 sysvar (pin…
45836a6 
…gcap#29936)
@XuHuaiyu
*: track the memory usage of IndexJoin more accurate (pingcap#29068)
4814557
@JayLZhou
br : migrate test-infra to testify for lightning/backend/importer (pi…
f45a8f0 
…ngcap#30073)
@Yisaer
planner: fix panic when the join key is scalarFunction (pingcap#30002)
05ccb8a
@sylzd
ddl: fix none info of alter table placement (pingcap#29929)
a90c988
xhebox and others added 21 commits December 15, 2021 20:42
@xhebox
ddl: add unit tests
73a7b8a 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: fix fmt
dee6c35 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox @AilinKid
ddl: typo
5b46f1b 
Co-authored-by: Arenatlx <ailinsilence4@gmail.com>
@xhebox
ddl: fix tests
88a0e3f 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: rename to BatchCreateTableWithInfo
5c50048 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: trace the error
d35fc31 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: comments
f43fa0f 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: cancle the job right
7348348 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: cancel the job right 2
ad5812e 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: report error if entry too large
1b70b8b 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: report error when table is duplicated
05aa57c 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: go fmt
e4582a7 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
infoschema: improve batch memory perf
7913c49 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: retain ID
84bdeaf 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: reduce log frequency
bd0b779 
Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox
ddl: fix tests
c5e7cb8 
Signed-off-by: xhe <xw897002528@gmail.com>
@fengou1
fix conflict
497bc0f
@fengou1
fix merge issue
ced6a21
@fengou1
fix issue that loss table restore
501fc52
@fengou1
refactoring code
17bd4dd
@fengou1
refactoring code
4115494
@ti-chi-bot
Copy link
Member

[REVIEW NOTIFICATION]

This pull request has not been approved.

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

@fengou1 fengou1 requested a review from a team as a code owner December 26, 2021 13:41
@ti-chi-bot ti-chi-bot added release-note-none Denotes a PR that doesn't merit a release note. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Dec 26, 2021
@ti-chi-bot ti-chi-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 26, 2021
@fengou1 fengou1 merged commit 6fa8234 into pingcap:br_ddl Dec 26, 2021
@fengou1 fengou1 deleted the merge_ddl branch December 12, 2022 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
release-note-none Denotes a PR that doesn't merit a release note. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.