Uploaded image for project: 'Spark'
  1. Spark
  2. SPARK-15207

Use Travis CI for Java Linter and JDK7/8 compilation test

    Details

    • Type: New Feature
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.1.0
    • Component/s: Build
    • Labels:
      None

      Description

      Currently, Java Linter is disabled in Jenkins tests.

      https://github.com/apache/spark/blob/master/dev/run-tests.py#L554

      However, as of today, Spark has 721 java files with 97362 code (without blank/comments). It's about 1/3 of Scala.

      --------------------------------------------------------------------------------
      Language                      files          blank        comment           code
      --------------------------------------------------------------------------------
      Scala                          2353          62819         124060         318747
      Java                            721          18617          23314          97362
      

      This issue 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.

      • Java Linter
      • JDK7/JDK8 maven compile

      Note that this issue 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 goal of this issue is to removing committer's overhead on linter-related PRs (the original PR and the fixation PR).

      By the way, historically, Spark used Travis CI before.

        Attachments

          Activity

            People

            • Assignee:
              dongjoon Dongjoon Hyun
              Reporter:
              dongjoon Dongjoon Hyun
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: