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
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1006
to look at the new patch set (#6).
Change subject: Add init parameter option for external functions.
..
Add init
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
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: Add init parameter option for external functions.
..
Patch Set 6:
@Till & Michael,
There are many comments from SonaQube that regarding to the existed coding
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1006
to look at the new patch set (#4).
Change subject: Add init parameter option for external functions.
..
Add init
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
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
Xikui Wang has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1068
Change subject: Add upsert option for feed
..
Add upsert option for feed
For ASTERIXDB-1567. Provide "upsert-feed"="true" for
feed
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
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
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1068
to look at the new patch set (#8).
Change subject: Add upsert option for feed
..
Add upsert option for feed
For
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1068
to look at the new patch set (#7).
Change subject: Add upsert option for feed
..
Add upsert option for feed
For
Xikui Wang has posted comments on this change.
Change subject: Add upsert option for feed
..
Patch Set 7:
@Till
Thanks for your comments. It's good to have someone helping me improve my
coding style and quality. I added one
Xikui Wang has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1012
Change subject: Fix for ASTERIXDB-1471
..
Fix for ASTERIXDB-1471
The line number is increased twice in setBuffer and updateLineColumn
Xikui Wang has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1006
Change subject: Add init parameter option for external functions.
..
Add init parameter option for external functions.
Add one
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1006
to look at the new patch set (#2).
Change subject: Add init parameter option for external functions.
..
Add init
Xikui Wang has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1002
Change subject: This patch includes following changes:
..
This patch includes following changes:
1. ExtendedTweetParser to parse more
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
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
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
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
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
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1259
to look at the new patch set (#11).
Change subject: Feed Connection Refactoring
..
Feed Connection Refactoring
1.
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1259
to look at the new patch set (#10).
Change subject: Feed Connection Refactoring
..
Feed Connection Refactoring
1.
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1259
to look at the new patch set (#8).
Change subject: WIP-Feed Connection Refactoring
..
WIP-Feed Connection
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1430
to look at the new patch set (#6).
Change subject: Modularize feed adaptors
..
Modularize feed adaptors
1. Load
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1430
to look at the new patch set (#7).
Change subject: Modularize feed adaptors
..
Modularize feed adaptors
1. Load
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1430
to look at the new patch set (#9).
Change subject: Modularize feed adaptors
..
Modularize feed adaptors
1. Load
Xikui Wang has posted comments on this change.
Change subject: Modularize feed adaptors
..
Patch Set 9:
@Mike B, The further modularization comment turns out to be very useful! Quite
some knowledge about Maven is gained from
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1430
to look at the new patch set (#5).
Change subject: Modularize feed adaptors
..
Modularize feed adaptors
1. Load
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1430
to look at the new patch set (#8).
Change subject: Modularize feed adaptors
..
Modularize feed adaptors
1. Load
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1259
to look at the new patch set (#13).
Change subject: Feed Connection Refactoring
..
Feed Connection Refactoring
1.
Xikui Wang has posted comments on this change.
Change subject: Enable Feed Changes to work with BAD project
..
Patch Set 1:
This change looks good to me. One general comment about this change. It seems
the repeatable flag is
Xikui Wang has abandoned this change.
Change subject: Add init parameter option for external functions.
..
Abandoned
This patch is too old and some of the modifications are not applicable due to
file deletion. Will submit
Xikui Wang has posted comments on this change.
Change subject: Added Procedures to BAD
..
Patch Set 5:
(6 comments)
I added several comments. Basically that's focusing on the EventListener and
related issues. You can also
Xikui Wang has posted comments on this change.
Change subject: Update gitignore file for BAD project
..
Patch Set 1:
@Steven,
As a test submit, I updated the .gitignore for bad project.
--
To view, visit
Xikui Wang has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1454
Change subject: Update gitignore file for BAD project
..
Update gitignore file for BAD project
Add several file filters for project
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1430
to look at the new patch set (#3).
Change subject: Modularize feed adaptors
..
Modularize feed adaptors
1. Load
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
Xikui Wang has posted comments on this change.
Change subject: Add upsert option for feed
..
Patch Set 8: Code-Review+1
Looks good to me too.
The integration sometimes failed at timeout. Not sure the reason.
--
To view,
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1535
to look at the new patch set (#2).
Change subject: WIP - Feed Adaptor and RecordReader Clean
..
WIP - Feed Adaptor
Xikui Wang has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1533
Change subject: Update the Feed Tutorial
..
Update the Feed Tutorial
1. Updated the Twitter and RSS part to work with the current new
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1535
to look at the new patch set (#3).
Change subject: Feed Adaptor and RecordReader Clean
..
Feed Adaptor and
Xikui Wang has posted comments on this change.
Change subject: Feed Adaptor and RecordReader Clean
..
Patch Set 3:
(3 comments)
Added one Q to StreamRecordReaderFactory
Xikui Wang has abandoned this change.
Change subject: Modularize feed adaptors
..
Abandoned
useful parts are picked up in another patch. The separate package for
twitter/rss feeds is delayed for future better ideas.
--
To
Xikui Wang has posted comments on this change.
Change subject: Updated BAD to work with recent Asterix changes
..
Patch Set 1:
(1 comment)
added one comment.
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1535
to look at the new patch set (#4).
Change subject: Feed Adaptor and RecordReader Clean
..
Feed Adaptor and
Xikui Wang has posted comments on this change.
Change subject: Update the Feed Tutorial
..
Patch Set 1:
(1 comment)
I think the IT failure is a false alarm... Will have it test again in next
revision.
Xikui Wang has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1534
Change subject: Feed Exception Revise
..
Feed Exception Revise
1. Add compliation exception for missing Twitter4J library. Try to
Xikui Wang has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1535
Change subject: WIP - Feed Adaptor and RecordReader Clean
..
WIP - Feed Adaptor and RecordReader Clean
1. Cleaned
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1534
to look at the new patch set (#2).
Change subject: Feed Exception Revise
..
Feed Exception Revise
1. Add
Hello Michael Blow, Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1068
to look at the new patch set (#9).
Change subject: Add upsert option for feed
..
Add
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1068
to look at the new patch set (#13).
Change subject: Add upsert option for feed
..
Add upsert option for feed
For
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1068
to look at the new patch set (#12).
Change subject: Add upsert option for feed
..
Add upsert option for feed
For
Xikui Wang has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1152
Change subject: Feed tutorial revise.
..
Feed tutorial revise.
1. Fix 1352.
2. Revise the twitter feed part to work with the new tweet
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1068
to look at the new patch set (#11).
Change subject: Add upsert option for feed
..
Add upsert option for feed
For
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1152
to look at the new patch set (#2).
Change subject: Feed tutorial revise.
..
Feed tutorial revise.
1. Fix 1352.
2.
Hello abdullah alamoudi, Michael Blow, Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1068
to look at the new patch set (#14).
Change subject: Add upsert option for feed
Hello Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1068
to look at the new patch set (#10).
Change subject: Add upsert option for feed
..
Add upsert option
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
..
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1259
to look at the new patch set (#3).
Change subject: Feed Connection Refactoring
..
Feed Connection Refactoring
1.
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1259
to look at the new patch set (#4).
Change subject: Feed Connection Refactoring
..
Feed Connection Refactoring
1.
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1259
to look at the new patch set (#5).
Change subject: Feed Connection Refactoring
..
Feed Connection Refactoring
1.
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
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
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
Xikui Wang has posted comments on this change.
Change subject: Introduce XML Reader & Parser
..
Patch Set 6:
(1 comment)
Xikui Wang has posted comments on this change.
Change subject: Add user-stream for Twitter Adaptor
..
Patch Set 2:
(1 comment)
The user whose access token is used in the feed creation, is the one that we
are monitoring.
Xikui Wang has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1272
Change subject: Add user-stream for Twitter Adaptor
..
Add user-stream for Twitter Adaptor
1. Add user-stream option for Twitter
Xikui Wang has posted comments on this change.
Change subject: Introduce XML Reader & Parser
..
Patch Set 6:
(1 comment)
Xikui Wang has posted comments on this change.
Change subject: Add user-stream for Twitter Adaptor
..
Patch Set 2:
(1 comment)
Yes. The user of twitter feed needs to apply for Twitter API using certain
twitter account.
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
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
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
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
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
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1259
to look at the new patch set (#2).
Change subject: Feed Connection Refactoring
..
Feed Connection Refactoring
1.
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
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
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
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
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
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 (#2).
Change subject: Introduce XML Adaptor & Parser
..
Introduce XML Adaptor & Parser
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 (#6).
Change subject: Introduce XML Reader & Parser
..
Introduce XML Reader & Parser
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 (#4).
Change subject: Introduce XML Reader & Parser
..
Introduce XML Reader & Parser
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1272
to look at the new patch set (#2).
Change subject: Add user-stream for Twitter Adaptor
..
Add user-stream for
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
..
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1374
to look at the new patch set (#5).
Change subject: Runtime ErrorCode fix in external data
..
Runtime ErrorCode fix
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
..
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
..
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
..
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
Xikui Wang has posted comments on this change.
Change subject: Feed Connection Refactoring
..
Patch Set 5:
(7 comments)
I added several comments to yours. I will try to do some cleaning and merge the
updates from master as
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
Hello Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1416
to look at the new patch set (#4).
Change subject: Load parser from classpath in ParserFactoryProvider
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1259
to look at the new patch set (#6).
Change subject: WIP-Feed Connection Refactoring
..
WIP-Feed Connection
Xikui Wang has posted comments on this change.
Change subject: WIP - Modularize feed adaptors
..
Patch Set 1:
@Till
I added one package with temporary name for the Twitter4j related classes. I
tried to move all adaptors into
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1430
to look at the new patch set (#2).
Change subject: WIP - Modularize feed adaptors
..
WIP - Modularize feed adaptors
1 - 100 of 838 matches
Mail list logo