Yingyi Bu has submitted this change and it was merged.
Change subject: TweetParser Extension
..
TweetParser Extension
This patch includes following changes:
1. ExtendedTweetParser to parse more than fix attributes.
2.
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 14:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/428/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 14:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2353/
--
To view, visit
Hello abdullah alamoudi, Yingyi Bu, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1002
to look at the new patch set (#14).
Change subject: TweetParser Extension
..
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 13: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/427/
Yingyi Bu has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 12:
@Xikui, there is a merge conflict. Can you rebase the current master? Thx!
--
To view, visit
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 12: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/388/
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 12: Integration-Tests-1
Integration Tests Timed Out
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/347/
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 12:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/347/
--
To view, visit
Xikui Wang has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 12:
Fixed it. Sorry about that.
--
To view, visit https://asterix-gerrit.ics.uci.edu/1002
To unsubscribe, visit
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 12:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2262/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1002
to look at the new patch set (#12).
Change subject: TweetParser Extension
..
TweetParser Extension
This patch
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 11:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2261/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1002
to look at the new patch set (#11).
Change subject: TweetParser Extension
..
TweetParser Extension
This patch
Till Westmann has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 10:
In a number of files the @Override annotation got moved into the same line as
the method declaration ... could you fix
Xikui Wang has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 10: Code-Review+1
I went through all the changes. It looks fine to me now.
--
To view, visit
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 10: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/292/
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1002
to look at the new patch set (#10).
Change subject: TweetParser Extension
..
TweetParser Extension
This patch
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 10:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2184/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 9:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2183/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1002
to look at the new patch set (#9).
Change subject: TweetParser Extension
..
TweetParser Extension
This patch
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 8:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/289/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1002
to look at the new patch set (#8).
Change subject: TweetParser Extension
..
TweetParser Extension
This patch
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 8:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2179/
--
To view, visit
Till Westmann has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 7:
(1 comment)
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 7: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/216/
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 7:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/216/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 6: Integration-Tests-1
Integration Tests Failed
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/151/ :
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 6:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/151/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1002
to look at the new patch set (#6).
Change subject: TweetParser Extension
..
TweetParser Extension
This patch
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 5: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/148/
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 5:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/148/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 5:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1982/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1002
to look at the new patch set (#5).
Change subject: TweetParser Extension
..
TweetParser Extension
This patch
Xikui Wang has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 4:
@Till
Thanks for your explanation. That makes totally sense to me. Will apply the
formatter and add test cases at the next
Xikui Wang has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 4:
@Till
Another submit addresses the Line # fix.
Just a side note on the coding style part. I got quite a few comments from
Xikui Wang has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 4:
@Abdullah,
I checked the TwitterObjectFactory.getRawJSON method. When the JsonStoreEnabled
is turned on. The
Till Westmann has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 4:
(10 comments)
With the exception of the change in Lexer.java only easy-to-address coding
style comments :)
The change
abdullah alamoudi has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 4:
Can you please add test cases that test those functionalities. They don't need
to be runtime tests within asterixdb
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1002
to look at the new patch set (#4).
Change subject: TweetParser Extension
..
TweetParser Extension
This patch
Jenkins has posted comments on this change.
Change subject: TweetParser Extension
..
Patch Set 3:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1904/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1002
to look at the new patch set (#3).
Change subject: TweetParser Extension
..
TweetParser Extension
This patch
Jenkins has posted comments on this change.
Change subject: TweetParser extension
..
Patch Set 2:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1903/
--
To view, visit
43 matches
Mail list logo