[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/20242 Great! Thank you! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user sameeragarwal commented on the issue: https://github.com/apache/spark/pull/20242 I've also updated the relevant jenkins job configs to automatically check for this in future (e.g., see https://amplab.cs.berkeley.edu/jenkins/job/spark-branch-2.3-lint/113/console). Thanks

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/20242 Thank you all! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user sameeragarwal commented on the issue: https://github.com/apache/spark/pull/20242 merging to master/2.3. Thanks! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20242 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86037/ Test PASSed. ---

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20242 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

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

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

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

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/20242 Retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/20242 This is the all for RC1 tag. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20242 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86015/ Test FAILed. ---

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20242 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

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

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user sameeragarwal commented on the issue: https://github.com/apache/spark/pull/20242 Thanks @dongwang218, LGTM It seems like the java linter checks are not included in https://amplab.cs.berkeley.edu/jenkins/job/spark-branch-2.3-lint/. I'll update the scripts so that

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user jerryshao commented on the issue: https://github.com/apache/spark/pull/20242 LGTM, @dongjoon-hyun is the current changes include all the lint issues, or you still have further changes? --- - To

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/20242 Thank you for review and approval, @HyukjinKwon . --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/20242 cc @srowen , @HyukjinKwon , @sameeragarwal . --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #20242: [MINOR][BUILD] Fix Java linter errors

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