Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0, 0.9.0
    • Fix Version/s: 0.8.0, 0.9.0
    • Component/s: conf and scripts
    • Labels:
      None

      Description

      Currently Hyoungjun improve TajoCli and log4j, it would be better to seperate this.
      please refer to TAJO-734

        Issue Links

          Activity

          Hide
          jhkim Jinho Kim added a comment -

          Created a review request against branch master in reviewboard
          https://reviews.apache.org/r/20442/

          Show
          jhkim Jinho Kim added a comment - Created a review request against branch master in reviewboard https://reviews.apache.org/r/20442/
          Hide
          tajoqa Tajo QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12640584/TAJO-768.patch
          against master revision 68f9dee.

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. The applied patch does not increase the total number of javadoc warnings.

          +1 checkstyle. The patch generated 0 code style errors.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed unit tests in tajo-dist.

          Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/348//testReport/
          Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/348//console

          This message is automatically generated.

          Show
          tajoqa Tajo QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12640584/TAJO-768.patch against master revision 68f9dee. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 javadoc. The applied patch does not increase the total number of javadoc warnings. +1 checkstyle. The patch generated 0 code style errors. +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed unit tests in tajo-dist. Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/348//testReport/ Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/348//console This message is automatically generated.
          Hide
          jhkim Jinho Kim added a comment -

          +1 for the patch.
          Thank you for your contribution! I'll commit it

          Show
          jhkim Jinho Kim added a comment - +1 for the patch. Thank you for your contribution! I'll commit it
          Hide
          jhkim Jinho Kim added a comment -

          I've just committed it to both master branch and branch-0.8.0.

          Show
          jhkim Jinho Kim added a comment - I've just committed it to both master branch and branch-0.8.0.
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Tajo-master-build #177 (See https://builds.apache.org/job/Tajo-master-build/177/)
          TAJO-768: Improve the log4j configuration. (hyoungjunkim via jinho) (jinossy: rev 0a76c998ed65dfed141a552a0ddb8e7ca23903a7)

          • CHANGES.txt
          • tajo-dist/src/main/bin/tajo
          • tajo-dist/src/main/conf/log4j.properties
          • tajo-dist/src/main/bin/tajo-daemon.sh
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #177 (See https://builds.apache.org/job/Tajo-master-build/177/ ) TAJO-768 : Improve the log4j configuration. (hyoungjunkim via jinho) (jinossy: rev 0a76c998ed65dfed141a552a0ddb8e7ca23903a7) CHANGES.txt tajo-dist/src/main/bin/tajo tajo-dist/src/main/conf/log4j.properties tajo-dist/src/main/bin/tajo-daemon.sh
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Tajo-0.8.0-build #76 (See https://builds.apache.org/job/Tajo-0.8.0-build/76/)
          TAJO-768: Improve the log4j configuration. (hyoungjunkim via jinho) (jinossy: rev 99c5c33b91bff9e55c9259686e66e0f8342f77f3)

          • tajo-dist/src/main/bin/tajo-daemon.sh
          • CHANGES.txt
          • tajo-dist/src/main/bin/tajo
          • tajo-dist/src/main/conf/log4j.properties
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Tajo-0.8.0-build #76 (See https://builds.apache.org/job/Tajo-0.8.0-build/76/ ) TAJO-768 : Improve the log4j configuration. (hyoungjunkim via jinho) (jinossy: rev 99c5c33b91bff9e55c9259686e66e0f8342f77f3) tajo-dist/src/main/bin/tajo-daemon.sh CHANGES.txt tajo-dist/src/main/bin/tajo tajo-dist/src/main/conf/log4j.properties

            People

            • Assignee:
              hjkim Hyoungjun Kim
              Reporter:
              jhkim Jinho Kim
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development