You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by denys kuzmenko via Review Board <no...@reviews.apache.org> on 2018/10/01 11:42:42 UTC
Re: Review Request 68683: Add new configuration to set the size of the
global compile lock
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68683/
-----------------------------------------------------------
(Updated Oct. 1, 2018, 11:42 a.m.)
Review request for hive, Zoltan Haindrich, Zoltan Haindrich, Naveen Gangam, and Peter Vary.
Bugs: HIVE-20535
https://issues.apache.org/jira/browse/HIVE-20535
Repository: hive-git
Description
-------
When removing the compile lock, it is quite risky to remove it entirely.
It would be good to provide a pool size for the concurrent compilation, so the administrator can limit the load
Diffs (updated)
-----
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java d1e6631975
ql/src/java/org/apache/hadoop/hive/ql/Driver.java dad2035362
ql/src/java/org/apache/hadoop/hive/ql/lock/CompileLock.java PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/lock/CompileLockFactory.java PRE-CREATION
ql/src/test/org/apache/hadoop/hive/ql/CompileLockTest.java PRE-CREATION
Diff: https://reviews.apache.org/r/68683/diff/8/
Changes: https://reviews.apache.org/r/68683/diff/7-8/
Testing
-------
Added CompileLockTest
File Attachments
----------------
HIVE-20535.1.patch
https://reviews.apache.org/media/uploaded/files/2018/09/13/41f5a84a-70e5-4882-99c1-1cf98c4364e4__HIVE-20535.1.patch
HIVE-20535.14.patch
https://reviews.apache.org/media/uploaded/files/2018/09/25/335b0f4b-ea94-41d4-881a-ec8bb870a376__HIVE-20535.14.patch
HIVE-20535.14.patch
https://reviews.apache.org/media/uploaded/files/2018/09/25/a92b6da2-eeba-46ee-9409-162653826172__HIVE-20535.14.patch
HIVE-20535.14.patch
https://reviews.apache.org/media/uploaded/files/2018/09/25/9db4cf76-9188-48fb-bd3d-5b28e43a791b__HIVE-20535.14.patch
Thanks,
denys kuzmenko