Yingyi Bu has submitted this change and it was merged.
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
More extension support is added. A user can now provide implementations
for the IExtension
Yingyi Bu has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 22: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1017
To unsubscribe, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 22: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 22:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/394/
--
To view,
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 22:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 22:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2315/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 1:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2314/
--
To view, visit
Yingyi Bu has abandoned this change.
Change subject: Add Asterix Extension Manager
..
Abandoned
--
To view, visit https://asterix-gerrit.ics.uci.edu/1095
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Yingyi Bu has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1095
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
More extension support is added. A user can now provide
Yingyi Bu has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 21: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1017
To unsubscribe, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 21: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 21:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/393/
--
To view,
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 19: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 21:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2310/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 21:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2309/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 21:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Hello Steven Jacobs, Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#21).
Change subject: Add Asterix Extension Manager
..
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 20:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Hello Steven Jacobs, Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#20).
Change subject: Add Asterix Extension Manager
..
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 20:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2308/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 19:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/392/
--
To view,
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 19:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2307/
--
To view, visit
Hello Steven Jacobs, Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#19).
Change subject: Add Asterix Extension Manager
..
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 17: Integration-Tests+1
Integration Tests Successful
Hello Steven Jacobs, Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#18).
Change subject: Add Asterix Extension Manager
..
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 18:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 18:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2306/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 17:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/390/
--
To view,
Hello Steven Jacobs, Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#17).
Change subject: Add Asterix Extension Manager
..
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 17:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 17:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2305/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 16:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Hello Steven Jacobs, Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#16).
Change subject: Add Asterix Extension Manager
..
Yingyi Bu has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 15:
(3 comments)
1. Can you make a pass over the Sonar Qube comments? I guess most of them can
be addressed except few
Steven Jacobs has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 15: Code-Review+1
I would +2, but I'm guessing Yingyi wants to do another pass?
--
To view, visit
Till Westmann has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 15: Code-Review+1
--
To view, visit https://asterix-gerrit.ics.uci.edu/1017
To unsubscribe, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 15: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 15:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/346/
--
To view,
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 15:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 15:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2260/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#15).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
abdullah alamoudi has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 14:
(16 comments)
abdullah alamoudi has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 14:
P.S
Maybe we can have the compiler figure out how to handle multiple connections
but that would be a large
abdullah alamoudi has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 14:
@Yingyi,
There is no problem with having multiple roots. I am the one using it and here
is what I need to
Yingyi Bu has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 14:
>>Why do we need to perturb hyracks jobs and modify those low level things?
I'm guessing it is because that the
Yingyi Bu has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 14:
(8 comments)
"2. the copy and setOperatorId are needed if one wants to take
operators from a compiled job and use
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 14: Integration-Tests-1
Integration Tests Failed
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 14:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/336/
--
To view,
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#14).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 14:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2239/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 14:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
abdullah alamoudi has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 13:
(49 comments)
Till Westmann has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 13:
(16 comments)
Second round of comments.
Steven Jacobs has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 13:
(10 comments)
A few comments
Till Westmann has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 13:
(27 comments)
One rounds of comments - didn't make it all the way through (had to fight with
some
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 13: Integration-Tests-1
Integration Tests Timed Out
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 13:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/316/
--
To view,
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 13:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 13:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2218/
--
To view, visit
abdullah alamoudi has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 7:
(22 comments)
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#13).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 12:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 12:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2187/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#12).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
abdullah alamoudi has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 7:
(25 comments)
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 11: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 11:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/290/
--
To view,
abdullah alamoudi has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 10:
@Yingyi, I will go over the inner comments but want to respond to the two outer
comments here:
1. insertion
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 11:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 11:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2181/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#11).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 10:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 10:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2180/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#10).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
Yingyi Bu has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 7:
(61 comments)
1. The handling of insertion for datasets with meta doesn't make sense to me.
Maybe I miss something.
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 9: Integration-Tests+1
Integration Tests Successful
Till Westmann has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 7:
(2 comments)
A few more comments for patchset 7 in StopActiveRuntimeOperatorDescriptor.java
and
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 9:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2174/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 9:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/285/
--
To view,
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#9).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 8:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 8:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2173/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#8).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
Till Westmann has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 6:
(1 comment)
Till Westmann has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 7:
(7 comments)
Only a partial review for now as I won't finish tonight.
I like the reduction of code in the
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 7: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 7:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2158/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#7).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
abdullah alamoudi has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 6:
(36 comments)
abdullah alamoudi has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 6:
Steven,
I will try to address the issue of multiple extensions in my next patch.
As of now, multiple
Steven Jacobs has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 6:
(2 comments)
Only a couple of minor comments. Overall, things look fine here. But I think
there may be a big
Till Westmann has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 6:
(41 comments)
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 6: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 6:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2094/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#6).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 4: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 4:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2092/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1017
to look at the new patch set (#4).
Change subject: Add Asterix Extension Manager
..
Add Asterix Extension Manager
abdullah alamoudi has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 3:
Sure. doing that now :)
+ addressing sonarqube comments
--
To view, visit
Michael Blow has posted comments on this change.
Change subject: Add Asterix Extension Manager
..
Patch Set 3:
(2 comments)
A whole bunch of these diffs look to be only WS changes, can we revert these to
make it easier on
1 - 100 of 109 matches
Mail list logo