Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-24 Thread abdullah alamoudi (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-24 Thread Michael Blow (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-24 Thread Michael Blow (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-24 Thread abdullah alamoudi (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-24 Thread Jenkins (Code Review)
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/

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-24 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-24 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-24 Thread Till Westmann (Code 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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-24 Thread Till Westmann (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-22 Thread Yingyi Bu (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-22 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-22 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-22 Thread abdullah alamoudi (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-22 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-22 Thread Jenkins (Code 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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-22 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-22 Thread Jenkins (Code 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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-22 Thread abdullah alamoudi (Code Review)
abdullah alamoudi has posted comments on this change. Change subject: Cast Record Before Insert for Open Indexes .. Patch Set 6: (4 comments)

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-22 Thread Yingyi Bu (Code Review)
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.

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-21 Thread Till Westmann (Code Review)
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 ...

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-21 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-21 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-21 Thread Jenkins (Code 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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-21 Thread Jenkins (Code 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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-21 Thread abdullah alamoudi (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-21 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-21 Thread Jenkins (Code 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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-21 Thread abdullah alamoudi (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-20 Thread Till Westmann (Code Review)
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.

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-17 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-17 Thread Jenkins (Code Review)
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/

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-17 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-17 Thread Jenkins (Code 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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-17 Thread abdullah alamoudi (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-17 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-17 Thread Jenkins (Code 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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-17 Thread abdullah alamoudi (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-17 Thread abdullah alamoudi (Code Review)
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.

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-14 Thread Yingyi Bu (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-14 Thread abdullah alamoudi (Code Review)
abdullah alamoudi has posted comments on this change. Change subject: Cast Record Before Insert for Open Indexes .. Patch Set 1: (1 comment)

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-14 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-14 Thread Jenkins (Code Review)
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/ --

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-14 Thread Jenkins (Code Review)
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

Change in asterixdb[master]: Cast Record Before Insert for Open Indexes

2016-07-14 Thread abdullah alamoudi (Code Review)
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