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

[guoyuepeng]

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

    https://github.com/apache/incubator-griffin/pull/444#discussion_r231798479
  
    --- Diff: 
service/src/main/java/org/apache/griffin/core/job/JobServiceImpl.java ---
    @@ -158,17 +162,22 @@ public JobServiceImpl() {
             } catch (SchedulerException e) {
                 LOGGER.error("Failed to get RUNNING jobs.", e);
                 throw new GriffinException
    -                    .ServiceException("Failed to get RUNNING jobs.", e);
    +                .ServiceException("Failed to get RUNNING jobs.", e);
             }
             return dataList;
         }
     
         @Override
         public AbstractJob addJob(AbstractJob job) throws Exception {
    +        JobEvent jobEvent = JobEvent.yieldJobEventBeforeCreation(null);
    --- End diff --
    
    @toyboxman Can we verify it by assert verify on called? instead of some 
exception throwed.
    currently test case , not straightforward to me.


---