Mailing-List: contact dev-help@hive.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hive.apache.org
Content-Type: multipart/alternative;
 boundary="===============7937062418478175877=="
MIME-Version: 1.0
Subject: Review Request 38894: HIVE-11995: Remove repetitively setting
 permissions in insert/load overwrite partition
From: "Chaoyu Tang" <ctang.ma@gmail.com>
To: "Szehon Ho" <szehon@cloudera.com>,
 "Sergio Pena" <sergio.pena@cloudera.com>
Cc: "hive" <dev@hive.apache.org>, "Chaoyu Tang" <ctang.ma@gmail.com>
Date: Wed, 30 Sep 2015 13:32:21 -0000
Message-ID: <20150930133221.23563.78350@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: "Chaoyu Tang" <noreply@reviews.apache.org>
Reply-To: "Chaoyu Tang" <ctang.ma@gmail.com>

--===============7937062418478175877==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


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

Review request for hive, Sergio Pena and Szehon Ho.


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


Repository: hive-git


Description
-------

Table and partition permissions are repetitively set in Hive.ReplaceFiles by inheritFromTable which sometimes is not necessary and causing performance issue. For example, in dyanamically insert/load overwrite multiple level of partitions (e.g. table/p1/p2/p3/p4), the p1,p2 p3 permissions are set repetitively.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 99896c6 

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


Testing
-------

TestExtendedAcls and TestFolderPermissions have existing test cases which passed in my local machine.
Submitted for precommit build


Thanks,

Chaoyu Tang


--===============7937062418478175877==--