-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15449/
-----------------------------------------------------------
(Updated Sept. 1, 2014, 5:14 a.m.)
Review request for hive.
Changes
-------
Fixed missing TimeValidators
Bugs: HIVE-5799
https://issues.apache.org/jira/browse/HIVE-5799
Repository: hive-git
Description
-------
Need some timeout facility for preventing resource leakages from instable or
bad clients.
Diffs (updated)
-----
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 74bb863
common/src/java/org/apache/hadoop/hive/conf/Validator.java cea9c41
hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java
bf2b24e
hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
be7134f
itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestMetaStoreAuthorization.java
a6a038a
itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestRetryingHMSHandler.java
39e7005
itests/hive-unit/src/test/java/org/apache/hive/jdbc/miniHS2/TestHiveServer2SessionTimeout.java
PRE-CREATION
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
9ae6d7a
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
a94a7a37
metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java b9cf701
metastore/src/java/org/apache/hadoop/hive/metastore/RetryingHMSHandler.java
84e6dcd
metastore/src/java/org/apache/hadoop/hive/metastore/RetryingMetaStoreClient.java
5410b45
metastore/src/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java
063dee6
metastore/src/test/org/apache/hadoop/hive/metastore/txn/TestTxnHandler.java
8287c60
ql/src/java/org/apache/hadoop/hive/ql/exec/AutoProgressor.java d7323cb
ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java cd017d8
ql/src/java/org/apache/hadoop/hive/ql/exec/Heartbeater.java 7fdb4e7
ql/src/java/org/apache/hadoop/hive/ql/exec/ScriptOperator.java 5b857e2
ql/src/java/org/apache/hadoop/hive/ql/exec/UDTFOperator.java afd7bcf
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 70047a2
ql/src/java/org/apache/hadoop/hive/ql/exec/mr/HadoopJobExecHelper.java
eb2851b
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java ebe9f92
ql/src/java/org/apache/hadoop/hive/ql/lockmgr/EmbeddedLockManager.java
11434a0
ql/src/java/org/apache/hadoop/hive/ql/lockmgr/zookeeper/ZooKeeperHiveLockManager.java
46044d0
ql/src/java/org/apache/hadoop/hive/ql/stats/jdbc/JDBCStatsAggregator.java
f636cff
ql/src/java/org/apache/hadoop/hive/ql/stats/jdbc/JDBCStatsPublisher.java
db62721
ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Initiator.java 3211759
ql/src/test/org/apache/hadoop/hive/ql/txn/compactor/TestInitiator.java
f34b5ad
ql/src/test/results/clientpositive/show_conf.q.out a3c814a
service/src/java/org/apache/hadoop/hive/service/HiveServer.java 32729f2
service/src/java/org/apache/hive/service/cli/CLIService.java ff5de4a
service/src/java/org/apache/hive/service/cli/OperationState.java 3e15f0c
service/src/java/org/apache/hive/service/cli/operation/Operation.java 0d6436e
service/src/java/org/apache/hive/service/cli/operation/OperationManager.java
2867301
service/src/java/org/apache/hive/service/cli/session/HiveSession.java 270e4a6
service/src/java/org/apache/hive/service/cli/session/HiveSessionBase.java
84e1c7e
service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java
4e5f595
service/src/java/org/apache/hive/service/cli/session/HiveSessionImplwithUGI.java
7668904
service/src/java/org/apache/hive/service/cli/session/SessionManager.java
17c1c7b
service/src/java/org/apache/hive/service/cli/thrift/ThriftBinaryCLIService.java
e5ce72f
service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java
86ed4b4
service/src/java/org/apache/hive/service/cli/thrift/ThriftHttpCLIService.java
21d1563
service/src/test/org/apache/hive/service/cli/CLIServiceTest.java d01e819
Diff: https://reviews.apache.org/r/15449/diff/
Testing
-------
Confirmed in the local environment.
Thanks,
Navis Ryu
