----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69914/#review212633 -----------------------------------------------------------
standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java Line 2509 (original), 2512 (patched) <https://reviews.apache.org/r/69914/#comment298460> We should add unit tests to make sure that this doesn't happen again. Can you add a test which would fail if this fix wasn't there? - Karthik Manamcheri On Feb. 7, 2019, 3:31 a.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69914/ > ----------------------------------------------------------- > > (Updated Feb. 7, 2019, 3:31 a.m.) > > > Review request for hive and Vihang Karajgaonkar. > > > Bugs: hive-21227 > https://issues.apache.org/jira/browse/hive-21227 > > > Repository: hive-git > > > Description > ------- > > HIVE-20776 introduces a change that causes regression for view access. > > Before the change, a user with select access of a view can get all columns of > a view with select access of a view that is derived from a partitioned table. > > With the change, that user cannot access that view. > > The reason is that > > When user accesses columns of a view, Hive needs to get the partitions of the > table that the view is derived from. The user name is the user who issues the > query to access the view. > The change in HIVE-20776 checks if user has access to a table before getting > its partitions. When user only has access of a view, not the access of a > table itself, this change denies the user access of the view. > The solution is when getting table partitions, do not filter on table at HMS > client > > > Diffs > ----- > > > standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java > a1826fa259d424c9f3d5a2f58a18f617355d586f > > > Diff: https://reviews.apache.org/r/69914/diff/1/ > > > Testing > ------- > > TestGetPartitions and TestListPartitions pass > > > Thanks, > > Na Li > >
