>From Peeyush Gupta <[email protected]>: Attention is currently required from: Ritik Raj. Peeyush Gupta 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 3: (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/ffe4dcc8_1525a0b3 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. I think the issue is with the paths gets reset every time or with the visitor or both. -- 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: 3 Gerrit-Owner: Ritik Raj <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-CC: Peeyush Gupta <[email protected]> Gerrit-Attention: Ritik Raj <[email protected]> Gerrit-Comment-Date: Wed, 19 Mar 2025 22:30:51 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
