Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 22:
Integration Tests Timed Out
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/953/ :
ABORTED
--
To
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 22:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND LOOK FOR API
abdullah alamoudi has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 22: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1150
To unsubscribe, visit
Hello abdullah alamoudi, Michael Carey, Michael Blow, Till Westmann, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#22).
Change subject: Enhanced Insert AQL
Steven Jacobs has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 21:
(5 comments)
I went ahead and made the in-line comment changes.
Till Westmann has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 21: Code-Review+2
(6 comments)
Just a few tiny comments. The fixes to the manual should probably happen, but
this could also
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 21: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/948/ :
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 21:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/948/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 21:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND LOOK FOR API
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 21:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3045/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 20: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/946/ :
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 20:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND LOOK FOR API
Steven Jacobs has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 19:
(3 comments)
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 20:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3042/
--
To view, visit
Hello abdullah alamoudi, Michael Carey, Michael Blow, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#20).
Change subject: Enhanced Insert AQL
Yingyi Bu has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 19:
(3 comments)
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 19: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/913/ :
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 17: Integration-Tests-1
Integration Tests Failed
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/900/ :
Yingyi Bu has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 17:
One more question:
What's the difference between ASYNC and ASYNC_DEFERRED?
--
To view, visit
Yingyi Bu has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 17:
(13 comments)
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 17:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/900/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 17:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2989/
--
To view, visit
Hello Michael Carey, Michael Blow, Steven Jacobs, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#17).
Change subject: Enhanced Insert AQL
Michael Blow has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 16: Code-Review+1
--
To view, visit https://asterix-gerrit.ics.uci.edu/1150
To unsubscribe, visit
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 16:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/887/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 16:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND LOOK FOR API
Michael Blow has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 15: Code-Review-2
(1 comment)
https://asterix-gerrit.ics.uci.edu/#/c/1150/15/asterixdb/asterix-server/pom.xml
File
abdullah alamoudi has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 15: Code-Review+1
I am good with this change. Till? Yingyi?
--
To view, visit https://asterix-gerrit.ics.uci.edu/1150
To
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 15: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/783/ :
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 15:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/783/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 15:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND LOOK FOR API
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 15:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2834/
--
To view, visit
Hello Michael Carey, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#15).
Change subject: Enhanced Insert AQL
..
Enhanced Insert AQL
The
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 14:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND LOOK FOR API
Hello Michael Carey, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#14).
Change subject: Enhanced Insert AQL
..
Enhanced Insert AQL
The
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 13:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND LOOK FOR API
Steven Jacobs has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 13:
Fixed the issue
--
To view, visit https://asterix-gerrit.ics.uci.edu/1150
To unsubscribe, visit
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 13:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2740/
--
To view, visit
Hello Michael Carey, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#13).
Change subject: Enhanced Insert AQL
..
Enhanced Insert AQL
The
Steven Jacobs has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 12: Code-Review-1
(1 comment)
Working on newly discovered issue
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 12: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/697/ :
Michael Blow has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 12:
(1 comment)
https://asterix-gerrit.ics.uci.edu/#/c/1150/12/asterixdb/asterix-server/pom.xml
File
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 12:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/697/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 12:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND LOOK FOR API
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 12:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2726/
--
To view, visit
Hello Michael Carey, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#12).
Change subject: Enhanced Insert AQL
..
Enhanced Insert AQL
The
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 11:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW CAREFULLY AND LOOK FOR API
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 11:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2725/
--
To view, visit
Hello Michael Carey, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#11).
Change subject: Enhanced Insert AQL
..
Enhanced Insert AQL
The
Steven Jacobs has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 10:
The extension operator was actually already being used. If you look at
SetupCommitExtensionOpRule, this rule used to be
abdullah alamoudi has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 10:
(6 comments)
Comments added.
Beside the comments, I am worried about the implications of the use of the
extension
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 10: Integration-Tests+1
Integration Tests Successful
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/660/ :
Michael Carey has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 10:
I couldn't figure out how to publish my comment on Till's comment - but the
gist is I like the variant:
insert into
Steven Jacobs has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 10:
@Mike: I'm not seeing your comment, maybe it hasn't been published yet?
Okay, I will start working on this change now. I
Michael Carey has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 10:
I added a comment on the syntax - I would like to second Till's change (modulo
finalizing the var-binding syntax).
--
To
Michael Carey has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 10: Code-Review-1
I added a common on the syntax - I would like to second Till's change (modulo
finalizing the var-binding
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 10:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/660/
--
To view, visit
Steven Jacobs has posted comments on this change.
Change subject: Enhanced Insert AQL
..
Patch Set 9:
(14 comments)
I addressed everything except for the documentation comment. Does this comment
fit more into the "It would
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#10).
Change subject: Enhanced Insert AQL
..
Enhanced Insert AQL
Added "return
Steven Jacobs has posted comments on this change.
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
inserted "returning [fieldName]" returns only the valuse inserted for
[fieldName] Allow commits to
Till Westmann has posted comments on this change.
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
inserted "returning [fieldName]" returns only the valuse inserted for
[fieldName] Allow commits to
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
inserted "returning [fieldName]" returns only the valuse inserted for
[fieldName] Allow commits to be
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
inserted "returning [fieldName]" returns only the valuse inserted for
[fieldName] Allow commits to be
Steven Jacobs has posted comments on this change.
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
inserted "returning [fieldName]" returns only the valuse inserted for
[fieldName] Allow commits to
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
inserted "returning [fieldName]" returns only the valuse inserted for
[fieldName] Allow commits to be
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#9).
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
inserted "returning [fieldName]" returns only the valuse inserted for
[fieldName] Allow commits to be
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
inserted "returning [fieldName]" returns only the valuse inserted for
[fieldName] Allow commits to be
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#8).
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
Jenkins has posted comments on this change.
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
inserted "returning [fieldName]" returns only the valuse inserted for
[fieldName] Allow commits to be
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1150
to look at the new patch set (#7).
Change subject: Enhanced Insert AQL Added "return records" and "returning
[fieldName]" "return records" returns to the user all records that were
71 matches
Mail list logo