[ 
https://issues.apache.org/jira/browse/HIVE-26148?focusedWorklogId=764059&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-764059
 ]

ASF GitHub Bot logged work on HIVE-26148:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 29/Apr/22 08:02
            Start Date: 29/Apr/22 08:02
    Worklog Time Spent: 10m 
      Work Description: pvary commented on code in PR #3218:
URL: https://github.com/apache/hive/pull/3218#discussion_r861554859


##########
ql/src/test/results/clientnegative/authorization_grant_table_allpriv.q.out:
##########
@@ -15,4 +15,4 @@ POSTHOOK: Output: default@table_priv_allf
 PREHOOK: query: GRANT ALL ON table_priv_allf TO USER user3
 PREHOOK: type: GRANT_PRIVILEGE
 PREHOOK: Output: default@table_priv_allf
-FAILED: Execution Error, return code 40000 from 
org.apache.hadoop.hive.ql.ddl.DDLTask. Permission denied: Principal 
[name=user2, type=USER] does not have following privileges for operation 
GRANT_PRIVILEGE [[SELECT with grant, UPDATE with grant, DELETE with grant] on 
Object [type=TABLE_OR_VIEW, name=default.table_priv_allf]]
+FAILED: Execution Error, return code 40000 from 
org.apache.hadoop.hive.ql.ddl.DDLTask. Permission denied: Principal 
[name=user2, type=USER] does not have following privileges for operation 
GRANT_PRIVILEGE [[SELECT with grant, UPDATE with grant, DELETE with grant] on 
Object [type=TABLE_OR_VIEW, name=null.default.table_priv_allf]]

Review Comment:
   this `null` seems problematic





Issue Time Tracking
-------------------

    Worklog Id:     (was: 764059)
    Time Spent: 1h 20m  (was: 1h 10m)

> Keep MetaStoreFilterHook interface compatibility after introducing catalogs
> ---------------------------------------------------------------------------
>
>                 Key: HIVE-26148
>                 URL: https://issues.apache.org/jira/browse/HIVE-26148
>             Project: Hive
>          Issue Type: Improvement
>          Components: Hive
>    Affects Versions: 3.0.0
>            Reporter: Wechar
>            Assignee: Wechar
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 4.0.0-alpha-1
>
>          Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> Hive 3.0 introduce catalog concept, when we upgrade hive dependency version 
> from 2.3 to 3.x, we found some interfaces of *MetaStoreFilterHook* are not 
> compatible:
> {code:bash}
>  git show ba8a99e115 -- 
> standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/MetaStoreFilterHook.java
> {code}
> {code:bash}
> --- 
> a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/MetaStoreFilterHook.java
> +++ 
> b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/MetaStoreFilterHook.java
>    /**
>     * Filter given list of tables
> -   * @param dbName
> -   * @param tableList
> +   * @param catName catalog name
> +   * @param dbName database name
> +   * @param tableList list of table returned by the metastore
>     * @return List of filtered table names
>     */
> -  public List<String> filterTableNames(String dbName, List<String> 
> tableList) throws MetaException;
> +  List<String> filterTableNames(String catName, String dbName, List<String> 
> tableList)
> +      throws MetaException;
> {code}
> We can remain the previous interfaces and use the default catalog to 
> implement.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to