kszucs commented on pull request #8088:
URL: https://github.com/apache/arrow/pull/8088#issuecomment-684881898
Just executed the ASV benchmarks on this PR and the master branch with the
two sample sizes and the results showed slighty better performance in both
cases:
```
==================== ========= =========
type PR master
-------------------- --------- ---------
int32 136±0ms 138±0ms
uint32 129±0ms 133±0ms
int64 131±0ms 138±0ms
uint64 131±0ms 137±0ms
float32 131±0ms 131±0ms
float64 130±0ms 132±0ms
bool 128±0ms 131±0ms
decimal 232±0ms 227±0ms
binary 149±0ms 180±0ms
binary10 141±0ms 151±0ms
ascii 158±0ms 160±0ms
unicode 300±0ms 319±0ms
int64 list 310±0ms 307±0ms
struct 224±0ms 241±0ms
struct from tuples 231±0ms 230±0ms
==================== ========= =========
```
Also compared the `libarrow_python` binary size:
master: 2.7M
this PR: 2.2M
So far there is no performance degradation - if we can trust the ASV
benchmarks.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
