Hello Jenkins,

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

    https://asterix-gerrit.ics.uci.edu/789

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

Change subject: ASTERIXDB-1381: InlineColumnAliasVisitor improvements.
......................................................................

ASTERIXDB-1381: InlineColumnAliasVisitor improvements.

-moved mapForRecordConstructor closer to usage location;
-documented the boolean argument to the visit method;
-documented what happens in mapForRecordConstructor if leftExpr is not a 
literal.

Change-Id: I82c5bc4568c11c474bec053e9f5162927c1c15d0
---
A 
asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-02/sugar-02.4.query.sqlpp
A 
asterix-app/src/test/resources/runtimets/results/group-by/core-02/core-02.2.adm
M 
asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/visitor/InlineColumnAliasVisitor.java
3 files changed, 159 insertions(+), 107 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/89/789/4
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/789
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I82c5bc4568c11c474bec053e9f5162927c1c15d0
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>

Reply via email to