Jackie-Jiang commented on code in PR #16021:
URL: https://github.com/apache/pinot/pull/16021#discussion_r2131348680
##########
pinot-query-runtime/src/test/java/org/apache/pinot/query/runtime/operator/SortOperatorTest.java:
##########
@@ -472,6 +473,29 @@ public void shouldHandleMultipleCollationKeysWithNulls() {
assertTrue(operator.nextBlock().isSuccess(), "expected EOS block to
propagate");
}
+ @Test
+ public void shouldPreservePrecision() {
+ // Given:
+ DataSchema schema = new DataSchema(new String[]{"sort"}, new
DataSchema.ColumnDataType[]{LONG});
+ // Insert 3 large long values which will be the same if converted to
double due to precision loss.
Review Comment:
These 3 values are the same if converted to double
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
