FrankChen021 commented on code in PR #19768:
URL: https://github.com/apache/druid/pull/19768#discussion_r3783670430
##########
processing/src/main/java/org/apache/druid/math/expr/ExpressionProcessing.java:
##########
@@ -45,19 +45,19 @@ public class ExpressionProcessing
@VisibleForTesting
public static void initializeForTests()
{
- INSTANCE = new ExpressionProcessingConfig(null, null, null, null);
+ INSTANCE = new ExpressionProcessingConfig(null, null, null, null, null);
}
@VisibleForTesting
public static void initializeForHomogenizeNullMultiValueStrings()
{
- INSTANCE = new ExpressionProcessingConfig(null, true, null, null);
+ INSTANCE = new ExpressionProcessingConfig(null, true, null, null, null);
}
@VisibleForTesting
public static void initializeForVectorApiTests()
{
- INSTANCE = new ExpressionProcessingConfig(null, null, null, true);
+ INSTANCE = new ExpressionProcessingConfig(null, null, null, true, true);
Review Comment:
[P1] Keep exact vector-math tests from enabling approximate SIMD
`initializeForVectorApiTests()` enables approximate math SIMD, but the
inherited exact-comparison suite uses `Assert.assertEquals` for functions such
as `sin` and `exp`. This PR documents VO_MATHLIB ULP differences and
tier-transition changes, so supported JDK/hardware combinations can now fail
these tests. Keep this helper's math flag off, or make the consistency
assertions tolerance-aware.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
