Skip to content

HLL: More avoiding allocations#3321

Closed
navis wants to merge 1 commit intoapache:masterfrom
navis:hll-chill-out2
Closed

HLL: More avoiding allocations#3321
navis wants to merge 1 commit intoapache:masterfrom
navis:hll-chill-out2

Conversation

@navis
Copy link
Copy Markdown
Contributor

@navis navis commented Aug 4, 2016

For #3314 (comment).

Realized it's really a trivial now, but here it is.

@drcrallen
Copy link
Copy Markdown
Contributor

@navis cool, are you able to get basic benchmarks for this?

@drcrallen drcrallen added this to the 0.9.2 milestone Aug 4, 2016
@navis
Copy link
Copy Markdown
Contributor Author

navis commented Aug 8, 2016

@drcrallen I did some tests and found it's barely meet the situation (swap and fold). The numbers are mostly identical. If we do some parallel folding it would be a little helpful but currently it seemed not useful. Can we close this?

@drcrallen
Copy link
Copy Markdown
Contributor

@navis I would rather not add complexity unless it has specific results. The results don't have to be much (5% improvement is totally worth it), but they should be measurable.

@drcrallen
Copy link
Copy Markdown
Contributor

@navis I'm going to remove milestone until you figure out if its reasonable to get benchmarks on it or not.

@drcrallen drcrallen removed this from the 0.9.2 milestone Aug 10, 2016
@navis
Copy link
Copy Markdown
Contributor Author

navis commented Aug 12, 2016

@drcrallen I can only see one or two swapping case from 1000+ rows merging and seemed not possible to make any improvement on performance under varied data / conditions. I can definitely say that we can ignore this.

@navis navis closed this Aug 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants