Dmitry Lychagin has posted comments on this change.

Change subject: [ASTERIXDB-2375][RT] Evaluate constant experession in SELECT 
only once
......................................................................


Patch Set 12:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/2621/12/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/comparisons/ComparisonHelper.java
File 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/comparisons/ComparisonHelper.java:

Line 450:             // if any exception happens, treat it as non-constant 
case.
> In the NCServiceExceutionIT, the passed in are malformed. In that case, the
Can we fix the testcase instead?


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2621
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iae4e78928da2bd63b2984b3624b88baed9b7cd73
Gerrit-PatchSet: 12
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang <xkk...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Dmitry Lychagin <dmitry.lycha...@couchbase.com>
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Taewoo Kim <wangs...@gmail.com>
Gerrit-Reviewer: Xikui Wang <xkk...@gmail.com>
Gerrit-HasComments: Yes

Reply via email to