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

(Updated Oct. 1, 2014, 7:27 a.m.)


Review request for hive, Carl Steinbach, Alan Gates, Navis Ryu, Prasad 
Mujumdar, and Sergey Shelukhin.


Changes
-------

Patch 2 adds descriptions for 7 parameters, removes angle brackets from 
descriptions of 2 parameters, improves descriptions of 24 parameters, and makes 
trivial edits and format changes to many more.
The non-trivial changes are listed in a comment on HIVE-6586, along with JIRAs 
and people who created the parameters.  A generated hive-default.xml.template 
file is attached to HIVE-6586 in case you don't want to bother generating your 
own copy.

Looks like I uploaded wrong, using Add File ... so redoing it with Update Diff.


Bugs: HIVE-6586
    https://issues.apache.org/jira/browse/HIVE-6586


Repository: hive


Description
-------

HIVE-6586 kept track of new configuration parameters and changes to parameter 
descriptions when HIVE-6037 moved parameter descriptions into HiveConf.java 
from hive-default.xml.template.

HIVE-6586.patch addresses all the fixes listed in the JIRA comments (except 
ones that had already been fixed), tidies up some line breaks, and makes minor 
edits to parameter descriptions.  It also revises the descriptions of 
hive.txn.xxx, hive.compactor.xxx, hive.server2.async.exec.shutdown.timeout, and 
hive.security.authorization.createtable.owner.grants.


Diffs (updated)
-----

  trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 1628586 

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


Testing
-------

Generated hive-default.xml.template (attached to HIVE-6586) from the new 
HiveConf.java and reviewed the changed parameter descriptions.


File Attachments (updated)
----------------

Patch 2, rebased and fixed some issues
  
https://reviews.apache.org/media/uploaded/files/2014/10/01/8e4b539e-2590-4d8e-b3b5-45175a051f9d__HIVE-6586.2.patch


Thanks,

Lefty Leverenz

Reply via email to