Xikui Wang has abandoned this change.
Change subject: Add record reader and parser for CAP messages
..
Abandoned
This change will be resubmitted to bad branch.
--
To view, visit https://asterix-gerrit.ics.uci.edu/1269
To
Xikui Wang has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 16:
Mike and I had a discussion about moving these research specific patches into
the BADAsterix
Xikui Wang has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 16:
Sorry for making so many changes in this patch. I found a couple of bugs when I
was addressing your
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 16: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 16:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/25/
--
To view,
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 16:
Integration Tests Started
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 16:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3518/
--
To view, visit
Hello Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1269
to look at the new patch set (#16).
Change subject: Add record reader and parser for CAP messages
..
Till Westmann has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 13:
(8 comments)
Submitting some comments on a older patchset - sorry.
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 15:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3511/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 15:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3510/
--
To view, visit
Hello Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1269
to look at the new patch set (#15).
Change subject: Add record reader and parser for CAP messages
..
Hello Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1269
to look at the new patch set (#14).
Change subject: Add record reader and parser for CAP messages
..
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 14:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3509/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 13: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 13:
Integration Tests Started
Hello Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1269
to look at the new patch set (#13).
Change subject: Add record reader and parser for CAP messages
..
abdullah alamoudi has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 12:
(2 comments)
Nothing beside what Till has already mentioned.
For the object allocation,
Till Westmann has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 12:
(10 comments)
Just a few comments - didn't think about your object allocation question ...
Xikui Wang has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 12:
(1 comment)
I replaced the XML2Json part with SAXParser. A lot of objects are introduced to
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 12: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 12:
Integration Tests Started
Hello Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1269
to look at the new patch set (#12).
Change subject: Add record reader and parser for CAP messages
..
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 12:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3414/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 11:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3412/
--
To view, visit
Hello Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1269
to look at the new patch set (#11).
Change subject: Add record reader and parser for CAP messages
..
Till Westmann has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 10: Code-Review-1
Marking -1 due to the org.json issue.
--
To view, visit
Till Westmann has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 10:
Also added Ian as a reviewer as he's currently working on
Till Westmann has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 10:
Indeed the license has been moved to category X [1] very recently and so we
should replace the
abdullah alamoudi has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 10:
this change looks good to me. There is however one issue:
http://www.json.org/license.html
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 10: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 10:
Integration Tests Started
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 10:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3298/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1269
to look at the new patch set (#10).
Change subject: Add record reader and parser for CAP messages
..
Add record
Xikui Wang has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 9:
@Till & Abdullah,
One thing I am not very sure in this patch is ASTERIXDB-1690. If you can help
me
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 9:
Integration Tests Started
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 9:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3244/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1269
to look at the new patch set (#9).
Change subject: Add record reader and parser for CAP messages
..
Add record
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 8: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 8:
Integration Tests Started
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 8:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3217/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1269
to look at the new patch set (#8).
Change subject: Add record reader and parser for CAP messages
..
Add record
Jenkins has posted comments on this change.
Change subject: Add record reader and parser for CAP messages
..
Patch Set 7:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3211/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1269
to look at the new patch set (#7).
Change subject: Add record reader and parser for CAP messages
..
Add record
44 matches
Mail list logo
