okumin commented on PR #5444:
URL: https://github.com/apache/hive/pull/5444#issuecomment-2380584073
I found the diff of stats_histogram.q would disappear with this change.
There could be some more changes on datasketches-java side.
```
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/ddl/ShowUtils.java
b/ql/src/java/org/apache/hadoop/hive/ql/ddl/ShowUtils.java
index 0d52c85289..ec197a3264 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/ddl/ShowUtils.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/ddl/ShowUtils.java
@@ -23,6 +23,7 @@
import org.apache.commons.lang3.StringUtils;
import org.apache.datasketches.kll.KllFloatsSketch;
import org.apache.datasketches.memory.Memory;
+import org.apache.datasketches.quantilescommon.QuantileSearchCriteria;
import org.apache.hadoop.fs.FSDataOutputStream;
import org.apache.hadoop.fs.FileSystem;
import org.apache.hadoop.fs.Path;
@@ -257,7 +258,7 @@ private static String convertHistogram(byte[] buffer,
ColumnStatisticsData._Fiel
final KllFloatsSketch kll =
KllFloatsSketch.heapify(Memory.wrap(buffer));
// to keep the visualization compact, we print only the quartiles (Q1,
Q2 and Q3),
// as min and max are displayed as separate statistics already
- final float[] quantiles = kll.getQuantiles(new double[]{ 0.25, 0.5,
0.75 });
+ final float[] quantiles = kll.getQuantiles(new double[]{ 0.25, 0.5,
0.75 }, QuantileSearchCriteria.EXCLUSIVE);
```
--
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]
