Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 5: Integration-Tests+1 Integration Tests Successful

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 5: Integration Tests Started

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 4: Integration-Tests+1 Integration Tests Successful

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 5: WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN: * asterixdb *

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 5: Build Started

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Steven Jacobs (Code Review)
Steven Jacobs has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 4: (3 comments) Regarding the two queries that you listed, I will look

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 4: Integration Tests Started

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Yingyi Bu (Code Review)
Yingyi Bu has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 4: (2 comments) I'm not sure how general the mechanism of

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 4: WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN: * asterixdb *

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 4: Build Started

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Steven Jacobs (Code Review)
Hello Michael Blow, Preston Carman, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1057 to look at the new patch set (#4). Change subject: Compile a plan that matches user query in the case of nonpure functions

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Steven Jacobs (Code Review)
Steven Jacobs has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 3: (9 comments)

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Steven Jacobs (Code Review)
Steven Jacobs has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 3: (3 comments) I addressed a few comments. I'll take a stab at the Sonar

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-25 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 3: Code-Review+1 (4 comments) A side comment: The access rule code need

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-19 Thread Michael Blow (Code Review)
Michael Blow has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 3: Code-Review+1 I don't see any issues other than a number of the

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-16 Thread Steven Jacobs (Code Review)
Steven Jacobs has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 3: I'll fix the headers alongside addressing your upcoming comments.

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-16 Thread Michael Blow (Code Review)
Michael Blow has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 3: Oops: /Stephen/Steven/ -- To view, visit

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-16 Thread Michael Blow (Code Review)
Michael Blow has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 3: Hi Stephen, I hope to complete a thorough review tomorrow; I did notice

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-09 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 3: Integration-Tests+1 Integration Tests Successful

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-09 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 3: Integration Tests Started

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-09 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 3: WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN: * asterixdb *

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-09 Thread Steven Jacobs (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1057 to look at the new patch set (#3). Change subject: Compile a plan that matches user query in the case of nonpure functions

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-08 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 2: Integration-Tests+1 Integration Tests Successful

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-08 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 2: Integration Tests Started

Change in asterixdb[master]: Compile a plan that matches user query in the case of nonpur...

2016-08-08 Thread Jenkins (Code Review)
Jenkins has posted comments on this change. Change subject: Compile a plan that matches user query in the case of nonpure functions .. Patch Set 1: Integration-Tests+1 Integration Tests Successful