[GitHub] [spark] LuciferYang commented on pull request #31374: [SPARK-34275][CORE][SQL][MLLIB] Replaces filter and size with count

2021-01-27 Thread GitBox


LuciferYang commented on pull request #31374:
URL: https://github.com/apache/spark/pull/31374#issuecomment-768832611


   thx @HyukjinKwon 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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



[GitHub] [spark] LuciferYang commented on pull request #31374: [SPARK-34275][CORE][SQL][MLLIB] Replaces filter and size with count

2021-01-27 Thread GitBox


LuciferYang commented on pull request #31374:
URL: https://github.com/apache/spark/pull/31374#issuecomment-768823455


   @HyukjinKwon 
   
   - branch-3.0: https://github.com/apache/spark/pull/31375
   - branch-2.4: https://github.com/apache/spark/pull/31376
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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



[GitHub] [spark] LuciferYang commented on pull request #31374: [SPARK-34275][CORE][SQL][MLLIB] Replaces filter and size with count

2021-01-27 Thread GitBox


LuciferYang commented on pull request #31374:
URL: https://github.com/apache/spark/pull/31374#issuecomment-768812793


   > Yeah, let's do that. When we fix such kind of things, the conflicts are 
one of the usual problems pointed out by people. It should be better to port 
them back.
   
   OK



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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



[GitHub] [spark] LuciferYang commented on pull request #31374: [SPARK-34275][CORE][SQL][MLLIB] Replaces filter and size with count

2021-01-27 Thread GitBox


LuciferYang commented on pull request #31374:
URL: https://github.com/apache/spark/pull/31374#issuecomment-768812492


   > BTW, they are all instances, right?
   Yes



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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



[GitHub] [spark] LuciferYang commented on pull request #31374: [SPARK-34275][CORE][SQL][MLLIB] Replaces filter and size with count

2021-01-27 Thread GitBox


LuciferYang commented on pull request #31374:
URL: https://github.com/apache/spark/pull/31374#issuecomment-768793669


   If need other PR for branch-3.0 and branch-2.4, please ping me



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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