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

Improve performance of markets #1608

Merged
merged 1 commit into from Feb 22, 2019

Conversation

Projects
4 participants
@nathanhourt
Copy link
Contributor

commented Feb 22, 2019

Always check whether a change is required before entering
database::modify() and do not call modify() if no change is necessary.

Improve performance of markets
Always check whether a change is required *before* entering
database::modify() and do not call modify() if no change is necessary.
@nathanhourt

This comment has been minimized.

Copy link
Contributor Author

commented Feb 22, 2019

Note: this is not a hardfork. :)

@pmconrad
Copy link
Contributor

left a comment

Nice find.

@oxarbitrage oxarbitrage merged commit bf60ddd into bitshares:develop Feb 22, 2019

1 of 2 checks passed

ci/dockercloud Your tests failed in Docker Cloud
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@pmconrad

This comment has been minimized.

Copy link
Contributor

commented Feb 22, 2019

Part of #982

@nathanhourt nathanhourt deleted the nathanhourt:market-performance branch Feb 22, 2019

@abitmore abitmore added this to In development in Feature Release (3.1.0) via automation Feb 27, 2019

@ryanRfox ryanRfox moved this from In development to Done in Feature Release (3.1.0) Feb 28, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.