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

Brock Noland commented on HIVE-5946:
------------------------------------

[~thejas] I created this patch a while back after I noticed that the DDL code 
is monolithic and poorly tested. Since you are now making changes to this code 
path, it might make sense to commit this. Additionally I make the code path 
"pluggable" but I am now rethinking that aspect. If the "pluggable" aspect 
doesn't make sense then I can remove that and rebase the patch.

> DDL authorization task factory should be better tested
> ------------------------------------------------------
>
>                 Key: HIVE-5946
>                 URL: https://issues.apache.org/jira/browse/HIVE-5946
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Brock Noland
>            Assignee: Brock Noland
>         Attachments: HIVE-5946.patch
>
>
> Thejas is working on various authorization issues and one element that might 
> be useful in that effort and increase test coverage and testability would be 
> perform authorization task creation in a factory.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to