[GitHub] spark issue #22254: [CATALYST] Add a call to apply method explicitly in comb...

2018-08-28 Thread SongYadong
Github user SongYadong commented on the issue: https://github.com/apache/spark/pull/22254 OK, I will close it. thanks for your review. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #22254: [CATALYST] Add a call to apply method explicitly in comb...

2018-08-28 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/22254 Such style changes alone are discouraged - trivial and not worth enough for a PR. Shall we close this and fix them when you touch other codes round here next time, or suggest to fix it later in

[GitHub] spark issue #22254: [CATALYST] Add a call to apply method explicitly in comb...

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

[GitHub] spark issue #22254: [CATALYST] Add a call to apply method explicitly in comb...

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

[GitHub] spark issue #22254: [CATALYST] Add a call to apply method explicitly in comb...

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