jnturton commented on a change in pull request #2432:
URL: https://github.com/apache/drill/pull/2432#discussion_r795058531
##########
File path:
exec/java-exec/src/test/java/org/apache/drill/exec/sql/TestAnalyze.java
##########
@@ -492,7 +492,7 @@ public void testHistogramWithDataTypes1() throws Exception {
.match();
query = "select 1 from dfs.tmp.employee1 where store_id < 15";
- String[] expectedPlan2 = {"Filter\\(condition.*\\).*rowcount = 676.*,.*",
+ String[] expectedPlan2 = {"Filter\\(condition.*\\).*rowcount = 699.*,.*",
"Scan.*columns=\\[`store_id`\\].*rowcount = 1128.0.*"};
Review comment:
@vdiravka yes, please see the comments on this PR from @tdunning. The
t-digest data structure changed a bit in 3.3 and estimates like this rowcount =
676 are expected to be a bit more accurate.
--
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]
