Wail Alkowaileet has posted comments on this change.

Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser
......................................................................


Patch Set 5:

isClosed doesn't mean it's optional or not optional. it means the field is 
defined despite the type. Optional fields has to be checked as a union type of 
(null, missing, actual-type). 

To produce the exception, use the following DDL (put breakpoint on the 
exception line: 221):

drop dataverse feeds if exists
create dataverse feeds;
use dataverse feeds;

create type Geo as closed {
    coordinates: [double],
    'type':string
}

create type Tweet as closed {
    id: int64,
    text:string,
    geo:Geo?  
}
create dataset Tweets (Tweet)
primary key id

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia27148cb10206b93dabf7655aed68f3004f96dfd
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Wail Alkowaileet <[email protected]>
Gerrit-Reviewer: Xikui Wang <[email protected]>
Gerrit-HasComments: No

Reply via email to