Hive
  1. Hive
  2. HIVE-3505

log4j template has logging threshold that hides all audit logs

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.10.0
    • Component/s: Logging
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      With the "template" for log4j configuration provided in the tarball, audit logging is hidden (it's logged as "INFO"). By making the log threshold a parameter, this information remains hidden when using the CLI (which is desired) but can be overridden when starting services to enable audit-logging.

      (This is primarily so that Hive is more functional out-of-the-box as installed by Apache Bigtop).

      1. HIVE-3505.patch.1
        0.8 kB
        Sean Mackrory
      2. HIVE-3505.patch.2
        0.7 kB
        Sean Mackrory
      3. HIVE-3505.patch.3
        1 kB
        Sean Mackrory
      4. HIVE-3505.patch.4
        1 kB
        Sean Mackrory

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          5m 54s 1 Sean Mackrory 24/Sep/12 23:17
          Patch Available Patch Available Open Open
          6m 46s 1 Carl Steinbach 24/Sep/12 23:24
          Open Open Resolved Resolved
          1d 6h 42m 1 Carl Steinbach 26/Sep/12 06:06
          Resolved Resolved Closed Closed
          106d 14h 47m 1 Ashutosh Chauhan 10/Jan/13 19:53
          Lefty Leverenz made changes -
          Link This issue is related to HIVE-1948 [ HIVE-1948 ]
          Ashutosh Chauhan made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Hide
          Ashutosh Chauhan added a comment -

          This issue is fixed and released as part of 0.10.0 release. If you find an issue which seems to be related to this one, please create a new jira and link this one with new jira.

          Show
          Ashutosh Chauhan added a comment - This issue is fixed and released as part of 0.10.0 release. If you find an issue which seems to be related to this one, please create a new jira and link this one with new jira.
          Hide
          Hudson added a comment -

          Integrated in Hive-trunk-hadoop2 #54 (See https://builds.apache.org/job/Hive-trunk-hadoop2/54/)
          HIVE-3505. log4j template has logging threshold that hides all audit logs (Sean Mackrory via cws) (Revision 1390278)

          Result = ABORTED
          cws : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390278
          Files :

          • /hive/trunk/common/src/java/conf/hive-log4j.properties
          • /hive/trunk/ql/src/java/conf/hive-exec-log4j.properties
          Show
          Hudson added a comment - Integrated in Hive-trunk-hadoop2 #54 (See https://builds.apache.org/job/Hive-trunk-hadoop2/54/ ) HIVE-3505 . log4j template has logging threshold that hides all audit logs (Sean Mackrory via cws) (Revision 1390278) Result = ABORTED cws : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390278 Files : /hive/trunk/common/src/java/conf/hive-log4j.properties /hive/trunk/ql/src/java/conf/hive-exec-log4j.properties
          Carl Steinbach made changes -
          Link This issue breaks HIVE-3523 [ HIVE-3523 ]
          Hide
          Hudson added a comment -

          Integrated in Hive-trunk-h0.21 #1698 (See https://builds.apache.org/job/Hive-trunk-h0.21/1698/)
          HIVE-3505. log4j template has logging threshold that hides all audit logs (Sean Mackrory via cws) (Revision 1390278)

          Result = FAILURE
          cws : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390278
          Files :

          • /hive/trunk/common/src/java/conf/hive-log4j.properties
          • /hive/trunk/ql/src/java/conf/hive-exec-log4j.properties
          Show
          Hudson added a comment - Integrated in Hive-trunk-h0.21 #1698 (See https://builds.apache.org/job/Hive-trunk-h0.21/1698/ ) HIVE-3505 . log4j template has logging threshold that hides all audit logs (Sean Mackrory via cws) (Revision 1390278) Result = FAILURE cws : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390278 Files : /hive/trunk/common/src/java/conf/hive-log4j.properties /hive/trunk/ql/src/java/conf/hive-exec-log4j.properties
          Carl Steinbach made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Hadoop Flags Reviewed [ 10343 ]
          Fix Version/s 0.10.0 [ 12320745 ]
          Resolution Fixed [ 1 ]
          Hide
          Carl Steinbach added a comment -

          Committed to trunk. Thanks Sean!

          Show
          Carl Steinbach added a comment - Committed to trunk. Thanks Sean!
          Hide
          Carl Steinbach added a comment -

          +1. Will commit if tests pass.

          Show
          Carl Steinbach added a comment - +1. Will commit if tests pass.
          Sean Mackrory made changes -
          Attachment HIVE-3505.patch.4 [ 12546408 ]
          Sean Mackrory made changes -
          Attachment HIVE-3505.patch.3 [ 12546402 ]
          Hide
          Sean Mackrory added a comment -

          New patch applies the same change to hive-exec-log4j.properties

          Show
          Sean Mackrory added a comment - New patch applies the same change to hive-exec-log4j.properties
          Hide
          Carl Steinbach added a comment -

          Generally speaking they should be the same. The way you have it setup in hive-log4j.properties looks like the right approach to me.

          Show
          Carl Steinbach added a comment - Generally speaking they should be the same. The way you have it setup in hive-log4j.properties looks like the right approach to me.
          Hide
          Sean Mackrory added a comment -

          log4j.threshold and hive.root.logger specify two different levels. Should they actually be the same, in reality or should only one be paramterized? Which would be most intuitive considering how this file gets used?

          Show
          Sean Mackrory added a comment - log4j.threshold and hive.root.logger specify two different levels. Should they actually be the same, in reality or should only one be paramterized? Which would be most intuitive considering how this file gets used?
          Hide
          Carl Steinbach added a comment -

          @Sean: Actually, would you mind making the same change to ql/src/java/conf/hive-exec-log4j.properties? Thanks.

          Show
          Carl Steinbach added a comment - @Sean: Actually, would you mind making the same change to ql/src/java/conf/hive-exec-log4j.properties? Thanks.
          Hide
          Carl Steinbach added a comment -

          +1. Will commit if tests pass.

          Show
          Carl Steinbach added a comment - +1. Will commit if tests pass.
          Sean Mackrory made changes -
          Attachment HIVE-3505.patch.2 [ 12546396 ]
          Hide
          Sean Mackrory added a comment -

          This patch fixes a typo that existed on these lines before this fix. This patch is also in SVN format (as opposed to git).

          Show
          Sean Mackrory added a comment - This patch fixes a typo that existed on these lines before this fix. This patch is also in SVN format (as opposed to git).
          Carl Steinbach made changes -
          Component/s Logging [ 12312594 ]
          Carl Steinbach made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Assignee Sean Mackrory [ mackrorysd ]
          Hide
          Carl Steinbach added a comment -

          @Sean: comments on reviewboard. Thanks.

          Show
          Carl Steinbach added a comment - @Sean: comments on reviewboard. Thanks.
          Sean Mackrory made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Sean Mackrory added a comment -
          Show
          Sean Mackrory added a comment - Code review here: https://reviews.apache.org/r/7238/
          Sean Mackrory made changes -
          Field Original Value New Value
          Attachment HIVE-3505.patch.1 [ 12546393 ]
          Sean Mackrory created issue -

            People

            • Assignee:
              Sean Mackrory
              Reporter:
              Sean Mackrory
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development