hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Sichi (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HIVE-1785) change Pre/Post Query Hooks to take in 1 parameter: HookContext
Date Thu, 18 Nov 2010 19:00:24 GMT

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


Mime
View raw message