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

Liyin Tang commented on HIVE-1785:
----------------------------------

How about adding one more layer over Pre/PostExecute interface, call it Hook.

So both ExecuteWithHookContext and Pre/PostExecute implements this Hook 
interface

During the run time, using reflection to see whether the hook is 
ExecuteWithHookContext  or Pre/PostExecute.




> change Pre/Post Query Hooks to take in 1 parameter: HookContext
> ---------------------------------------------------------------
>
>                 Key: HIVE-1785
>                 URL: https://issues.apache.org/jira/browse/HIVE-1785
>             Project: Hive
>          Issue Type: Improvement
>          Components: Query Processor
>            Reporter: Namit Jain
>            Assignee: Liyin Tang
>         Attachments: hive-1785_3.patch, hive_1785_1.patch, hive_1785_2.patch
>
>
> This way, it would be possible to add new parameters to the hooks without 
> changing the existing hooks.
> This will be a incompatible change, and all the hooks need to change to the 
> new API

-- 
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