You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu> on 2026/04/26 14:56:57 UTC

[M] Change in asterixdb[lumina]: [ASTERIXDB-3761][COMP] Replace meta() inside subplans

From Ali Alsuliman <al...@gmail.com>:

Attention is currently required from: Ali Alsuliman.

Hello Anon. E. Moose #1000171, Jenkins, 

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21163?usp=email

to look at the new patch set (#2).

The following approvals got outdated and were removed:
Contrib-2 by Anon. E. Moose #1000171, Integration-Tests-1 by Jenkins


Change subject: [ASTERIXDB-3761][COMP] Replace meta() inside subplans
......................................................................

[ASTERIXDB-3761][COMP] Replace meta() inside subplans

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- Propagate meta() transformer from the outer plan to inside the subplan to be able to replace meta() references with meta variable.
- Don't generate a job spec for a created function. When a function contains references to meta(), trying to generate a job spec will fail because there is no data-scan yet in a function definition. Resolving meta() will be done when the function is actually used in a query.

Ext-ref: MB-69901

Change-Id: I25b7972b6f4a7602cdecdfa6f055bf7ad41b51c7
---
M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/MetaFunctionToMetaVariableRule.java
M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/SessionConfig.java
M asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A asterixdb/asterix-app/src/test/resources/optimizerts/queries/meta/meta_in_subplan_1.sqlpp
A asterixdb/asterix-app/src/test/resources/optimizerts/results/meta/meta_in_subplan_1.plan
M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/meta/meta_after_gby/meta_after_gby.4.query.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/results/meta/meta_after_gby/meta_after_gby.4.adm
M asterixdb/asterix-app/src/test/resources/runtimets/sqlpp_queries.xml
8 files changed, 144 insertions(+), 9 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/63/21163/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21163?usp=email
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: lumina
Gerrit-Change-Id: I25b7972b6f4a7602cdecdfa6f055bf7ad41b51c7
Gerrit-Change-Number: 21163
Gerrit-PatchSet: 2
Gerrit-Owner: Ali Alsuliman <al...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Attention: Ali Alsuliman <al...@gmail.com>