Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Hive, mail # dev - Review Request 14523: HIVE-5486 HiveServer2 should create base scratch directories at startup


+
Prasad Mujumdar 2013-10-08, 01:29
+
Prasad Mujumdar 2013-10-08, 01:34
+
Brock Noland 2013-10-08, 21:30
+
Prasad Mujumdar 2013-10-09, 05:12
Copy link to this message
-
Re: Review Request 14523: HIVE-5486 HiveServer2 should create base scratch directories at startup
Prasad Mujumdar 2013-10-09, 06:34

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

(Updated Oct. 9, 2013, 6:34 a.m.)
Review request for hive.
Changes
-------

Changes per review feedback
Bugs: HIVE-5486
    https://issues.apache.org/jira/browse/HIVE-5486
Repository: hive-git
Description
-------

With impersonation enabled, the same base directory is used by all sessions/queries. For a new deployment, this directory gets created on first invocation by the user running that session. This would cause directory permission conflict for other users.
The patch is creating base scratch dirs at startup if it doesn't exist.
Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java d0895e1
  service/src/java/org/apache/hive/service/cli/CLIService.java 1a7f338
  service/src/java/org/apache/hive/service/cli/session/HiveSessionImplwithUGI.java ae7bb6b
  service/src/test/org/apache/hive/service/cli/TestEmbeddedThriftBinaryCLIService.java da325da
  service/src/test/org/apache/hive/service/cli/TestScratchDir.java PRE-CREATION

Diff: https://reviews.apache.org/r/14523/diff/
Testing
-------

Added new test
Thanks,

Prasad Mujumdar

+
Brock Noland 2013-10-09, 15:35
+
Thejas Nair 2013-10-17, 02:05
+
Prasad Mujumdar 2013-10-17, 18:45
+
Prasad Mujumdar 2013-10-17, 18:49
+
Prasad Mujumdar 2013-10-19, 18:50