hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Liyin Tang (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HIVE-1785) change Pre/Post Query Hooks to take in 1 parameter: HookContext
Date Wed, 17 Nov 2010 19:32:15 GMT

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


Mime
View raw message