asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yingyi Bu (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: ASTERIXDB-1223: let incompatiable comparisons return null.
Date Tue, 07 Jun 2016 23:34:49 GMT
Hello Jenkins,

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

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

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

Change subject: ASTERIXDB-1223: let incompatiable comparisons return null.
......................................................................

ASTERIXDB-1223: let incompatiable comparisons return null.

- make null/missing/true/false case-insensitive;
- add tests for incompatible comparisons;
- add tests for group-by/order-by with MISSING valued keys.

Change-Id: I8130d505f810631231d3755941ce4de64313501b
---
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/comparison/incompatible/incompatible.1.query.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/null-missing/group-by/group-by.1.query.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/null-missing/order-by/order-by.1.query.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/results/comparison/incompatible/incompatible.1.adm
A asterixdb/asterix-app/src/test/resources/runtimets/results/null-missing/group-by/group-by.1.adm
A asterixdb/asterix-app/src/test/resources/runtimets/results/null-missing/order-by/order-by.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/comparisons/AbstractComparisonEvaluator.java
10 files changed, 144 insertions(+), 13 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8130d505f810631231d3755941ce4de64313501b
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <buyingyi@gmail.com>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>

Mime
View raw message