[GitHub] spark issue #22077: [SPARK-25084][SQL] "distribute by" on multiple columns (...

2018-08-11 Thread wangyum
Github user wangyum commented on the issue:

https://github.com/apache/spark/pull/22077
  
I’d prefer the title like this: `[BACKPORT-2.3][SPARK-25084][SQL]...`


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22077: [SPARK-25084][SQL] "distribute by" on multiple columns (...

2018-08-11 Thread yucai
Github user yucai commented on the issue:

https://github.com/apache/spark/pull/22077
  
ok to test.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22077: [SPARK-25084][SQL] "distribute by" on multiple columns (...

2018-08-11 Thread yucai
Github user yucai commented on the issue:

https://github.com/apache/spark/pull/22077
  
@cloud-fan Kindly help review.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22077: [SPARK-25084][SQL] "distribute by" on multiple columns (...

2018-08-11 Thread yucai
Github user yucai commented on the issue:

https://github.com/apache/spark/pull/22077
  
LGTM.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22077: [SPARK-25084][SQL] "distribute by" on multiple columns (...

2018-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22077
  
Can one of the admins verify this patch?


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22077: [SPARK-25084][SQL] "distribute by" on multiple columns (...

2018-08-11 Thread yucai
Github user yucai commented on the issue:

https://github.com/apache/spark/pull/22077
  
@LantaoJin, could you modify the title and description?


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22077: [SPARK-25084][SQL] "distribute by" on multiple columns (...

2018-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22077
  
Can one of the admins verify this patch?


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22077: [SPARK-25084][SQL] "distribute by" on multiple columns (...

2018-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22077
  
Can one of the admins verify this patch?


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22077: [SPARK-25084][SQL] "distribute by" on multiple columns (...

2018-08-11 Thread LantaoJin
Github user LantaoJin commented on the issue:

https://github.com/apache/spark/pull/22077
  
Thanks @yucai , please review this.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org