[
https://issues.apache.org/jira/browse/HIVE-1785?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
John Sichi updated HIVE-1785:
-----------------------------
Status: Open (was: Patch Available)
Discussed with Liyin, and there are a few more things we should do here:
* add Javadoc for Hook, HookContext, and ExecuteWithHookContext
* add deprecation annotation for old PreExecute/PostExecute
* rewrite existing hook implementations in Hive codebase to implement new
interface
* run Facebook-specific tests to validate backwards-compatibility during the
transition
> 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_4.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.
