adriangb commented on PR #9628:
URL: https://github.com/apache/arrow-rs/pull/9628#issuecomment-4173808100

   @emkornfield sorry for outdating your review. I removed a couple functions 
that you referenced (some you pointed out as being unused, I pushed some half 
finished commits).
   
   I've rewritten to use an analytical fpp estimate that avoids the cost of 
more expensive fpp calculations or calculating the fpp with every fold. I added 
benchmark numbers to the PR description.


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