Change in asterixdb[master]: [WIP] adding new range multi partitioning operator

2019-06-17 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3223 ) Change subject: [WIP] adding new range multi partitioning operator .. Patch Set 18: (2 comments) After removing these interfaces,

Change in asterixdb[master]: [WIP] adding new range multi partitioning operator

2019-05-30 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3223 ) Change subject: [WIP] adding new range multi partitioning operator .. Patch Set 15: (2 comments) Continue to review the included

Change in asterixdb[master]: [ASTERIXDB-2450][DOC] make docs match implementation

2019-02-08 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: [ASTERIXDB-2450][DOC] make docs match implementation .. Patch Set 3: Code-Review+1 Not sure why only a change to documentation causes the system to be

Change in asterixdb[master]: [NO ISSUE] Interval join tests written in SQL++

2019-01-16 Thread Preston Carman (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/3111 to look at the new patch set (#3). Change subject: [NO ISSUE] Interval join tests written in SQL++

Change in asterixdb[master]: [NO ISSUE] Interval join tests written in SQL++

2019-01-14 Thread Preston Carman (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/3111 to look at the new patch set (#2). Change subject: [NO ISSUE] Interval join tests written in SQL++

Change in asterixdb[master]: Set GlobalJobDataFactory on JobSpec

2017-05-07 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: Set GlobalJobDataFactory on JobSpec .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1726 To unsubscribe, visit

Change in asterixdb[master]: Range connector updates to support interval partitioning.

2016-10-17 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/803 to look at the new patch set (#4). Change subject: Range connector updates to support interval partitioning. .. Range

Change in asterixdb[master]: Adding three interval join algorithms.

2016-10-04 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/457 to look at the new patch set (#25). Change subject: Adding three interval join algorithms. .. Adding three interval

Change in asterixdb[master]: Interval join test questions for AQL.

2016-10-04 Thread Preston Carman (Code Review)
Preston Carman has submitted this change and it was merged. Change subject: Interval join test questions for AQL. .. Interval join test questions for AQL. The system had tests for checking the interval comparison functions,

Change in asterixdb[master]: Interval join test questions for AQL.

2016-10-03 Thread Preston Carman (Code Review)
Hello Steven Jacobs, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1234 to look at the new patch set (#4). Change subject: Interval join test questions for AQL. .. Interval

Change in asterixdb[master]: Interval join test questions for AQL.

2016-10-03 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: Interval join test questions for AQL. .. Patch Set 2: (1 comment)

Change in asterixdb[master]: Interval join test questions for AQL.

2016-10-03 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1234 to look at the new patch set (#3). Change subject: Interval join test questions for AQL. .. Interval join test

Change in asterixdb[master]: Interval join test questions for AQL.

2016-09-30 Thread Preston Carman (Code Review)
Preston Carman has uploaded a new patch set (#2). Change subject: Interval join test questions for AQL. .. Interval join test questions for AQL. Change-Id: I3e492d1afba693a50cb75918399c26b8ce19899a --- M

Change in asterixdb[master]: Interval join test questions for AQL.

2016-09-30 Thread Preston Carman (Code Review)
Preston Carman has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1234 Change subject: Interval join test questions for AQL. .. Interval join test questions for AQL. Change-Id:

Change in asterixdb[master]: Adding three interval join algorithms.

2016-09-30 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/457 to look at the new patch set (#24). Change subject: Adding three interval join algorithms. .. Adding three interval

Change in asterixdb[master]: Updated DeletableFrameTupleAppender to support reusing index...

2016-09-28 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: Updated DeletableFrameTupleAppender to support reusing index slots. .. Patch Set 1: (9 comments)

Change in asterixdb[master]: Updated DeletableFrameTupleAppender to support reusing index...

2016-09-26 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: Updated DeletableFrameTupleAppender to support reusing index slots. .. Patch Set 1: (1 comment)

Change in asterixdb[master]: Updated DeletableFrameTupleAppender to support reusing index...

2016-09-26 Thread Preston Carman (Code Review)
Preston Carman has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1214 Change subject: Updated DeletableFrameTupleAppender to support reusing index slots. .. Updated DeletableFrameTupleAppender to

Change in asterixdb[master]: Code clean up and sonar fixes.

2016-09-08 Thread Preston Carman (Code Review)
Preston Carman has submitted this change and it was merged. Change subject: Code clean up and sonar fixes. .. Code clean up and sonar fixes. Change-Id: I55e2fbca5480ede8ff6bf47e843ca15c3d9d26d4 Reviewed-on:

Change in asterixdb[master]: Code clean up and sonar fixes.

2016-09-08 Thread Preston Carman (Code Review)
Preston Carman has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1157 Change subject: Code clean up and sonar fixes. .. Code clean up and sonar fixes. Change-Id:

Change in asterixdb[master]: ASTERIXDB-1635 Fix for overlap-bins start for dates.

2016-09-08 Thread Preston Carman (Code Review)
Hello Steven Jacobs, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1151 to look at the new patch set (#2). Change subject: ASTERIXDB-1635 Fix for overlap-bins start for dates.

Change in asterixdb[master]: Adding three interval join algorithms.

2016-09-07 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/457 to look at the new patch set (#23). Change subject: Adding three interval join algorithms. .. Adding three interval

Change in asterixdb[master]: ASTERIXDB-1635 Fix for overlap-bins start for dates.

2016-09-06 Thread Preston Carman (Code Review)
Preston Carman has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1151 Change subject: ASTERIXDB-1635 Fix for overlap-bins start for dates. .. ASTERIXDB-1635 Fix for overlap-bins start for dates.

Change in asterixdb[master]: RangeGenerator aggfunc for the numeric/asciiString datatype ...

2016-09-02 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: RangeGenerator aggfunc for the numeric/asciiString datatype based on parallel streaming histogram. The following commits from your working branch will be included:

Change in asterixdb[master]: ASTERIXDB-1608, ASTERIXDB-1617 Match user query for nonpure ...

2016-09-02 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: ASTERIXDB-1608, ASTERIXDB-1617 Match user query for nonpure function calls .. Patch Set 7: Is this change ready now? -- To view, visit

Change in asterixdb[master]: RangeGenerator aggfunc for the numeric/asciiString datatype ...

2016-08-25 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: RangeGenerator aggfunc for the numeric/asciiString datatype based on parallel streaming histogram. The following commits from your working branch will be included:

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]: ASTERIXDB-1587 Made upsert look in the correct dataverse for...

2016-08-17 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: ASTERIXDB-1587 Made upsert look in the correct dataverse for datatypes .. Patch Set 1: Code-Review+2 Change looks good. Just confirm the tests pass. -- To

Change in asterixdb[master]: Range connector updates to support interval partitioning.

2016-07-17 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/803 to look at the new patch set (#2). Change subject: Range connector updates to support interval partitioning. .. Range

Change in asterixdb[master]: Adding three interval join algorithms.

2016-07-16 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/457 to look at the new patch set (#22). Change subject: Adding three interval join algorithms. .. Adding three interval

Change in asterixdb[master]: Sonar fixes and other clean up items.

2016-07-16 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: Sonar fixes and other clean up items. .. Patch Set 6: (1 comment) Those changes can be saved for an other round of clean up.

Change in asterixdb[master]: Sonar fixes and other clean up items.

2016-07-16 Thread Preston Carman (Code Review)
Preston Carman has submitted this change and it was merged. Change subject: Sonar fixes and other clean up items. .. Sonar fixes and other clean up items. Change-Id: Ieb72a0fef3c181282cc4c71b881a7cb52da86818 Reviewed-on:

Change in asterixdb[master]: Sonar fixes and other clean up items.

2016-07-16 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: Sonar fixes and other clean up items. .. Patch Set 4: (1 comment)

Change in asterixdb[master]: Sonar fixes and other clean up items.

2016-07-15 Thread Preston Carman (Code Review)
Preston Carman has posted comments on this change. Change subject: Sonar fixes and other clean up items. .. Patch Set 4: (4 comments)

Change in asterixdb[master]: Sonar fixes and other clean up items.

2016-07-15 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/999 to look at the new patch set (#5). Change subject: Sonar fixes and other clean up items. .. Sonar fixes and other

Change in asterixdb[master]: Sonar fixes and other clean up items.

2016-07-14 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/999 to look at the new patch set (#4). Change subject: Sonar fixes and other clean up items. .. Sonar fixes and other

Change in asterixdb[master]: Sonar fixes and other clean up items.

2016-07-14 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/999 to look at the new patch set (#3). Change subject: Sonar fixes and other clean up items. .. Sonar fixes and other

Change in asterixdb[master]: Sonar fixes and other clean up items.

2016-07-14 Thread Preston Carman (Code Review)
Preston Carman has uploaded a new patch set (#2). Change subject: Sonar fixes and other clean up items. .. Sonar fixes and other clean up items. Change-Id: Ieb72a0fef3c181282cc4c71b881a7cb52da86818 --- M

Change in asterixdb[master]: Sonar fixes and other clean up items.

2016-07-14 Thread Preston Carman (Code Review)
Preston Carman has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/999 Change subject: Sonar fixes and other clean up items. .. Sonar fixes and other clean up items. Change-Id:

Change in asterixdb[master]: Moved array list classes to hyracks-storage-common.

2016-07-14 Thread Preston Carman (Code Review)
Preston Carman has submitted this change and it was merged. Change subject: Moved array list classes to hyracks-storage-common. .. Moved array list classes to hyracks-storage-common. * The move allows more projects to reused

Change in asterixdb[master]: Moved array list classes to hyracks-storage-common.

2016-07-14 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/996 to look at the new patch set (#2). Change subject: Moved array list classes to hyracks-storage-common. .. Moved array

Change in asterixdb[master]: Updated exchange operator names to include 'exchange' in the...

2016-07-14 Thread Preston Carman (Code Review)
Preston Carman has submitted this change and it was merged. Change subject: Updated exchange operator names to include 'exchange' in the class name. .. Updated exchange operator names to include 'exchange' in the class name.

Change in asterixdb[master]: Updated exchange operator names to include 'exchange' in the...

2016-07-14 Thread Preston Carman (Code Review)
Hello Yingyi Bu, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/995 to look at the new patch set (#2). Change subject: Updated exchange operator names to include 'exchange' in the class name.

Change in asterixdb[master]: Updated exchange operator names to include 'exchange' in the...

2016-07-14 Thread Preston Carman (Code Review)
Preston Carman has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/995 Change subject: Updated exchange operator names to include 'exchange' in the class name. .. Updated exchange operator names to

Change in asterixdb[master]: Adding three interval join algorithms.

2016-07-11 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/457 to look at the new patch set (#20). Change subject: Adding three interval join algorithms. .. Adding three interval

Change in asterixdb[master]: Fixed the nested loop join algorithm to use join memory para...

2016-06-21 Thread Preston Carman (Code Review)
Preston Carman has submitted this change and it was merged. Change subject: Fixed the nested loop join algorithm to use join memory parameter. .. Fixed the nested loop join algorithm to use join memory parameter. Change-Id:

Change in asterixdb[master]: Fixed the nested loop join algorithm to use join memory para...

2016-06-21 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/953 to look at the new patch set (#2). Change subject: Fixed the nested loop join algorithm to use join memory parameter.

Change in asterixdb[master]: Adding functions to return unit-time from date, datetime, an...

2016-06-07 Thread Preston Carman (Code Review)
Preston Carman has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/913 Change subject: Adding functions to return unit-time from date, datetime, and time. .. Adding functions to return unit-time from

Change in asterixdb[master]: Adding three interval join algorithms.

2016-06-03 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/457 to look at the new patch set (#19). Change subject: Adding three interval join algorithms. .. Adding three interval

Change in asterixdb[master]: Adding three interval join algorithms.

2016-05-27 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/457 to look at the new patch set (#18). Change subject: Adding three interval join algorithms. .. Adding three interval

Change in asterixdb[master]: Adding three interval join algorithms.

2016-05-27 Thread Preston Carman (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/457 to look at the new patch set (#17). Change subject: Adding three interval join algorithms. .. Adding three interval