----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69914/#review212667 -----------------------------------------------------------
Ship it! Ship It! - Peter Vary On febr. 8, 2019, 6:48 de, Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69914/ > ----------------------------------------------------------- > > (Updated febr. 8, 2019, 6:48 de) > > > 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 > > standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/TestFilterHooks.java > 6a0d0aa8840d9fc6799c16463a70ed6f0cc2c354 > > > Diff: https://reviews.apache.org/r/69914/diff/2/ > > > Testing > ------- > > TestGetPartitions and TestListPartitions pass > > > Thanks, > > Na Li > >
