Uploaded image for project: 'Tajo'
  1. Tajo
  2. TAJO-1032

Improve TravisCI scripts to adjust log4j log level

    Details

    • Type: Task
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.10.0
    • Component/s: Build, Unit Test
    • Labels:
      None

      Description

      Since Travi CI limits the size of standard output log up to 4MB, our Travis CI script prints out only selected lines of unit tests by using grep command. Although this manner limits the output log size, it makes error diagnosis very hard. So, we need to solve it.

      One suggestion is to adjust the log level of log4j used for unit tests instead of grep commend. I expect that this manner will limit the log size as well as make error diagnosis easier.

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-CODEGEN-build #77 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/77/)
        TAJO-1032: Improve TravisCI scripts to adjust log4j log level. (jinho) (jhkim: rev 080f4e1454fb8dfabe1c15e58d3c1eff8ecd6f6d)

        • .travis.yml
        • CHANGES
        • tajo-core/src/test/java/org/apache/tajo/TajoTestingCluster.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-CODEGEN-build #77 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/77/ ) TAJO-1032 : Improve TravisCI scripts to adjust log4j log level. (jinho) (jhkim: rev 080f4e1454fb8dfabe1c15e58d3c1eff8ecd6f6d) .travis.yml CHANGES tajo-core/src/test/java/org/apache/tajo/TajoTestingCluster.java
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #435 (See https://builds.apache.org/job/Tajo-master-build/435/)
        TAJO-1032: Improve TravisCI scripts to adjust log4j log level. (jinho) (jhkim: rev 080f4e1454fb8dfabe1c15e58d3c1eff8ecd6f6d)

        • .travis.yml
        • CHANGES
        • tajo-core/src/test/java/org/apache/tajo/TajoTestingCluster.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #435 (See https://builds.apache.org/job/Tajo-master-build/435/ ) TAJO-1032 : Improve TravisCI scripts to adjust log4j log level. (jinho) (jhkim: rev 080f4e1454fb8dfabe1c15e58d3c1eff8ecd6f6d) .travis.yml CHANGES tajo-core/src/test/java/org/apache/tajo/TajoTestingCluster.java
        Hide
        jhkim Jinho Kim added a comment -

        committed it!
        Thank you for quick review!

        Show
        jhkim Jinho Kim added a comment - committed it! Thank you for quick review!
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

        https://github.com/apache/tajo/pull/226

        Show
        githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/tajo/pull/226
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/226#issuecomment-61604640

        +1

        The change looks straightforward.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/226#issuecomment-61604640 +1 The change looks straightforward.
        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user jinossy opened a pull request:

        https://github.com/apache/tajo/pull/226

        TAJO-1032: Improve TravisCI scripts to adjust log4j log level

        I've change the logging level to 'mvn clean install -DLOG_LEVEL=WARN'

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/jinossy/tajo TAJO-1032

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/tajo/pull/226.patch

        To close this pull request, make a commit to your master/trunk branch
        with (at least) the following in the commit message:

        This closes #226


        commit 9eb88233fedddcc6c06a9ff82b3f43e70483a47e
        Author: jhkim <jhkim@apache.org>
        Date: 2014-11-04T06:32:36Z

        TAJO-1032: Improve TravisCI scripts to adjust log4j log level

        commit cf47b212aa85156bac75ff121f87e9c1aac9d1f7
        Author: jhkim <jhkim@apache.org>
        Date: 2014-11-04T06:35:16Z

        remove double quote


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user jinossy opened a pull request: https://github.com/apache/tajo/pull/226 TAJO-1032 : Improve TravisCI scripts to adjust log4j log level I've change the logging level to 'mvn clean install -DLOG_LEVEL=WARN' You can merge this pull request into a Git repository by running: $ git pull https://github.com/jinossy/tajo TAJO-1032 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/226.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #226 commit 9eb88233fedddcc6c06a9ff82b3f43e70483a47e Author: jhkim <jhkim@apache.org> Date: 2014-11-04T06:32:36Z TAJO-1032 : Improve TravisCI scripts to adjust log4j log level commit cf47b212aa85156bac75ff121f87e9c1aac9d1f7 Author: jhkim <jhkim@apache.org> Date: 2014-11-04T06:35:16Z remove double quote

          People

          • Assignee:
            jhkim Jinho Kim
            Reporter:
            hyunsik Hyunsik Choi
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development