[ 
https://issues.apache.org/jira/browse/SENTRY-1201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15260400#comment-15260400
 ] 

Lenni Kuff commented on SENTRY-1201:
------------------------------------

One comment on the patch:

* Instead of "tmp", could you use a better variable name - perhaps "fqTblName"? 
* Add a preconditions check to ensure the array size after splitting == 2

> Sentry ignores database prefix for MSCK statement
> -------------------------------------------------
>
>                 Key: SENTRY-1201
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1201
>             Project: Sentry
>          Issue Type: Bug
>          Components: Hive Binding
>            Reporter: Sravya Tirukkovalur
>            Assignee: Sravya Tirukkovalur
>         Attachments: SENTRY-1201.1.patch, SENTRY-1201.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to