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

Cherry-pick 4 PRs related to tikv-importer to release-2.1 #4369

Merged
merged 4 commits into from
Mar 14, 2019

Conversation

kennytm
Copy link
Contributor

@kennytm kennytm commented Mar 13, 2019

Signed-off-by: Lonng chris@lonng.org
Signed-off-by: kennytm kennytm@gmail.com

What have you changed? (mandatory)

Cherry-picks the following 4 PRs for 2.1.6 release:

What are the type of the changes? (mandatory)

  • Improvement (change which is an improvement to an existing feature)
  • Bug fix (change which fixes an issue)

How has this PR been tested? (mandatory)

Tested manually using a data set with size 6.5 TB

Does this PR affect documentation (docs) update? (mandatory)

No

Does this PR affect tidb-ansible update? (mandatory)

Yes (will be filed separately, together with Lightning changes)

Refer to a related PR or issue link (optional)

See above

Benchmark result if necessary (optional)

Add a few positive/negative examples (optional)

This makes the SST size to upload larger, which reduces the number of SST
files needed to ingested, and improve overall speed.

Signed-off-by: kennytm <kennytm@gmail.com>
…ikv#4348)

* import: store the intermediate SST files on disk instead of memory

The bottle-neck of Importer is in SST ingestion on the TiKV side, not
disk I/O. Storing these SST in RAM will just increase the chance Importer
getting killed by OOM.

Signed-off-by: kennytm <kennytm@gmail.com>
* import: put index and filter in block cache to restrict memory usage

Signed-off-by: Lonng <chris@lonng.org>
Signed-off-by: kennytm <kennytm@gmail.com>
Signed-off-by: kennytm <kennytm@gmail.com>
@kennytm kennytm force-pushed the importer-cherry-pick-batch-1 branch from 3cbc72e to b207a99 Compare March 13, 2019 13:19
@kennytm
Copy link
Contributor Author

kennytm commented Mar 13, 2019

PTAL @overvenus @huachaohuang

Copy link
Member

@overvenus overvenus left a comment

Choose a reason for hiding this comment

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

👍

@AndreMouche AndreMouche added type/cherry-pick Type: PR - Cherry pick status/LGT1 Indicates that a PR has LGTM 1. labels Mar 14, 2019
@huachaohuang huachaohuang merged commit 229c423 into tikv:release-2.1 Mar 14, 2019
@kennytm kennytm deleted the importer-cherry-pick-batch-1 branch March 14, 2019 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/LGT1 Indicates that a PR has LGTM 1. type/cherry-pick Type: PR - Cherry pick
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants