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

(Updated Aug. 4, 2014, 2:12 a.m.)


Review request for hive.


Changes
-------

Addressed comments


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


Repository: hive-git


Description
-------

Currently with external metastore, direct sql can only be disabled via 
metastore config globally. Perhaps it makes sense to have the ability to 
propagate the setting per query from client to override the metastore setting, 
e.g. if one particular query causes it to fail.


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 15bc0a3 
  common/src/java/org/apache/hadoop/hive/conf/SystemVariables.java ee98d17 
  
itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java
 9e416b5 
  metastore/if/hive_metastore.thrift 55f41db 
  metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java 
b74868b 
  metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java 
4c9a597 
  metastore/src/java/org/apache/hadoop/hive/metastore/IHMSHandler.java 1675751 
  metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java 
d6e849f 
  
metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreEventListener.java 
c28c46a 
  metastore/src/java/org/apache/hadoop/hive/metastore/RetryingHMSHandler.java 
86172b9 
  
metastore/src/java/org/apache/hadoop/hive/metastore/events/ConfigChangeEvent.java
 PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java a7e50ad 
  ql/src/java/org/apache/hadoop/hive/ql/processors/SetProcessor.java 2baa24a 
  service/src/java/org/apache/hive/service/cli/session/HiveSessionBase.java 
4c3164e 
  service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java 
b39d64d 
  service/src/java/org/apache/hive/service/cli/session/SessionManager.java 
c2f0495 

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


Testing
-------


Thanks,

Navis Ryu

Reply via email to