Hello Anon. E. Moose (1000171), Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/3436
to look at the new patch set (#2).
Change subject: [NO ISSUE][COMP] Do not inline non-functional LET clauses
......................................................................
[NO ISSUE][COMP] Do not inline non-functional LET clauses
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- When inlining LET clauses in LET ... SELECT ...
skip those that contain non-functional expressions
Change-Id: I3740745e2e8c8a0bb11aa7908e68c682dd9a5553
---
M
asterixdb/asterix-app/src/test/resources/optimizerts/results/query-issue562_ps.plan
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/misc/random/random.2.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/misc/random/random.2.adm
M
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/util/FunctionUtil.java
M
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/visitor/base/AbstractQueryExpressionVisitor.java
M
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/SqlppQueryRewriter.java
M
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/visitor/InlineWithExpressionVisitor.java
A
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/CheckNonFunctionalExpressionVisitor.java
A
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/base/AbstractSqlppContainsExpressionVisitor.java
M
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/ExternalFunctionInfo.java
10 files changed, 454 insertions(+), 25 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/36/3436/2
--
To view, visit https://asterix-gerrit.ics.uci.edu/3436
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3740745e2e8c8a0bb11aa7908e68c682dd9a5553
Gerrit-Change-Number: 3436
Gerrit-PatchSet: 2
Gerrit-Owner: Dmitry Lychagin <dmitry.lycha...@couchbase.com>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
