>From Ritik <[email protected]>:

Attention is currently required from: Peeyush Gupta.
Ritik has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19529 )

Change subject: [ASTERIXDB-3582][COMP] Restricting to one unnest filter 
pushdown for a datasource
......................................................................


Patch Set 4:

(1 comment)

File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/pushdown/processor/ColumnFilterPushdownProcessor.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19529/comment/db79341d_676382c8
PS3, Line 151:         if 
(checkerVisitor.containsMultipleArrayPaths(paths.values())) {
             :             // Cannot pushdown a filter with multiple unnest
             :             // TODO allow rewindable column readers for filters
             :             // TODO this is a bit conservative (maybe too 
conservative) as we can push part of expression down
             :             return;
             :         }
> There seems to be a check already to avoid pushing down filter with multiple 
> unnest. […]
Done



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19529
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: ionic
Gerrit-Change-Id: I4f335db4c7c455c1276ea5be9b14ce1e8ee92d74
Gerrit-Change-Number: 19529
Gerrit-PatchSet: 4
Gerrit-Owner: Ritik <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-CC: Peeyush Gupta <[email protected]>
Gerrit-Attention: Peeyush Gupta <[email protected]>
Gerrit-Comment-Date: Sat, 22 Mar 2025 02:38:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Peeyush Gupta <[email protected]>
Gerrit-MessageType: comment

Reply via email to