carloea2 opened a new issue, #4144:
URL: https://github.com/apache/texera/issues/4144

   ### What happened?
   
   In `AggregationOperation.scala`, the helper `minValue` uses 
`Double.MIN_VALUE` for `AttributeType.DOUBLE`. In Java/Scala, 
`Double.MIN_VALUE` is the smallest **positive** non-zero double, not the 
most-negative value. This makes the “minimum seed” for DOUBLE aggregations 
incorrect when the intended behavior is to represent the lowest possible value 
(per discussion #4049, the intent is **negative infinity**).
   
   ### How to reproduce?
   
   1. Open 
`texera/common/workflow-operator/src/main/scala/org/apache/amber/operator/aggregate/AggregationOperation.scala`.
   2. Locate `minValue(attributeType: AttributeType)` (around the `case 
AttributeType.DOUBLE` branch).
   3. Note it returns `java.lang.Double.MIN_VALUE`.
   4. Expected: the value should be `Double.NEGATIVE_INFINITY`.
   
   ### Version
   
   1.1.0-incubating (Pre-release/Master)
   
   ### Commit Hash (Optional)
   
   _No response_
   
   ### What browsers are you seeing the problem on?
   
   _No response_
   
   ### Relevant log output
   
   ```shell
   
   ```


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