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

Support on_empty_cmd observer in Proxy #121

Merged
merged 24 commits into from
Jul 14, 2022
Merged

Support on_empty_cmd observer in Proxy #121

merged 24 commits into from
Jul 14, 2022

Conversation

CalvinNeo
Copy link
Member

What problem does this PR solve?

Issue Number: ref pingcap/tiflash#5170

Problem Summary:
Support on_empty_cmd in Proxy

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

CalvinNeo and others added 23 commits July 4, 2022 21:48
@CalvinNeo
f
0b7bf7f 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
fix
fea5a4a 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
add engine_tiflash with this version of engine_rocks
67c47da 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
fmt
c96c0c3 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
Merge remote-tracking branch 'upstream/raftstore-proxy-6.2' into try-…
a3d6722 
…add-new-mock-engine-store-in-6.2
@CalvinNeo
add engine_tiflash actually
e3a716b 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
fix
9bfec36 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
fix some tests
5afb804 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
f
0f38d9d 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
it runs if we do_write everytime
ed1708b 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
add ensure_no_common_unrecognized_keys to config checker
8ab1acb 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
reorg tests
5237c17 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo @ti-chi-bot
raftstore: Implement coprocessor observer on_empty_cmd (tikv#12851)
4d4c583 
ref tikv#12849

Support new observers on_empty_cmd.

Signed-off-by: CalvinNeo <[email protected]>

Co-authored-by: Ti Chi Robot <[email protected]>
@CalvinNeo
fix tests
b147018 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
move proxy-related code in component/server into proxy_server
3d50bcc 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
Merge remote-tracking branch 'upstream/raftstore-proxy-6.2' into try-…
8833251 
…add-new-mock-engine-store-in-6.2
@CalvinNeo
move codes in component/server/src/setup.rs into proxy_server
f0ad93d 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
support new ffis in mock-engine-store and new-mock-engine-store
453eb87 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
add disclaimer
573efc6 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
enlength some test time
af8b448 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
f
2066ee7 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo
Merge remote-tracking branch 'upstream/raftstore-proxy-6.2' into migr…
7e27c5b 
…ate-empty-cmd
@CalvinNeo
fmt
f016ce6 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo CalvinNeo changed the title Migrate empty cmd Support on_empty_cmd observer in Proxy Jul 14, 2022
@CalvinNeo
fix tests
13e69dd 
Signed-off-by: CalvinNeo <[email protected]>
@CalvinNeo CalvinNeo merged commit fbd8e6f into pingcap:raftstore-proxy-6.2 Jul 14, 2022
@CalvinNeo CalvinNeo mentioned this pull request Aug 26, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lidezhu lidezhu lidezhu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@CalvinNeo @lidezhu