hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From " (JIRA)" <>
Subject [jira] [Commented] (HIVE-2364) Make performance logging configurable.
Date Wed, 10 Aug 2011 00:15:29 GMT

] commented on HIVE-2364:

This is an automatically generated e-mail. To reply, visit:

Review request for hive and Ning Zhang.


I created a new class PerfLogger, which wraps the old functionality of Utilities' PerfLogBegin
and PerfLogBegin methods.  I also added a config variable hive.exec.perf.logger.  The value
of this variable can be changed to point to a subclass of PerfLogger which can customize the
code in PerfLogBegin and PerfLogEnd.

This addresses bug HIVE-2364.


  trunk/common/src/java/org/apache/hadoop/hive/conf/ 1155439 
  trunk/conf/hive-default.xml 1155439 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/ 1155439 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/ 1155439 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/log/ PRE-CREATION 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/optimizer/ppr/ 1155439 



I ran the code as it is and made sure it continued to log the performance messages as before.

I also created a subclass of PerfLogger and used its class name as the value of hive.exec.perf.logger,
and verified the subclass's code was run.



> Make performance logging configurable.
> --------------------------------------
>                 Key: HIVE-2364
>                 URL:
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Kevin Wilfong
>            Assignee: Kevin Wilfong
>         Attachments: HIVE-2364.1.patch.txt
> Currently, the way we measure the length of time spent by a piece of code is using the
methods PerfLogBegin and PerfLogEnd in the Utilities class.  If we made the class responsible
for logging this data configurable, it would allow for more flexibility, in particular, we
would not be restricted to logging to an Apache Commons Log.

This message is automatically generated by JIRA.
For more information on JIRA, see:


View raw message