Github user toyboxman commented on the issue:

    https://github.com/apache/incubator-griffin/pull/444
  
    summary of comments
    1.considering general usage, I do not use 'GriffinJobEventManager' but 
actually name it 'GriffinEventManager'
    2.I keep empty implementation for JobEventHook.onEvent(). because I think 
it should be done in another change
    3.Measure event could follow job event to implement
    
    @guoyuepeng @chemikadze 


---

Reply via email to