hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Phabricator (JIRA)" <>
Subject [jira] [Commented] (HIVE-2779) Improve hooks run in Driver
Date Fri, 09 Nov 2012 23:37:13 GMT


Phabricator commented on HIVE-2779:

kevinwilfong has closed the revision "HIVE-2779 [jira] Improve hooks run in Driver".


To: JIRA, njain, kevinwilfong
Cc: njain

> Improve hooks run in Driver
> ---------------------------
>                 Key: HIVE-2779
>                 URL:
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Kevin Wilfong
>            Assignee: Kevin Wilfong
>             Fix For: 0.9.0
>         Attachments: ASF.LICENSE.NOT.GRANTED--HIVE-2779.D1599.1.patch, ASF.LICENSE.NOT.GRANTED--HIVE-2779.D1599.2.patch,
> There are some small improvements that can be made to the hooks which are run in the
> 1) The code to get hooks has been clearly just been copy+pasted for each of Pre/Post/OnFailure/SemanticAnalyzer
hooks.  This code should be consolidated into a single method.
> 2) There is a lot more information available to SemanticAnalyzer hooks which ran after
semantic analysis than to those that run before, such as inputs and outputs.  We should make
some of this information available to those hooks, preferably through HiveSemanticAnalyzerHookContext,
so that existing hooks aren't broken.
> 3) Currently, possibly unintentionally, hooks are initialized and run in the order they
appear in the comma separated list that is the value of the configuration variable.  This
is a useful property, we should add comments indicating this is desired and add a unit test
to enforce it.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message