Change in asterixdb-bad[master]: [NO ISSUE][BAD] Coordinated change for window functions (AST...

2019-01-03 Thread Dmitry Lychagin (Code Review) 
Dmitry Lychagin has submitted this change and it was merged.

Change subject: [NO ISSUE][BAD] Coordinated change for window functions 
(ASTERIXDB-2488)
..


[NO ISSUE][BAD] Coordinated change for window functions (ASTERIXDB-2488)

Change-Id: I2799ef800c67f76d2d6dda17c9e619c63b0bcf4a
---
M 
asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced-index-only.plan
M 
asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced.plan
M asterix-bad/src/test/resources/optimizerts/results/channel/channel-create.plan
M asterix-bad/src/test/resources/optimizerts/results/channel/channel-push.plan
M 
asterix-bad/src/test/resources/optimizerts/results/channel/channel-subscribe.plan
M 
asterix-bad/src/test/resources/optimizerts/results/channel/channel-unsubscribe.plan
6 files changed, 70 insertions(+), 70 deletions(-)

Approvals:
  Jenkins: Verified
  Dmitry Lychagin: Looks good to me, approved



diff --git 
a/asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced-index-only.plan
 
b/asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced-index-only.plan
index 308c8b1..8786d79 100644
--- 
a/asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced-index-only.plan
+++ 
b/asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced-index-only.plan
@@ -1,17 +1,17 @@
 -- NOTIFY_BROKERS  |PARTITIONED|
   -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
--- PRE_CLUSTERED_GROUP_BY[$$141, $$channelExecutionTime]  |PARTITIONED|
+-- PRE_CLUSTERED_GROUP_BY[$$150, $$channelExecutionTime]  |PARTITIONED|
 {
   -- AGGREGATE  |LOCAL|
 -- NESTED_TUPLE_SOURCE  |LOCAL|
 }
   -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
--- STABLE_SORT [$$141(ASC), $$channelExecutionTime(ASC)]  |PARTITIONED|
-  -- HASH_PARTITION_EXCHANGE [$$141, $$channelExecutionTime]  
|PARTITIONED|
+-- STABLE_SORT [$$150(ASC), $$channelExecutionTime(ASC)]  |PARTITIONED|
+  -- HASH_PARTITION_EXCHANGE [$$150, $$channelExecutionTime]  
|PARTITIONED|
 -- PRE_SORTED_DISTINCT_BY  |PARTITIONED|
   -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
--- STABLE_SORT [$$116(ASC)]  |PARTITIONED|
-  -- HASH_PARTITION_EXCHANGE [$$116]  |PARTITIONED|
+-- STABLE_SORT [$$125(ASC)]  |PARTITIONED|
+  -- HASH_PARTITION_EXCHANGE [$$125]  |PARTITIONED|
 -- STREAM_PROJECT  |PARTITIONED|
   -- COMMIT  |PARTITIONED|
 -- STREAM_PROJECT  |PARTITIONED|
@@ -21,7 +21,7 @@
 -- STREAM_PROJECT  |PARTITIONED|
   -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
 -- INSERT_DELETE  |PARTITIONED|
-  -- HASH_PARTITION_EXCHANGE [$$111]  
|PARTITIONED|
+  -- HASH_PARTITION_EXCHANGE [$$120]  
|PARTITIONED|
 -- ASSIGN  |PARTITIONED|
   -- STREAM_PROJECT  |PARTITIONED|
 -- ASSIGN  |PARTITIONED|
@@ -33,12 +33,12 @@
 -- STREAM_SELECT  
|PARTITIONED|
   -- STREAM_PROJECT  
|PARTITIONED|
 -- 
ONE_TO_ONE_EXCHANGE  |PARTITIONED|
-  -- 
HYBRID_HASH_JOIN [$$126][$$125]  |PARTITIONED|
--- 
HASH_PARTITION_EXCHANGE [$$126]  |PARTITIONED|
+  -- 
HYBRID_HASH_JOIN [$$135][$$134]  |PARTITIONED|
+-- 
HASH_PARTITION_EXCHANGE [$$135]  |PARTITIONED|
   -- 
STREAM_PROJECT  |PARTITIONED|
 -- 
ONE_TO_ONE_EXCHANGE  |PARTITIONED|
-  -- 
HYBRID_HASH_JOIN [$$124, $$122][$$117, $$118]  |PARTITIONED|
--- 
HASH_PARTITION_EXCHANGE [$$124, $$122]  |PARTITIONED|
+  -- 
HYBRID_HASH_JOIN [$$133, $$131][$$126, $$127]  |PARTITIONED|
+-- 
HASH_PARTITION_EXCHANGE [$$133, $$131]  |PARTITIONED|
   -- 
STREAM_PROJECT  |PARTITIONED|

Change in asterixdb-bad[master]: [NO ISSUE][BAD] Coordinated change for window functions (AST...

2019-01-03 Thread Dmitry Lychagin (Code Review) 
Dmitry Lychagin has posted comments on this change.

Change subject: [NO ISSUE][BAD] Coordinated change for window functions 
(ASTERIXDB-2488)
..


Patch Set 2: Code-Review+2

Fwd Xikui's +2

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/3051
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2799ef800c67f76d2d6dda17c9e619c63b0bcf4a
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Ali Alsuliman 
Gerrit-Reviewer: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Xikui Wang 
Gerrit-HasComments: No

Change in asterixdb-bad[master]: [NO ISSUE][BAD] Coordinated change for window functions (AST...

2018-12-28 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][BAD] Coordinated change for window functions 
(ASTERIXDB-2488)
..


Patch Set 2:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/365/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/3051
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2799ef800c67f76d2d6dda17c9e619c63b0bcf4a
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Xikui Wang 
Gerrit-HasComments: No

Change in asterixdb-bad[master]: [NO ISSUE][BAD] Coordinated change for window functions (AST...

2018-12-05 Thread Xikui Wang (Code Review) 
Xikui Wang has posted comments on this change.

Change subject: [NO ISSUE][BAD] Coordinated change for window functions 
(ASTERIXDB-2488)
..


Patch Set 1: Code-Review+2

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/3051
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2799ef800c67f76d2d6dda17c9e619c63b0bcf4a
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Xikui Wang 
Gerrit-HasComments: No

Change in asterixdb-bad[master]: [NO ISSUE][BAD] Coordinated change for window functions (AST...

2018-12-04 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][BAD] Coordinated change for window functions 
(ASTERIXDB-2488)
..


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/360/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/3051
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2799ef800c67f76d2d6dda17c9e619c63b0bcf4a
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb-bad[master]: [NO ISSUE][BAD] Coordinated change for window functions (AST...

2018-12-04 Thread Dmitry Lychagin (Code Review) 
Dmitry Lychagin has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/3051

Change subject: [NO ISSUE][BAD] Coordinated change for window functions 
(ASTERIXDB-2488)
..

[NO ISSUE][BAD] Coordinated change for window functions (ASTERIXDB-2488)

Change-Id: I2799ef800c67f76d2d6dda17c9e619c63b0bcf4a
---
M 
asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced-index-only.plan
M 
asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced.plan
M asterix-bad/src/test/resources/optimizerts/results/channel/channel-create.plan
M asterix-bad/src/test/resources/optimizerts/results/channel/channel-push.plan
M 
asterix-bad/src/test/resources/optimizerts/results/channel/channel-subscribe.plan
M 
asterix-bad/src/test/resources/optimizerts/results/channel/channel-unsubscribe.plan
6 files changed, 70 insertions(+), 70 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad 
refs/changes/51/3051/1

diff --git 
a/asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced-index-only.plan
 
b/asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced-index-only.plan
index 308c8b1..8786d79 100644
--- 
a/asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced-index-only.plan
+++ 
b/asterix-bad/src/test/resources/optimizerts/results/channel/channel-advanced-index-only.plan
@@ -1,17 +1,17 @@
 -- NOTIFY_BROKERS  |PARTITIONED|
   -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
--- PRE_CLUSTERED_GROUP_BY[$$141, $$channelExecutionTime]  |PARTITIONED|
+-- PRE_CLUSTERED_GROUP_BY[$$150, $$channelExecutionTime]  |PARTITIONED|
 {
   -- AGGREGATE  |LOCAL|
 -- NESTED_TUPLE_SOURCE  |LOCAL|
 }
   -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
--- STABLE_SORT [$$141(ASC), $$channelExecutionTime(ASC)]  |PARTITIONED|
-  -- HASH_PARTITION_EXCHANGE [$$141, $$channelExecutionTime]  
|PARTITIONED|
+-- STABLE_SORT [$$150(ASC), $$channelExecutionTime(ASC)]  |PARTITIONED|
+  -- HASH_PARTITION_EXCHANGE [$$150, $$channelExecutionTime]  
|PARTITIONED|
 -- PRE_SORTED_DISTINCT_BY  |PARTITIONED|
   -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
--- STABLE_SORT [$$116(ASC)]  |PARTITIONED|
-  -- HASH_PARTITION_EXCHANGE [$$116]  |PARTITIONED|
+-- STABLE_SORT [$$125(ASC)]  |PARTITIONED|
+  -- HASH_PARTITION_EXCHANGE [$$125]  |PARTITIONED|
 -- STREAM_PROJECT  |PARTITIONED|
   -- COMMIT  |PARTITIONED|
 -- STREAM_PROJECT  |PARTITIONED|
@@ -21,7 +21,7 @@
 -- STREAM_PROJECT  |PARTITIONED|
   -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
 -- INSERT_DELETE  |PARTITIONED|
-  -- HASH_PARTITION_EXCHANGE [$$111]  
|PARTITIONED|
+  -- HASH_PARTITION_EXCHANGE [$$120]  
|PARTITIONED|
 -- ASSIGN  |PARTITIONED|
   -- STREAM_PROJECT  |PARTITIONED|
 -- ASSIGN  |PARTITIONED|
@@ -33,12 +33,12 @@
 -- STREAM_SELECT  
|PARTITIONED|
   -- STREAM_PROJECT  
|PARTITIONED|
 -- 
ONE_TO_ONE_EXCHANGE  |PARTITIONED|
-  -- 
HYBRID_HASH_JOIN [$$126][$$125]  |PARTITIONED|
--- 
HASH_PARTITION_EXCHANGE [$$126]  |PARTITIONED|
+  -- 
HYBRID_HASH_JOIN [$$135][$$134]  |PARTITIONED|
+-- 
HASH_PARTITION_EXCHANGE [$$135]  |PARTITIONED|
   -- 
STREAM_PROJECT  |PARTITIONED|
 -- 
ONE_TO_ONE_EXCHANGE  |PARTITIONED|
-  -- 
HYBRID_HASH_JOIN [$$124, $$122][$$117, $$118]  |PARTITIONED|
--- 
HASH_PARTITION_EXCHANGE [$$124, $$122]  |PARTITIONED|
+  -- 
HYBRID_HASH_JOIN [$$133, $$131][$$126, $$127]  |PARTITIONED|
+-- 
HASH_PARTITION_EXCHANGE [$$133, $$131]  |PARTITIONED|
   -- 
STREAM_PROJECT  |PARTITIONED|