rok commented on PR #34323:
URL: https://github.com/apache/arrow/pull/34323#issuecomment-1443328398

   You are currently test strings of length 0 - 1024 with uniform length 
distribution. Maybe longer ones could be interesting, or maybe a range from say 
4 - 20 or just all the of length 8. Or maybe a mix of 95% uniform distribution 
of  0 - 10 and 5% uniform distribution of 500 - 1000.  Perhaps you can try 
locally if you get interesting results and only include different benchmarks.
   
   > By the way, I think it's trickey that, DELTA_LENGTH would be powerful when 
compression is enabled. However, currently we don't test compression during 
encoding benchmark
   
   Maybe that's worth opening another issue?


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to