Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

update to iavl 0.11.0 #2281

Merged
merged 1 commit into from
Sep 9, 2018
Merged

update to iavl 0.11.0 #2281

merged 1 commit into from
Sep 9, 2018

Conversation

ValarDragon
Copy link
Contributor

This has a measurable state machine performance improvement (2% at minimum according to simulation). I think we should try to get this into the next SDK release.

  • Targeted PR against correct branch (see CONTRIBUTING.md)

  • Added entries in PENDING.md with issue #

  • rereviewed Files changed in the github PR explorer


For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@codecov
Copy link

codecov bot commented Sep 8, 2018

Codecov Report

Merging #2281 into develop will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           develop    #2281   +/-   ##
========================================
  Coverage    63.83%   63.83%           
========================================
  Files          140      140           
  Lines         8666     8666           
========================================
  Hits          5532     5532           
  Misses        2754     2754           
  Partials       380      380

Copy link
Contributor

@liamsi liamsi left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@cwgoes cwgoes left a comment

Choose a reason for hiding this comment

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

utACK

@cwgoes cwgoes merged commit 43ccbd4 into develop Sep 9, 2018
@cwgoes cwgoes deleted the dev/update_iavl branch September 9, 2018 15:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants