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 2022/10/02 22:46:25 UTC

Change in asterixdb[master]: [ASTERIXDB-3059][SQL] Support for SELECT .. EXCLUDE ..

From Glenn Galvizo <gg...@uci.edu>:

Attention is currently required from: Glenn Galvizo.
Hello Jenkins, Anon. E. Moose #1000171, 

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

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17103

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

Change subject: [ASTERIXDB-3059][SQL] Support for SELECT .. EXCLUDE ..
......................................................................

[ASTERIXDB-3059][SQL] Support for SELECT .. EXCLUDE ..

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

details:
- adding syntactic sugar rewrite to specify what fields to
  EXCLUDE in a SELECT clause
- fixing result type computer for OBJECT_REMOVE_FIELDS.

Change-Id: Ib076d130d89077964396fcbf51602488c9e90682
---
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/select-star/select-exclude/select-exclude.2.update.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/results/select-star/select-exclude/select-exclude.4.adm
M asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/RecordRemoveFieldsTypeComputer.java
M asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/DeepCopyVisitor.java
M asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/clause/SelectClause.java
M asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppCloneAndSubstituteVariablesVisitor.java
A asterixdb/asterix-app/src/test/resources/runtimets/results/select-star/select-exclude/select-exclude.6.adm
M asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/SqlppQueryRewriter.java
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/select-star/select-exclude/select-exclude.4.query.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/select-star/select-exclude/select-exclude.5.query.sqlpp
M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/sqlpp/ParserTestExecutor.java
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/select-star/select-exclude/select-exclude.3.query.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/select-star/select-exclude/select-exclude.1.ddl.sqlpp
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
A asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/visitor/SelectExcludeRewriteSugarVisitor.java
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/select-star/select-exclude/select-exclude.6.query.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/results/select-star/select-exclude/select-exclude.3.adm
M asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/SqlppFunctionBodyRewriter.java
A asterixdb/asterix-app/src/test/resources/runtimets/results/select-star/select-exclude/select-exclude.5.adm
20 files changed, 586 insertions(+), 11 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/03/17103/7
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17103
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Ib076d130d89077964396fcbf51602488c9e90682
Gerrit-Change-Number: 17103
Gerrit-PatchSet: 7
Gerrit-Owner: Glenn Galvizo <gg...@uci.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Attention: Glenn Galvizo <gg...@uci.edu>
Gerrit-MessageType: newpatchset