hive-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Liyin Tang <liyint...@gmail.com>
Subject change Pre/Post Query Hooks to take only 1 parameter: HookContext
Date Wed, 24 Nov 2010 03:39:53 GMT
Hi All
Before Hive-1785 committed, the old hook interface will take 4 or 5
parameters to run, which is not scalable for adding more complicated hooks.
Hooks may need more information than these 4 or 5 parameters.
Hive should support to add more hooks without changing the existing hook
interface.
So a better solution is to create a HookContext and all the hook should only
run with this HookContext.

In Hive-1785, the patch still supports the old interface but makes them as
deprecated. New implemented hooks should follow the new interface:
ExecuteWithHookContext. Also the patch has changed all the existing Hooks,
as examples, to follow this new interface


Happy Thanksgiving
Best Regards
Liyin

Mime
View raw message