gianm commented on code in PR #20607:
URL: https://github.com/apache/druid/pull/20607#discussion_r4366264882


##########
sql/src/main/java/org/apache/druid/sql/calcite/rel/DruidQuery.java:
##########
@@ -834,7 +839,7 @@ static Pair<DataSource, Filtration> getFiltration(
 
     // Time filters within the datasource narrow the query intervals too, 
which lets the broker prune segments.
     final Filtration queryFiltration =
-        Filtration.create(filter, getBaseIntervals(dataSource, 
rowSignature)).optimize(rowSignature);
+        Filtration.create(filter, getBaseIntervals(dataSource, timeCondition, 
rowSignature)).optimize(rowSignature);

Review Comment:
   I believe this is inaccurate. I added two restricted-table tests and they 
worked as expected.



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

Reply via email to