asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitry Lychagin (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: [ASTERIXDB-2280][IDX] Fix Index on Optional Nested Field
Date Fri, 23 Mar 2018 23:50:33 GMT
Dmitry Lychagin has posted comments on this change.

Change subject: [ASTERIXDB-2280][IDX] Fix Index on Optional Nested Field
......................................................................


Patch Set 9:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/2477/9/asterixdb/asterix-app/src/test/resources/runtimets/results/nestrecords/nested-optional-pk/nested-optional-pk.1.adm
File asterixdb/asterix-app/src/test/resources/runtimets/results/nestrecords/nested-optional-pk/nested-optional-pk.1.adm:

PS9, Line 1: [
The expected result is an error. We don't need to create an empty reference result file for
this testcase then. Can we remove this file?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If098ff4a45db3c4e8b65d098cbb0940c8b7ab845
Gerrit-PatchSet: 9
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Luo Chen <cluo8@uci.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Dmitry Lychagin <dmitry.lychagin@couchbase.com>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Luo Chen <cluo8@uci.edu>
Gerrit-Reviewer: Taewoo Kim <taewok2@uci.edu>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>
Gerrit-Reviewer: abdullah alamoudi <bamousaa@gmail.com>
Gerrit-HasComments: Yes

Mime
View raw message