----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25682/#review55256 -----------------------------------------------------------
trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java <https://reviews.apache.org/r/25682/#comment95645> Rather than saying "to turn on Hive transactions" this should read "as part of turning on Hive transactions". This change alone won't turn on transactions. trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java <https://reviews.apache.org/r/25682/#comment95647> Same comment as above about turning on transactions. - Alan Gates On Oct. 1, 2014, 7:27 a.m., Lefty Leverenz wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > 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 > ---------------- > > 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 > >
