Skip to content

feat(col.bool): improve performance#13

Merged
ernado merged 1 commit intomainfrom
feat/col-bool-speedup
Jan 2, 2022
Merged

feat(col.bool): improve performance#13
ernado merged 1 commit intomainfrom
feat/col-bool-speedup

Conversation

@ernado
Copy link
Copy Markdown
Collaborator

@ernado ernado commented Jan 2, 2022

name                     old time/op    new time/op    delta
ColBool_DecodeColumn-32    23.7µs ± 1%    23.2µs ± 2%     ~     (p=0.095 n=5+5)
ColBool_EncodeColumn-32    28.0µs ± 2%    21.9µs ± 2%  -21.72%  (p=0.008 n=5+5)

name                     old speed      new speed      delta
ColBool_DecodeColumn-32  2.11GB/s ± 1%  2.16GB/s ± 2%     ~     (p=0.095 n=5+5)
ColBool_EncodeColumn-32  1.79GB/s ± 2%  2.28GB/s ± 2%  +27.73%  (p=0.008 n=5+5)

kudos to @cristaloleg

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 2, 2022

Codecov Report

Merging #13 (787892f) into main (8706ec0) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #13   +/-   ##
=======================================
  Coverage   79.56%   79.56%           
=======================================
  Files         105      105           
  Lines        4242     4242           
=======================================
  Hits         3375     3375           
  Misses        699      699           
  Partials      168      168           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8706ec0...787892f. Read the comment docs.

@ernado ernado merged commit dedf126 into main Jan 2, 2022
@ernado ernado deleted the feat/col-bool-speedup branch January 2, 2022 14:30
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.

2 participants