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

Add the query_txid_plugin as a option #1954

Closed
wants to merge 5 commits into from

Conversation

bijianing97
Copy link

1.The pull request is the change to the #1906 ,set query_txid_plugin as an option.
You can compile the plugin by using cmake -D LOAD_QUERY_TXID_PLUGIN=ON before make and if you don't want to use it ,just cmake ..

2.I tested the impact of this plugin on replay speed.The number of the test blocks is 21000000.
Without the query_txid_plugin :

[by size: 0.02174%   1554075 of 7148588026]   [by num: 0.04744%   10000 of 21077164] localtime: 22,9:25:47
End:
[by size: 99.27695%   7096900348 of 7148588026]   [by num: 99.63390%   21000000 of 21077164] localtime: 22,12:47:17

Total time :3:21:30

With the query_txid_plugin:

[by size: 0.02187%   1554075 of 7106416350]   [by num: 0.04759%   10000 of 21013463] localtime: 21,14:14:24
End:
[by size: 99.86609%   7096900348 of 7106416350]   [by num: 99.93593%   21000000 of 21013463] localtime: 21,18:9:54

Total time : 3:55:30

Delayed by about 34 minutes

@pmconrad pmconrad changed the base branch from release to develop August 22, 2019 14:23
@pmconrad
Copy link
Contributor

Thanks for the numbers. 17% is significant but better than I'd have assumed.
Please rebase on current develop and force-push.

Can we close #1906 then? It looks broken now.

@bijianing97
Copy link
Author

yes, you can close the #1906,i will rebase on the develp if there is no problem tomorrow

@pmconrad
Copy link
Contributor

Closing in favour of #1957

@pmconrad pmconrad closed this Aug 23, 2019
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.

3 participants