fallintoplace opened a new pull request, #1007:
URL: https://github.com/apache/arrow-go/pull/1007

   ### Rationale for this change
   
   Floating-point statistics use the largest finite values as their initial 
min/max bounds. Those are not identity values when a column contains only 
positive or negative infinity, so the writer records a finite bound that was 
not present in the data.
   
   ### What changes are included in this PR?
   
   - Seed FLOAT, DOUBLE, and FLOAT16 statistics with positive and negative 
infinity.
   - Treat an inverted min/max pair as the empty or all-NaN sentinel.
   - Cover all-positive-infinity and all-negative-infinity inputs for all three 
types.
   
   ### Are these changes tested?
   
   Yes. The focused statistics tests, including the existing NaN and 
signed-zero coverage, pass.


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