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

Update proxy for batch-system optimization #1844

Merged
merged 1 commit into from
Apr 30, 2021
Merged

Update proxy for batch-system optimization #1844

merged 1 commit into from
Apr 30, 2021

Conversation

solotzg
Copy link
Contributor

@solotzg solotzg commented Apr 28, 2021 •
edited
Loading

Signed-off-by: Zhigao Tong [email protected]

What problem does this PR solve?

Problem Summary:

What is changed and how it works?

What's Changed:

  • Revert "snap: use zstd to compress the snapshot (tikv#9947)", because we need to decode sst faster.
  • Remove useless resolved ts worker
  • Merge tikv master for batch-system optimization

Related changes

  • PR to update pingcap/docs/pingcap/docs-cn:
  • Need to cherry-pick to the release branch:

Check List

Tests

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

Release note

  • No release note

@solotzg solotzg self-assigned this Apr 28, 2021
@solotzg
Copy link
Contributor Author

solotzg commented Apr 28, 2021

/run-all-tests

@solotzg solotzg requested a review from birdstorm April 29, 2021 02:23
birdstorm
birdstorm approved these changes Apr 30, 2021
View reviewed changes
Copy link
Contributor

@birdstorm birdstorm left a comment

Choose a reason for hiding this comment

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

lgtm

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Apr 30, 2021
@solotzg solotzg merged commit 269b91e into pingcap:master Apr 30, 2021
@solotzg solotzg deleted the update-proxy branch April 30, 2021 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@birdstorm birdstorm birdstorm approved these changes

Assignees

@solotzg solotzg

Labels
status/LGT1 Indicates that a PR has LGTM 1.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@solotzg @birdstorm @ti-srebot