hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kevin Wilfong" <kevinwilf...@fb.com>
Subject Review Request: Add method to PerfLogger to perform cleanup/final steps.
Date Thu, 22 Sep 2011 19:35:00 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2023/
-----------------------------------------------------------

Review request for hive, Yongqiang He and Ning Zhang.


Summary
-------

I added a method called finishPerfLogger, which is called at the end of the run method in
Driver to allow PerfLogger subclasses to perform any cleanup tasks or logging that requires
all the perf stats to have been calculated.  It takes as parameters a Log, to be consistent
with the other methods in PerfLogger, as well as the QueryPlan from Driver in order to allow
information such as the query ID to be logged.


This addresses bug HIVE-2461.
    https://issues.apache.org/jira/browse/HIVE-2461


Diffs
-----

  trunk/ql/src/java/org/apache/hadoop/hive/ql/Driver.java 1173881 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/log/PerfLogger.java 1173881 

Diff: https://reviews.apache.org/r/2023/diff


Testing
-------

I created a subclass of PerfLogger which implemented the finishPerfLogger method, and verified
it was successfully called and executed.

I also ran some of the unit tests to verify nothing was broken.


Thanks,

Kevin


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message