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.

Sign up for GitHub

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

Jump to bottom

Lightweight partition stats #58

Merged
merged 6 commits into from
Dec 1, 2017
Merged

Lightweight partition stats #58

merged 6 commits into from
Dec 1, 2017

Conversation

db7
Copy link
Collaborator

@db7 db7 commented Nov 30, 2017

See #57

@db7 db7 requested review from SamiHiltunen and frairon November 30, 2017 10:07
frairon
frairon approved these changes Nov 30, 2017
View reviewed changes
Copy link
Contributor

@frairon frairon left a comment •
edited
Loading

Choose a reason for hiding this comment

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

looks good to me.
Do you have an idea why the one test failed?

@db7 db7 force-pushed the partition-stats branch from efb5604 to 64c7eab Compare December 1, 2017 07:39
@db7 db7 merged commit c396afb into master Dec 1, 2017
@db7 db7 deleted the partition-stats branch December 1, 2017 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@frairon frairon frairon approved these changes

@SamiHiltunen SamiHiltunen Awaiting requested review from SamiHiltunen

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@db7 @frairon