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

release 3.1: port tiflash support from master to release3.1 #14537

Merged
merged 54 commits into from
Feb 14, 2020
Merged

release 3.1: port tiflash support from master to release3.1 #14537

merged 54 commits into from
Feb 14, 2020

Conversation

lzmhhh123
Copy link
Contributor

No description provided.

lzmhhh123 and others added 30 commits January 17, 2020 11:47
@lzmhhh123
planner: split avg to count and sum for TableReader cop task (pingcap…
49b9bb0 
…#11926)
@lzmhhh123
*: add max/avg cop response time for TableReader, IndexReader and Ind…
c32ba61 
…exLookupReader. (pingcap#12003)
@lzmhhh123
tikv: make region request can send to flash store (pingcap#11652)
07b1b29
@lzmhhh123
executor: support table reader executor read from tiflash. (pingcap#1…
7fc76d6 
…2371)
@lzmhhh123
planner: support a hint to read from tiflash in planner (pingcap#12479)
3e2b38e
@lzmhhh123
fix test
1955cb5
@wshwsh12 @lzmhhh123
planner: fix wrong request data type when pushing down avg aggfuncs (p…
7d88032 
…ingcap#12051)
@lzmhhh123
fix test
5e430b8
@lzmhhh123
planner: do not convert filter to range for table scan with tiflash (p…
cfdff94 
…ingcap#12682)
@lzmhhh123
planner: fix wrong full range for table scan in tiflash (pingcap#12756)
5216314
@lzmhhh123
tikv: retry region request when tiflash store not found (pingcap#12928)
e1e20eb
@crazycs520 @lzmhhh123
*: Support LOCK/UNLOCK TABLES feature (pingcap#10343)
bfc0481
@crazycs520 @lzmhhh123
*: add admin cleanup table lock syntax support (pingcap#10423)
4310725
@crazycs520 @lzmhhh123
ddl: fix race in table lock config (pingcap#10848)
ba7777a
@crazycs520 @lzmhhh123
lock: add delay clean table lock when session close (pingcap#11038)
4a24368
@crazycs520 @lzmhhh123
*: support the TiFlash replica of table (pingcap#12453)
4fc82fa
@eurekaka @lzmhhh123
*: refactor cost model formulas and constants (pingcap#10581)
f8657bd
@eurekaka @lzmhhh123
*: replace cost model factor constants with system variable (pingcap#…
38d99c3 
…12367)
@lzmhhh123
planner: support cost model for tiflash table scan (pingcap#12868)
507d057
@lzmhhh123
planner: fix bug cost model for tiflash do not work (pingcap#13111)
bac96d7
@lzmhhh123
planner, session: add isolation read with engine type (pingcap#12997)
67b66c7
@lzmhhh123
planner: make hint read_from_storage take effect for tikv (pingcap#…
12a9adf 
…13578)
@lzmhhh123
planner: do not push keep descending order to tiflash (pingcap#13572)
35fe5a3
@windtalker @lzmhhh123
planner: prune cop task for tiflash (pingcap#13131)
df87a51
@lzmhhh123
coprocessor: fix wrong cop task range for tiflash. (pingcap#13292)
38e586d
@lzmhhh123
tikv: do not refresh TiFlash work index when region is refreshed in c…
1cc0836 
…ache (pingcap#14205)
@lzmhhh123
planner: do not filter access paths by isolation read for `mysql.Syst…
239a94d 
…emDB` (pingcap#14309)
@lzmhhh123
planner: fix some select condition do not filter by tiflash (pingcap#…
11923c3 
…14502)
@lzmhhh123
fix test
c9c4951
@lzmhhh123
fix test
c270a98
@eurekaka
Copy link
Contributor

/rebuild

@eurekaka
Copy link
Contributor

/run-unit-test

eurekaka
eurekaka approved these changes Feb 14, 2020
View reviewed changes
Copy link
Contributor

@eurekaka eurekaka left a comment

Choose a reason for hiding this comment

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

LGTM

@eurekaka
Copy link
Contributor

/merge

@sre-bot sre-bot added the status/can-merge Indicates a PR has been approved by a committer. label Feb 14, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Feb 14, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Feb 14, 2020

@lzmhhh123 merge failed.

@lzmhhh123
Copy link
Contributor Author

/run-common-test tidb-test=pr/988

@lzmhhh123
Copy link
Contributor Author

/run-common-test tidb-test=pr/988
/run-integeration-common-test tidb-test=pr/988

@lzmhhh123
Copy link
Contributor Author

/run-common-test tidb-test=pr/988

4 similar comments
@lzmhhh123
Copy link
Contributor Author

/run-common-test tidb-test=pr/988

@lzmhhh123
Copy link
Contributor Author

/run-common-test tidb-test=pr/988

@lzmhhh123
Copy link
Contributor Author

/run-common-test tidb-test=pr/988

@lzmhhh123
Copy link
Contributor Author

/run-common-test tidb-test=pr/988

@lzmhhh123
Copy link
Contributor Author

/run-integeration-common-test tidb-test=pr/988

@lzmhhh123
Copy link
Contributor Author

/run-integration-common-test tidb-test=pr/988

@lzmhhh123
Copy link
Contributor Author

/run-common-test tidb-test=pr/988

@lzmhhh123
Copy link
Contributor Author

/run-unit-test

1 similar comment
@lzmhhh123
Copy link
Contributor Author

/run-unit-test

zz-jason
zz-jason approved these changes Feb 14, 2020
View reviewed changes
Copy link
Member

@zz-jason zz-jason left a comment

Choose a reason for hiding this comment

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

LGTM

@eurekaka eurekaka merged commit 1bea8ee into pingcap:release-3.1 Feb 14, 2020
@lzmhhh123 lzmhhh123 deleted the release-3.1/cherry-pick-tiflash branch February 14, 2020 11:29
@zimulala zimulala mentioned this pull request Feb 14, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@crazycs520 crazycs520 crazycs520 left review comments

@zz-jason zz-jason zz-jason approved these changes

@eurekaka eurekaka eurekaka approved these changes

@coocood coocood Awaiting requested review from coocood

Assignees
No one assigned
Labels
status/can-merge Indicates a PR has been approved by a committer.
Projects
None yet
Milestone
v3.1.0-rc
Development

Successfully merging this pull request may close these issues.

None yet
9 participants
@lzmhhh123 @eurekaka @sre-bot @zz-jason @crazycs520 @bb7133 @wshwsh12 @windtalker @lysu