Hive
  1. Hive
  2. HIVE-6768

remove hcatalog/webhcat/svr/src/main/config/override-container-log4j.properties

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.13.0
    • Fix Version/s: 0.14.0
    • Component/s: WebHCat
    • Labels:
      None

      Description

      now that MAPREDUCE-5806 is fixed we can remove override-container-log4j.properties and and all the logic around this which was introduced in HIVE-5511 to work around MAPREDUCE-5806

      NO PRECOMMIT TESTS

      1. HIVE-6768.patch
        17 kB
        Eugene Koifman

        Issue Links

          Activity

          Hide
          Thejas M Nair added a comment -

          This has been fixed in 0.14 release. Please open new jira if you see any issues.

          Show
          Thejas M Nair added a comment - This has been fixed in 0.14 release. Please open new jira if you see any issues.
          Hide
          Thejas M Nair added a comment -

          Patch committed to trunk. Thanks for the contribution Eugene!

          Show
          Thejas M Nair added a comment - Patch committed to trunk. Thanks for the contribution Eugene!
          Hide
          Thejas M Nair added a comment -

          +1

          Show
          Thejas M Nair added a comment - +1
          Hide
          Eugene Koifman added a comment -

          Thejas M Nair,[~hashutosh] the attached patch reverts all changes that were part of HIVE-5511 needed to handle the 'special' override-container-log4j, just like the bug description says. HIVE-5511 also included some refactoring, which should not be reverted.

          Show
          Eugene Koifman added a comment - Thejas M Nair , [~hashutosh] the attached patch reverts all changes that were part of HIVE-5511 needed to handle the 'special' override-container-log4j, just like the bug description says. HIVE-5511 also included some refactoring, which should not be reverted.
          Hide
          Thejas M Nair added a comment -

          Eugene Koifman Can you respond on Ashutosh's comment ?

          Looks like most of the changes in HIVE-5511 are not specific to the issue, but were general cleanup. And the attached patch reverts changes that were specific to the log handling.

          Show
          Thejas M Nair added a comment - Eugene Koifman Can you respond on Ashutosh's comment ? Looks like most of the changes in HIVE-5511 are not specific to the issue, but were general cleanup. And the attached patch reverts changes that were specific to the log handling.
          Hide
          Ashutosh Chauhan added a comment -

          In addition to this file, I assume we also need to revert changes introduced in HIVE-5511 Eugene Koifman would you like to attach a patch for this?

          Show
          Ashutosh Chauhan added a comment - In addition to this file, I assume we also need to revert changes introduced in HIVE-5511 Eugene Koifman would you like to attach a patch for this?

            People

            • Assignee:
              Eugene Koifman
              Reporter:
              Eugene Koifman
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development