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

speed up tinkerpop test #208

Merged
merged 1 commit into from
Nov 21, 2018
Merged

speed up tinkerpop test #208

merged 1 commit into from
Nov 21, 2018

Conversation

javeme
Copy link
Contributor

@javeme javeme commented Nov 21, 2018

how to:
1.speed up rocksdb backend by truncating tables
2.don't need to clear if the database is empty
3.don't need to clear variables if it's empty
4.don't need to commit hbase each len-prefix for delete index-label

potential problems:
1.mysql may block if create task vertex-label when truncating table
2.rocksdb may miss CF if create task vertex-label when truncating table
3.hbase may block if truncate hbase with version<2.0

improve #14

Change-Id: I4b2393aea8b0fc63c1886e984a576a1f5808b25c

@javeme javeme added the ci-cd Build or deploy label Nov 21, 2018
@javeme javeme mentioned this pull request Nov 21, 2018
@javeme javeme force-pushed the speed-up-tinkerpop-test-rebase branch from db905e4 to 52f1d3a Compare November 21, 2018 11:22
how to:
1.speed up rocksdb backend by truncating tables
2.don't need to clear if the database is empty
3.don't need to clear variables if it's empty
4.don't need to commit hbase each len-prefix for delete index-label

potential problems:
1.mysql may block if create task vertex-label when truncating table
2.rocksdb may miss CF if create task vertex-label when truncating table
3.hbase may block if truncate hbase with version<2.0

improve #14

Change-Id: I4b2393aea8b0fc63c1886e984a576a1f5808b25c
@javeme javeme force-pushed the speed-up-tinkerpop-test-rebase branch from 52f1d3a to ceac1bb Compare November 21, 2018 11:42
@zhoney zhoney merged commit 5ef25cf into master Nov 21, 2018
@zhoney zhoney deleted the speed-up-tinkerpop-test-rebase branch November 21, 2018 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-cd Build or deploy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants