abdullah alamoudi has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 10:
Thanks Mike :)
--
To view, visit https://asterix-gerrit.ics.uci.edu/997
To unsubscribe, visit
Michael Blow has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 9: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/997
To unsubscribe, visit
Michael Blow has submitted this change and it was merged.
Change subject: Cast Record Before Insert for Open Indexes
..
Cast Record Before Insert for Open Indexes
This change modifies the insert plan to perform cast for open
abdullah alamoudi has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 9:
I agree. I added them and even I am confused :)
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 9:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/153/
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 9:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1987/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 9:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW
Hello Yingyi Bu, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/997
to look at the new patch set (#9).
Change subject: Cast Record Before Insert for Open Indexes
..
Cast
Till Westmann has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 8: Code-Review+2
I think that I got confused by the big number of added constructors.
I think that we
Yingyi Bu has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 8: Code-Review+1
LGTM.
Not sure if that's still a problem:
"Seems that $1s would be replaced by 'e', not
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 8: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 8:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1973/
--
To view, visit
abdullah alamoudi has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 8:
I believe that I have addressed all comments.
Once this gets merged, I can rebase my other
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 8:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 7:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1971/
--
To view, visit
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 7:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 7:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1970/
--
To view, visit
abdullah alamoudi has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 6:
(4 comments)
Yingyi Bu has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 6: Code-Review+1
(1 comment)
Just one minor comment.
Till Westmann has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 6: Code-Review+1
(3 comments)
Still needs Yingyi's rewriting expertise ...
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 6: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 6:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 5:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 5:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1954/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/997
to look at the new patch set (#5).
Change subject: Cast Record Before Insert for Open Indexes
..
Cast Record Before
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 4:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 4:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1952/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/997
to look at the new patch set (#4).
Change subject: Cast Record Before Insert for Open Indexes
..
Cast Record Before
Till Westmann has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 3:
(7 comments)
Mostly comments on exception handling/error codes.
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 3: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 3:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/101/
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 3:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 3:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1918/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/997
to look at the new patch set (#3).
Change subject: Cast Record Before Insert for Open Indexes
..
Cast Record Before
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 2:
WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack
PLEASE REVIEW
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 2:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1917/
--
To view, visit
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/997
to look at the new patch set (#2).
Change subject: Cast Record Before Insert for Open Indexes
..
Cast Record Before
abdullah alamoudi has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 1:
(5 comments)
I think that I have addressed all the comments.
Let me know if I missed anything.
Yingyi Bu has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 1:
(5 comments)
1. I do not understand why the change must be in InlineVariableRule? Could we
have a
abdullah alamoudi has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 1:
(1 comment)
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 1: Integration-Tests+1
Integration Tests Successful
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 1:
Integration Tests Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/82/
--
Jenkins has posted comments on this change.
Change subject: Cast Record Before Insert for Open Indexes
..
Patch Set 1:
Build Started
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1879/
--
To view, visit
abdullah alamoudi has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/997
Change subject: Cast Record Before Insert for Open Indexes
..
Cast Record Before Insert for Open Indexes
This change modifies the
44 matches
Mail list logo