[GitHub] incubator-griffin pull request #444: Define griffin plain-vanilla hook.

[chemikadze]

Github user chemikadze commented on a diff in the pull request:

    https://github.com/apache/incubator-griffin/pull/444#discussion_r230578657
  
    --- Diff: 
service/src/main/java/org/apache/griffin/core/integration/JobEventHook.java ---
    @@ -0,0 +1,11 @@
    +package org.apache.griffin.core.integration;
    +
    +import org.apache.griffin.core.exception.GriffinException;
    +
    +public class JobEventHook implements GriffinHook {
    +    @Override
    +    public void onEvent(GriffinEvent event) throws GriffinException {
    +        throw new GriffinException.UnImplementedException(
    +            "This method needs to be reimplemented by event-consuming 
purpose");
    --- End diff --
    
    By the way, if this was for testing purposes, we could have test hook 
defined in test code itself (like in case of hive metadata service tests).


---