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

Redirect stdout of maven test to /dev/null in Travis CI script

    Details

    • Type: Task
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9.0
    • Component/s: Build
    • Labels:
      None

      Description

      As I mentioned in TAJO-887, Travis CI has some limitation where output log size < 4MB. Recently, output size is likely to exceed 4MB. This patch redirects stdout to /dev/null in order to reduce the log size. Because the stderr will be kept, we can still find the error causes from the message.

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user hyunsik opened a pull request:

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

        TAJO-890: Redirect stdout of maven test to /dev/null in Travis CI script...

        ....

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

        $ git pull https://github.com/hyunsik/tajo TAJO-890

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

        https://github.com/apache/tajo/pull/47.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 #47


        commit 7d4f2507308794ece46bed41a339e1bcbbe55367
        Author: Hyunsik Choi <hyunsik@apache.org>
        Date: 2014-06-25T02:37:32Z

        TAJO-890: Redirect stdout of maven test to /dev/null in Travis CI script.


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user hyunsik opened a pull request: https://github.com/apache/tajo/pull/47 TAJO-890 : Redirect stdout of maven test to /dev/null in Travis CI script... .... You can merge this pull request into a Git repository by running: $ git pull https://github.com/hyunsik/tajo TAJO-890 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/47.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 #47 commit 7d4f2507308794ece46bed41a339e1bcbbe55367 Author: Hyunsik Choi <hyunsik@apache.org> Date: 2014-06-25T02:37:32Z TAJO-890 : Redirect stdout of maven test to /dev/null in Travis CI script.
        Hide
        blrunner Jaehwa Jung added a comment -

        +1

        Thanks Hyunsik.
        It looks good and 'mvn clean install' finished successfully.
        Push it.

        Show
        blrunner Jaehwa Jung added a comment - +1 Thanks Hyunsik. It looks good and 'mvn clean install' finished successfully. Push it.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

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

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

        thank you for the quick review. committed it to master.

        Show
        hyunsik Hyunsik Choi added a comment - thank you for the quick review. committed it to master.
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #261 (See https://builds.apache.org/job/Tajo-master-build/261/)
        TAJO-890: Redirect stdout of maven test to /dev/null in Travis CI script. (hyunsik: rev 48c6d1a38fa28cef14bf610328004fc68a734b83)

        • CHANGES
        • .travis.yml
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #261 (See https://builds.apache.org/job/Tajo-master-build/261/ ) TAJO-890 : Redirect stdout of maven test to /dev/null in Travis CI script. (hyunsik: rev 48c6d1a38fa28cef14bf610328004fc68a734b83) CHANGES .travis.yml

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development