hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Prasad Mujumdar" <pras...@cloudera.com>
Subject Re: Review Request 14523: HIVE-5486 HiveServer2 should create base scratch directories at startup
Date Wed, 09 Oct 2013 06:34:34 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message