{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/12980","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/12980/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/12980/comments","events_url":"https://api.github.com/repos/apache/spark/issues/12980/events","html_url":"https://github.com/apache/spark/pull/12980","id":153629534,"number":12980,"title":"[SPARK-15207][BUILD] Use Travis CI for Java Linter and JDK7/8 compilation test","user":{"login":"dongjoon-hyun","id":9700541,"avatar_url":"https://avatars.githubusercontent.com/u/9700541?v=3","gravatar_id":"","url":"https://api.github.com/users/dongjoon-hyun","html_url":"https://github.com/dongjoon-hyun","followers_url":"https://api.github.com/users/dongjoon-hyun/followers","following_url":"https://api.github.com/users/dongjoon-hyun/following{/other_user}","gists_url":"https://api.github.com/users/dongjoon-hyun/gists{/gist_id}","starred_url":"https://api.github.com/users/dongjoon-hyun/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dongjoon-hyun/subscriptions","organizations_url":"https://api.github.com/users/dongjoon-hyun/orgs","repos_url":"https://api.github.com/users/dongjoon-hyun/repos","events_url":"https://api.github.com/users/dongjoon-hyun/events{/privacy}","received_events_url":"https://api.github.com/users/dongjoon-hyun/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":23,"created_at":"2016-05-08T01:55:34Z","updated_at":"2016-05-10T06:19:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/12980","html_url":"https://github.com/apache/spark/pull/12980","diff_url":"https://github.com/apache/spark/pull/12980.diff","patch_url":"https://github.com/apache/spark/pull/12980.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\nCurrently, Java Linter is disabled in Jenkins tests.\r\n\r\nhttps://github.com/apache/spark/blob/master/dev/run-tests.py#L554\r\n\r\nHowever, as of today, Spark has 721 java files with 97362 code (without blank/comments). It's about 1/3 of Scala.\r\n```\r\n--------------------------------------------------------------------------------\r\nLanguage                      files          blank        comment           code\r\n--------------------------------------------------------------------------------\r\nScala                          2353          62819         124060         318747\r\nJava                            721          18617          23314          97362\r\n```\r\n\r\nThis PR aims to take advantage of Travis CI to handle the following static analysis by adding a single file, `.travis.yml` without any additional burden on the existing servers.\r\n\r\n- Java Linter\r\n- JDK7/JDK8 maven compile\r\n\r\nNote that this PR does not propose to remove some of the above work items from the Jenkins. It's possible, but we need to observe the Travis CI stability for a while. The main goal of this issue is to remove committer's overhead on linter-related PRs (the original PR and the fixation PR).\r\n\r\n## How was this patch tested?\r\n\r\nPass the Travis CI tests. Please see the following link.\r\n\r\nhttps://travis-ci.org/dongjoon-hyun/spark/builds/128595350\r\nhttps://travis-ci.org/dongjoon-hyun/spark/builds/128708372"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/218068656","html_url":"https://github.com/apache/spark/pull/12980#issuecomment-218068656","issue_url":"https://api.github.com/repos/apache/spark/issues/12980","id":218068656,"user":{"login":"SparkQA","id":8108735,"avatar_url":"https://avatars.githubusercontent.com/u/8108735?v=3","gravatar_id":"","url":"https://api.github.com/users/SparkQA","html_url":"https://github.com/SparkQA","followers_url":"https://api.github.com/users/SparkQA/followers","following_url":"https://api.github.com/users/SparkQA/following{/other_user}","gists_url":"https://api.github.com/users/SparkQA/gists{/gist_id}","starred_url":"https://api.github.com/users/SparkQA/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SparkQA/subscriptions","organizations_url":"https://api.github.com/users/SparkQA/orgs","repos_url":"https://api.github.com/users/SparkQA/repos","events_url":"https://api.github.com/users/SparkQA/events{/privacy}","received_events_url":"https://api.github.com/users/SparkQA/received_events","type":"User","site_admin":false},"created_at":"2016-05-10T06:19:24Z","updated_at":"2016-05-10T06:19:24Z","body":"**[Test build #58204 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58204/consoleFull)** for PR 12980 at commit [`2583abc`](https://github.com/apache/spark/commit/2583abc7926fd73a69e26861f618a9a0e4ae3de5).\n * This patch passes all tests.\n * This patch merges cleanly.\n * This patch adds no public classes."}}