alamb commented on PR #7871:
URL: https://github.com/apache/arrow-rs/pull/7871#issuecomment-3038892297

   > ... at least, confident we caught all the ones in code paths this specific 
unit test exercises. But it's tricky -- I tried to make a visual sweep of all 
`XXX_with_shallow_validation` methods and didn't notice anything else. But then 
I re-ran the unit test "benchmark" and saw that it was still oddly slow. Found 
the third missed site during a second, more motivated, code inspection pass.
   
   Can we make your test into an actual benchmark (aka something like 
https://github.com/apache/arrow-rs/blob/main/parquet-variant/benches/variant_builder.rs
 )?


-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to