Preston Carman has posted comments on this change.

Change subject: ASTERIXDB-1102: VarSize Encoding to store length of String and 
ByteArray
......................................................................


Patch Set 4:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/450/4/asterix-app/src/test/resources/runtimets/queries/string/matches11/matches11.3.query.aql
File 
asterix-app/src/test/resources/runtimets/queries/string/matches11/matches11.3.query.aql:

Line 28: matches("hello",null),
> Yes, it is related to the null-element-list issue. I add the null to work a
We should make sure that a new test is added to test this the documented issue 
since this test will not be doing it.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I41fff780f5c071742ef10129d83c8f945d5886d7
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Jianfeng Jia <[email protected]>
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Jianfeng Jia <[email protected]>
Gerrit-Reviewer: Preston Carman <[email protected]>
Gerrit-Reviewer: Taewoo Kim <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-HasComments: Yes

Reply via email to