[GitHub] spark issue #23247: [SPARK-26294][CORE]Delete Unnecessary If statement

Github user srowen commented on the issue: https://github.com/apache/spark/pull/23247 Merged to master --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #23247: [SPARK-26294][CORE]Delete Unnecessary If statement

Github user wangjiaochun commented on the issue: https://github.com/apache/spark/pull/23247 ok! thanks, I will focus on commit bug later. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #23247: [SPARK-26294][CORE]Delete Unnecessary If statement

Github user wangjiaochun commented on the issue: https://github.com/apache/spark/pull/23247 > @wangjiaochun, I think you better stop fixing trivial stuff in each PR. Those stuff can be fixed when the codes around here is fixed, or let other people fix it later. ok! thanks, I

[GitHub] spark issue #23247: [SPARK-26294][CORE]Delete Unnecessary If statement

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/23247 **[Test build #4456 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4456/testReport)** for PR 23247 at commit

[GitHub] spark issue #23247: [SPARK-26294][CORE]Delete Unnecessary If statement

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/23247 **[Test build #4456 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4456/testReport)** for PR 23247 at commit

[GitHub] spark issue #23247: [SPARK-26294][CORE]Delete Unnecessary If statement

Github user srowen commented on the issue: https://github.com/apache/spark/pull/23247 These aren't worth the time it takes us to review them and merge them, honestly. Little cleanup can be OK if it makes an appreciable difference in speed or readability, and if you can find many

[GitHub] spark issue #23247: [SPARK-26294][CORE]Delete Unnecessary If statement

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/23247 @wangjiaochun, I think you better stop fixing trivial stuff in each PR. Those stuff can be fixed when the codes around here is fixed, or let other people fix it later. ---

[GitHub] spark issue #23247: [SPARK-26294][CORE]Delete Unnecessary If statement

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23247 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #23247: [SPARK-26294][CORE]Delete Unnecessary If statement

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23247 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #23247: [SPARK-26294][CORE]Delete Unnecessary If statement

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23247 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional