Hive
  1. Hive
  2. HIVE-4513

disable hivehistory logs by default

    Details

      Description

      HiveHistory log files (hive_job_log_hive_*.txt files) store information about hive query such as query string, plan , counters and MR job progress information.

      There is no mechanism to delete these files and as a result they get accumulated over time, using up lot of disk space.
      I don't think this is used by most people, so I think it would better to turn this off by default. Jobtracker logs already capture most of this information, though it is not as structured as history logs.

      1. HIVE-4513.6.patch
        43 kB
        Thejas M Nair
      2. HIVE-4513.5.patch
        42 kB
        Thejas M Nair
      3. HIVE-4513.4.patch
        38 kB
        Thejas M Nair
      4. HIVE-4513.3.patch
        38 kB
        Thejas M Nair
      5. HIVE-4513.2.patch
        38 kB
        Thejas M Nair
      6. HIVE-4513.1.patch
        37 kB
        Thejas M Nair

        Issue Links

          Activity

          Hide
          Thejas M Nair added a comment -

          Blocked by HIVE-4500 as it relies on config parameter being introduced in that patch.

          Show
          Thejas M Nair added a comment - Blocked by HIVE-4500 as it relies on config parameter being introduced in that patch.
          Hide
          Thejas M Nair added a comment -

          Having this on by default can cause problems if you have same machine and config being used for large number of queries. For example, while doing some stress tests on HiveServer2, the mount with /tmp became full because hive history files (around 20k of them) took 2.5 GB.

          Show
          Thejas M Nair added a comment - Having this on by default can cause problems if you have same machine and config being used for large number of queries. For example, while doing some stress tests on HiveServer2, the mount with /tmp became full because hive history files (around 20k of them) took 2.5 GB.
          Hide
          Thejas M Nair added a comment -

          HIVE-4513.1.patch - Uses a proxy class that makes HiveHistory no-op, if hive history is disabled.

          Show
          Thejas M Nair added a comment - HIVE-4513 .1.patch - Uses a proxy class that makes HiveHistory no-op, if hive history is disabled.
          Hide
          Thejas M Nair added a comment -

          HIVE-4513.2.patch -
          SessionState temp file gets created in same dir as hive history dir, but it was relying on hivehistory to create that dir. Now it independently does a -create-if-not-exist.

          Show
          Thejas M Nair added a comment - HIVE-4513 .2.patch - SessionState temp file gets created in same dir as hive history dir, but it was relying on hivehistory to create that dir. Now it independently does a -create-if-not-exist.
          Hide
          Thejas M Nair added a comment -

          HIVE-4513.3.patch - additional comments

          Show
          Thejas M Nair added a comment - HIVE-4513 .3.patch - additional comments
          Hide
          Thejas M Nair added a comment -

          review board link with HIVE-4513.3.patch - https://reviews.apache.org/r/11029/

          Show
          Thejas M Nair added a comment - review board link with HIVE-4513 .3.patch - https://reviews.apache.org/r/11029/
          Hide
          Thejas M Nair added a comment -

          HIVE-4513.4.patch

          • add @Override to interface functions being implemented in HiveHistoryImpl
          • Removing javadoc duplication in HiveHistoryImpl. It will automatically inherit the documentation from interface.
          • Logging the exception in code unrelated to patch, to partly address Brock's concern. Since the code is not part of the patch, I don't want to increase the scope to address that concern.
          Show
          Thejas M Nair added a comment - HIVE-4513 .4.patch add @Override to interface functions being implemented in HiveHistoryImpl Removing javadoc duplication in HiveHistoryImpl. It will automatically inherit the documentation from interface. Logging the exception in code unrelated to patch, to partly address Brock's concern. Since the code is not part of the patch, I don't want to increase the scope to address that concern.
          Hide
          Ashutosh Chauhan added a comment -

          Thejas M Nair Left some comments on RB.

          Show
          Ashutosh Chauhan added a comment - Thejas M Nair Left some comments on RB.
          Hide
          Ashutosh Chauhan added a comment -

          Canceling patch for now.

          Show
          Ashutosh Chauhan added a comment - Canceling patch for now.
          Hide
          Thiruvel Thirumoolan added a comment -

          Hi Thejas, are you working on this patch?

          Show
          Thiruvel Thirumoolan added a comment - Hi Thejas, are you working on this patch?
          Hide
          Thejas M Nair added a comment -

          Thiruvel Thirumoolan Interesting that you asked me today, I am working on it right now!

          Show
          Thejas M Nair added a comment - Thiruvel Thirumoolan Interesting that you asked me today, I am working on it right now!
          Hide
          Thejas M Nair added a comment -

          HIVE-4513.5.patch - patch addressing review comments.
          Also, removed use of hive history from TestCliDriver.vm and TestHBaseCliDriver.vm as it does not get used. When there is a failure that gets logged in hive history, it also results in client exit code being non zero. So the use of hivehistory there is redundant.

          I will upload the updated patch to reviewboard once it is up again.

          Show
          Thejas M Nair added a comment - HIVE-4513 .5.patch - patch addressing review comments. Also, removed use of hive history from TestCliDriver.vm and TestHBaseCliDriver.vm as it does not get used. When there is a failure that gets logged in hive history, it also results in client exit code being non zero. So the use of hivehistory there is redundant. I will upload the updated patch to reviewboard once it is up again.
          Hide
          Edward Capriolo added a comment -

          +1 I never found them of much use.

          Show
          Edward Capriolo added a comment - +1 I never found them of much use.
          Hide
          Hive QA added a comment -

          Overall: -1 at least one tests failed

          Here are the results of testing the latest attachment:
          https://issues.apache.org/jira/secure/attachment/12597018/HIVE-4513.5.patch

          ERROR: -1 due to 4 failed/errored test(s), 2775 tests executed
          Failed tests:

          org.apache.hadoop.hive.service.TestHiveServerSessions.testSessionVars
          org.apache.hcatalog.mapreduce.TestSequenceFileReadWrite.testSequenceTableWriteReadMR
          org.apache.hadoop.hive.cli.TestMinimrCliDriver.testCliDriver_infer_bucket_sort_bucketed_table
          org.apache.hadoop.hive.cli.TestNegativeCliDriver.testNegativeCliDriver_script_broken_pipe1
          

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

          Messages:

          Executing org.apache.hive.ptest.execution.PrepPhase
          Executing org.apache.hive.ptest.execution.ExecutionPhase
          Executing org.apache.hive.ptest.execution.ReportingPhase
          Tests failed with: TestsFailedException: 4 tests failed
          

          This message is automatically generated.

          Show
          Hive QA added a comment - Overall : -1 at least one tests failed Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12597018/HIVE-4513.5.patch ERROR: -1 due to 4 failed/errored test(s), 2775 tests executed Failed tests: org.apache.hadoop.hive.service.TestHiveServerSessions.testSessionVars org.apache.hcatalog.mapreduce.TestSequenceFileReadWrite.testSequenceTableWriteReadMR org.apache.hadoop.hive.cli.TestMinimrCliDriver.testCliDriver_infer_bucket_sort_bucketed_table org.apache.hadoop.hive.cli.TestNegativeCliDriver.testNegativeCliDriver_script_broken_pipe1 Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/359/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/359/console Messages: Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase Tests failed with: TestsFailedException: 4 tests failed This message is automatically generated.
          Hide
          Thejas M Nair added a comment -

          HIVE-4513.6.patch - addresses review comments.

          Also fixes race condition the race condition that was causing the TestHiveServerSessions.testSessionVars test failure. This race condition gets exposed when hivehistory is disabled, because when hive history is enabled it attempts to create the same dir this way, but on failure it just logs a warning.

          Show
          Thejas M Nair added a comment - HIVE-4513 .6.patch - addresses review comments. Also fixes race condition the race condition that was causing the TestHiveServerSessions.testSessionVars test failure. This race condition gets exposed when hivehistory is disabled, because when hive history is enabled it attempts to create the same dir this way, but on failure it just logs a warning.
          Hide
          Thejas M Nair added a comment -

          I found another issue while making these changes, - SessionState temp file gets created in history file directory, created HIVE-5055 to track that.

          Show
          Thejas M Nair added a comment - I found another issue while making these changes, - SessionState temp file gets created in history file directory, created HIVE-5055 to track that.
          Hide
          Hive QA added a comment -

          Overall: -1 at least one tests failed

          Here are the results of testing the latest attachment:
          https://issues.apache.org/jira/secure/attachment/12597289/HIVE-4513.6.patch

          ERROR: -1 due to 1 failed/errored test(s), 2776 tests executed
          Failed tests:

          org.apache.hadoop.hive.cli.TestNegativeMinimrCliDriver.testNegativeCliDriver_mapreduce_stack_trace_hadoop20
          

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

          Messages:

          Executing org.apache.hive.ptest.execution.PrepPhase
          Executing org.apache.hive.ptest.execution.ExecutionPhase
          Executing org.apache.hive.ptest.execution.ReportingPhase
          Tests failed with: TestsFailedException: 1 tests failed
          

          This message is automatically generated.

          Show
          Hive QA added a comment - Overall : -1 at least one tests failed Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12597289/HIVE-4513.6.patch ERROR: -1 due to 1 failed/errored test(s), 2776 tests executed Failed tests: org.apache.hadoop.hive.cli.TestNegativeMinimrCliDriver.testNegativeCliDriver_mapreduce_stack_trace_hadoop20 Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/385/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/385/console Messages: Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase Tests failed with: TestsFailedException: 1 tests failed This message is automatically generated.
          Hide
          Thejas M Nair added a comment -

          Regarding the pre-commit test result, testNegativeCliDriver_mapreduce_stack_trace_hadoop20 is a flaky test tracked in HIVE-4851.
          Reviewboard has the latest patch.

          Show
          Thejas M Nair added a comment - Regarding the pre-commit test result, testNegativeCliDriver_mapreduce_stack_trace_hadoop20 is a flaky test tracked in HIVE-4851 . Reviewboard has the latest patch.
          Hide
          Ashutosh Chauhan added a comment -

          +1

          Show
          Ashutosh Chauhan added a comment - +1
          Hide
          Thiruvel Thirumoolan added a comment -

          Thanks Thejas M Nair. +1. I guess we can close the duplicates HIVE-1708 and HIVE-3779.

          We back-ported this to Hive10 and it works as expected. Chris Drome had some comments on the patch. These fall in the vicinity but should be addressed as a separate JIRA.

          1. HiveHistoryViewer.java: Its good as "private void init()"
          2. HiveHistoryUtil.java: parseLine() method is not thread-safe. It uses parseBuffer which could be modified by multiple threads. Currently only HWA uses it.

          Show
          Thiruvel Thirumoolan added a comment - Thanks Thejas M Nair . +1. I guess we can close the duplicates HIVE-1708 and HIVE-3779 . We back-ported this to Hive10 and it works as expected. Chris Drome had some comments on the patch. These fall in the vicinity but should be addressed as a separate JIRA. 1. HiveHistoryViewer.java: Its good as "private void init()" 2. HiveHistoryUtil.java: parseLine() method is not thread-safe. It uses parseBuffer which could be modified by multiple threads. Currently only HWA uses it.
          Hide
          Thiruvel Thirumoolan added a comment -

          > Chris Drome had some comments on the patch. These fall in the vicinity but should be addressed as a separate JIRA.

          Created HIVE-5071 to address thread safety issues.

          Show
          Thiruvel Thirumoolan added a comment - > Chris Drome had some comments on the patch. These fall in the vicinity but should be addressed as a separate JIRA. Created HIVE-5071 to address thread safety issues.
          Hide
          Thejas M Nair added a comment -

          Thiruvel Thirumoolan Thanks for the feedback and pointers to the duplicate jiras. Yes, I think it is better to address thread safety issue in another jira, as it is not a regression introduced by this patch, the issue was there when the function was in HiveHistory.java (but I should have noticed the bug!)

          Show
          Thejas M Nair added a comment - Thiruvel Thirumoolan Thanks for the feedback and pointers to the duplicate jiras. Yes, I think it is better to address thread safety issue in another jira, as it is not a regression introduced by this patch, the issue was there when the function was in HiveHistory.java (but I should have noticed the bug!)
          Hide
          Hive QA added a comment -

          Overall: +1 all checks pass

          Here are the results of testing the latest attachment:
          https://issues.apache.org/jira/secure/attachment/12597289/HIVE-4513.6.patch

          SUCCESS: +1 2850 tests passed

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

          Messages:

          Executing org.apache.hive.ptest.execution.PrepPhase
          Executing org.apache.hive.ptest.execution.ExecutionPhase
          Executing org.apache.hive.ptest.execution.ReportingPhase
          

          This message is automatically generated.

          Show
          Hive QA added a comment - Overall : +1 all checks pass Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12597289/HIVE-4513.6.patch SUCCESS: +1 2850 tests passed Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/412/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/412/console Messages: Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase This message is automatically generated.
          Hide
          Ashutosh Chauhan added a comment -

          Committed to trunk. Thanks, Thejas!

          Show
          Ashutosh Chauhan added a comment - Committed to trunk. Thanks, Thejas!
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hive-trunk-h0.21 #2265 (See https://builds.apache.org/job/Hive-trunk-h0.21/2265/)
          HIVE-4513 : disable hivehistory logs by default (Thejas Nair via Ashutosh Chauhan) (hashutosh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1513445)

          • /hive/trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
          • /hive/trunk/conf/hive-default.xml.template
          • /hive/trunk/hbase-handler/src/test/templates/TestHBaseCliDriver.vm
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryImpl.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryProxyHandler.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryUtil.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryViewer.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java
          • /hive/trunk/ql/src/test/org/apache/hadoop/hive/ql/history/TestHiveHistory.java
          • /hive/trunk/ql/src/test/templates/TestCliDriver.vm
          Show
          Hudson added a comment - FAILURE: Integrated in Hive-trunk-h0.21 #2265 (See https://builds.apache.org/job/Hive-trunk-h0.21/2265/ ) HIVE-4513 : disable hivehistory logs by default (Thejas Nair via Ashutosh Chauhan) (hashutosh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1513445 ) /hive/trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java /hive/trunk/conf/hive-default.xml.template /hive/trunk/hbase-handler/src/test/templates/TestHBaseCliDriver.vm /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryImpl.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryProxyHandler.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryUtil.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryViewer.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java /hive/trunk/ql/src/test/org/apache/hadoop/hive/ql/history/TestHiveHistory.java /hive/trunk/ql/src/test/templates/TestCliDriver.vm
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hive-trunk-hadoop2 #356 (See https://builds.apache.org/job/Hive-trunk-hadoop2/356/)
          HIVE-4513 : disable hivehistory logs by default (Thejas Nair via Ashutosh Chauhan) (hashutosh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1513445)

          • /hive/trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
          • /hive/trunk/conf/hive-default.xml.template
          • /hive/trunk/hbase-handler/src/test/templates/TestHBaseCliDriver.vm
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryImpl.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryProxyHandler.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryUtil.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryViewer.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java
          • /hive/trunk/ql/src/test/org/apache/hadoop/hive/ql/history/TestHiveHistory.java
          • /hive/trunk/ql/src/test/templates/TestCliDriver.vm
          Show
          Hudson added a comment - FAILURE: Integrated in Hive-trunk-hadoop2 #356 (See https://builds.apache.org/job/Hive-trunk-hadoop2/356/ ) HIVE-4513 : disable hivehistory logs by default (Thejas Nair via Ashutosh Chauhan) (hashutosh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1513445 ) /hive/trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java /hive/trunk/conf/hive-default.xml.template /hive/trunk/hbase-handler/src/test/templates/TestHBaseCliDriver.vm /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryImpl.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryProxyHandler.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryUtil.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryViewer.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java /hive/trunk/ql/src/test/org/apache/hadoop/hive/ql/history/TestHiveHistory.java /hive/trunk/ql/src/test/templates/TestCliDriver.vm
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hive-trunk-hadoop2-ptest #56 (See https://builds.apache.org/job/Hive-trunk-hadoop2-ptest/56/)
          HIVE-4513 : disable hivehistory logs by default (Thejas Nair via Ashutosh Chauhan) (hashutosh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1513445)

          • /hive/trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
          • /hive/trunk/conf/hive-default.xml.template
          • /hive/trunk/hbase-handler/src/test/templates/TestHBaseCliDriver.vm
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryImpl.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryProxyHandler.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryUtil.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryViewer.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java
          • /hive/trunk/ql/src/test/org/apache/hadoop/hive/ql/history/TestHiveHistory.java
          • /hive/trunk/ql/src/test/templates/TestCliDriver.vm
          Show
          Hudson added a comment - FAILURE: Integrated in Hive-trunk-hadoop2-ptest #56 (See https://builds.apache.org/job/Hive-trunk-hadoop2-ptest/56/ ) HIVE-4513 : disable hivehistory logs by default (Thejas Nair via Ashutosh Chauhan) (hashutosh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1513445 ) /hive/trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java /hive/trunk/conf/hive-default.xml.template /hive/trunk/hbase-handler/src/test/templates/TestHBaseCliDriver.vm /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryImpl.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryProxyHandler.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryUtil.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryViewer.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java /hive/trunk/ql/src/test/org/apache/hadoop/hive/ql/history/TestHiveHistory.java /hive/trunk/ql/src/test/templates/TestCliDriver.vm
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hive-trunk-hadoop1-ptest #126 (See https://builds.apache.org/job/Hive-trunk-hadoop1-ptest/126/)
          HIVE-4513 : disable hivehistory logs by default (Thejas Nair via Ashutosh Chauhan) (hashutosh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1513445)

          • /hive/trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
          • /hive/trunk/conf/hive-default.xml.template
          • /hive/trunk/hbase-handler/src/test/templates/TestHBaseCliDriver.vm
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryImpl.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryProxyHandler.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryUtil.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryViewer.java
          • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java
          • /hive/trunk/ql/src/test/org/apache/hadoop/hive/ql/history/TestHiveHistory.java
          • /hive/trunk/ql/src/test/templates/TestCliDriver.vm
          Show
          Hudson added a comment - FAILURE: Integrated in Hive-trunk-hadoop1-ptest #126 (See https://builds.apache.org/job/Hive-trunk-hadoop1-ptest/126/ ) HIVE-4513 : disable hivehistory logs by default (Thejas Nair via Ashutosh Chauhan) (hashutosh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1513445 ) /hive/trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java /hive/trunk/conf/hive-default.xml.template /hive/trunk/hbase-handler/src/test/templates/TestHBaseCliDriver.vm /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryImpl.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryProxyHandler.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryUtil.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryViewer.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java /hive/trunk/ql/src/test/org/apache/hadoop/hive/ql/history/TestHiveHistory.java /hive/trunk/ql/src/test/templates/TestCliDriver.vm
          Hide
          Ashutosh Chauhan added a comment -

          This issue has been fixed and released as part of 0.12 release. If you find further issues, please create a new jira and link it to this one.

          Show
          Ashutosh Chauhan added a comment - This issue has been fixed and released as part of 0.12 release. If you find further issues, please create a new jira and link it to this one.
          Hide
          Lefty Leverenz added a comment -

          This adds the configuration parameter hive.session.history.enabled in HiveConf.java and hive-default.xml.template, so it needs to be documented in the wiki here:

          Show
          Lefty Leverenz added a comment - This adds the configuration parameter hive.session.history.enabled in HiveConf.java and hive-default.xml.template, so it needs to be documented in the wiki here: Hive Configuration Properties

            People

            • Assignee:
              Thejas M Nair
              Reporter:
              Thejas M Nair
            • Votes:
              1 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development