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

Namit Jain commented on HIVE-700:
---------------------------------

I dont think we should allow DROP FUNCTION - we should only support
DROP TEMPORARY FUNCTION - this cannot drop the functions that were not added 
temporarily.
If not already there, FunctionRegistry needs to be enhanced to store the fact 
whether a function is temporary or not.

Also, 

1. Can you add a negative test which tries to drop a permanent function and 
fails.
2. Can you change the existing tests with a 'create temporary function' to have 
a 'drop temporary function' at the end ?


> Fix test error by adding "DROP FUNCTION"
> ----------------------------------------
>
>                 Key: HIVE-700
>                 URL: https://issues.apache.org/jira/browse/HIVE-700
>             Project: Hadoop Hive
>          Issue Type: Bug
>            Reporter: Zheng Shao
>            Assignee: Min Zhou
>         Attachments: HIVE-700.1.patch
>
>
> Since we added "Show Functions" in HIVE-580, test results will depend on what 
> temporary functions are added to the system.
> We should add the capability of "DROP FUNCTION", and do that at the end of 
> those "create function" tests to make sure the "show functions" results are 
> deterministic.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to