Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-03 Thread Xikui Wang (Code Review)
Xikui Wang has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1339 Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Fix ASTERIXDB-1609 and OrderedList bug in TweetParser 1. For AS

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-03 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3242/ -- To view,

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-03 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tes

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-03 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 1: Integration-Tests-1 Integration Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterix-ger

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-03 Thread Xikui Wang (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1339 to look at the new patch set (#2). Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Fix ASTER

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-03 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3243/ -- To view,

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-03 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 2: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tes

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-03 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 2: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-06 Thread Wail Alkowaileet (Code Review)
Wail Alkowaileet has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 2: Hi Xikui, I tried to see the output of the new parser. I noticed something different: {

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-06 Thread Xikui Wang (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1339 to look at the new patch set (#3). Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Fix ASTER

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-06 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3258/ -- To view,

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-06 Thread Xikui Wang (Code Review)
Xikui Wang has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 3: Hi Wail, There are some tweets come with 'entities' like this: {"urls":[],"hashtags":[],"use

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-06 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 3: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tes

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-06 Thread Xikui Wang (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1339 to look at the new patch set (#4). Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Fix ASTER

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-06 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 4: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3259/ -- To view,

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-06 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 4: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tes

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-06 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 4: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-07 Thread Wail Alkowaileet (Code Review)
Wail Alkowaileet has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 4: For some reason, I cannot inline my comment. I will follow this format: {line# : code c

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-08 Thread Xikui Wang (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1339 to look at the new patch set (#5). Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Fix ASTER

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-08 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 5: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3268/ -- To view,

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-08 Thread Xikui Wang (Code Review)
Xikui Wang has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 4: Hi Wail, Thanks for your comments! My replies are as follow: Line 220: Here the incoming dat

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-08 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 5: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tes

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-08 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 5: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-09 Thread Wail Alkowaileet (Code Review)
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 desp

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-14 Thread Xikui Wang (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1339 to look at the new patch set (#6). Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Fix ASTER

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-14 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 6: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3295/ -- To view,

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-14 Thread Xikui Wang (Code Review)
Xikui Wang has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 6: @Wail, Yes. You are right. I misunderstood it. :) Fixed it in the updated patch. Sorry for

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-14 Thread Xikui Wang (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1339 to look at the new patch set (#7). Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Fix ASTER

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-14 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 7: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3296/ -- To view,

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-14 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 7: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tes

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-15 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 7: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-15 Thread Wail Alkowaileet (Code Review)
Wail Alkowaileet has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 7: Code-Review+1 -- To view, visit https://asterix-gerrit.ics.uci.edu/1339 To unsubscribe,

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-15 Thread Steven Jacobs (Code Review)
Steven Jacobs has posted comments on this change. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Patch Set 7: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1339 To unsubscribe, vi

Change in asterixdb[master]: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser

2016-11-29 Thread Steven Jacobs (Code Review)
Steven Jacobs has submitted this change and it was merged. Change subject: Fix ASTERIXDB-1609 and OrderedList bug in TweetParser .. Fix ASTERIXDB-1609 and OrderedList bug in TweetParser 1. For ASTERIXDB-1609, add UNION type che