Re: Review Request 11029: HIVE-4513 - disable hivehistory logs by default

2013-08-10 Thread Thejas Nair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11029/ --- (Updated Aug. 10, 2013, 4:24 p.m.) Review request for hive. Changes ---

Re: Review Request 11029: HIVE-4513 - disable hivehistory logs by default

2013-08-10 Thread Thejas Nair
On June 3, 2013, 9:03 p.m., Ashutosh Chauhan wrote: ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryImpl.java, line 86 https://reviews.apache.org/r/11029/diff/2/?file=290954#file290954line86 In case of incorrect config, should this throw an exception instead of silent