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

Fix txn2 #33

Closed
wants to merge 27 commits into from
Closed

Fix txn2 #33

wants to merge 27 commits into from

Conversation

lhy1024
Copy link
Owner

@lhy1024 lhy1024 commented Dec 21, 2023

What problem does this PR solve?

Issue Number: Close #xxx

What is changed and how does it work?

Check List

Tests

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

Code changes

Side effects

  • Possible performance regression
  • Increased code complexity
  • Breaking backward compatibility

Related changes

Release note

None.

rleungx and others added 25 commits December 14, 2023 10:21
ref tikv#7418

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
… follower (tikv#7554)

ref tikv#7300

- Implement retry mechanism based on the leader and follower.
- Move method definitions into a separate file. 
- Use a sturct `requestInfo` to gather the parameters.

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
close tikv#7564, ref tikv/tikv#15909

Enable `evict-slow-store` scheduler by default.

Signed-off-by: lucasliang <[email protected]>
ref tikv#7298

Add `test-real-cluster` command to the root Makefile.

Signed-off-by: JmPotato <[email protected]>
…ikv#7565)

ref tikv#7300

When creating a client, it is required to pass in the `source` parameter to further distinguish the source of logs and requests.

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
…v#7563)

close tikv#7561

Pass TLS info to dashboard to fix TiKV heap profiling

Signed-off-by: Connor1996 <[email protected]>
ref tikv#4813

Fix some errors detected by `testifylint`.

Signed-off-by: JmPotato <[email protected]>
close tikv#7418

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
…e count (tikv#7581)

close tikv#7584

The healthy orphan peer should be the last one to be removed only if there are extra peers to keep the high availablility.

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
close tikv#7585

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
)

ref tikv#7298

Rename the real-cluster test to a more general naming.

Signed-off-by: JmPotato <[email protected]>
close tikv#7593

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ref tikv#7583

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
close tikv#7562

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
close tikv#7546

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ref tikv#7274

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ref tikv#4813

Fix all errors detected by testifylint and enable it.

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ref tikv#4813

Fix the TSO consistency test.

Signed-off-by: JmPotato <[email protected]>
ref tikv#7300

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ref tikv#7298

tests/realcluster: support local real cluster test

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Signed-off-by: lhy1024 <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Merging #33 (2a203a7) into master (f51f913) will increase coverage by 0.03%.
The diff coverage is 77.36%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
+ Coverage   74.59%   74.62%   +0.03%     
==========================================
  Files         454      457       +3     
  Lines       50142    50440     +298     
==========================================
+ Hits        37403    37642     +239     
- Misses       9409     9441      +32     
- Partials     3330     3357      +27     
Flag Coverage Δ
unittests 74.62% <77.36%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Signed-off-by: lhy1024 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.