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

2018-11-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-griffin/pull/444 ---

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

2018-11-09 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r232269364 --- Diff: service/src/main/java/org/apache/griffin/core/event/GriffinEventListeners.java --- @@ -0,0 +1,50 @@ +package org.apache.griffin.c

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

2018-11-08 Thread 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() {

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

2018-11-08 Thread gavlyukovskiy
Github user gavlyukovskiy commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r231981734 --- Diff: service/src/main/java/org/apache/griffin/core/event/GriffinEventListeners.java --- @@ -0,0 +1,50 @@ +package org.apache.griff

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

2018-11-06 Thread guoyuepeng
Github user guoyuepeng commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r231369447 --- Diff: service/src/test/java/org/apache/griffin/core/job/EventServiceTest.java --- @@ -0,0 +1,69 @@ +package org.apache.griffin.core.job

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

2018-11-06 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r231352539 --- Diff: service/src/main/java/org/apache/griffin/core/event/GriffinEventListeners.java --- @@ -0,0 +1,51 @@ +package org.apache.griffin.c

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

2018-11-06 Thread guoyuepeng
Github user guoyuepeng commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r231173157 --- Diff: service/src/main/java/org/apache/griffin/core/event/GriffinEventListeners.java --- @@ -0,0 +1,51 @@ +package org.apache.griffin.

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

2018-11-05 Thread gavlyukovskiy
Github user gavlyukovskiy commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r231008571 --- Diff: service/src/main/java/org/apache/griffin/core/event/GriffinEventListeners.java --- @@ -0,0 +1,51 @@ +package org.apache.griff

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

2018-11-04 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230622329 --- Diff: service/src/main/resources/application.properties --- @@ -62,3 +62,5 @@ elasticsearch.scheme=http livy.uri=http://localhost:8998/b

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

2018-11-04 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230622132 --- Diff: service/src/main/java/org/apache/griffin/core/job/JobServiceImpl.java --- @@ -158,17 +162,22 @@ public JobServiceImpl() {

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

2018-11-04 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230621999 --- Diff: service/src/main/java/org/apache/griffin/core/job/JobServiceImpl.java --- @@ -158,17 +162,22 @@ public JobServiceImpl() {

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

2018-11-04 Thread 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

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

2018-11-04 Thread chemikadze
Github user chemikadze commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230578522 --- Diff: service/src/main/java/org/apache/griffin/core/job/JobServiceImpl.java --- @@ -158,17 +162,22 @@ public JobServiceImpl() {

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

2018-11-04 Thread chemikadze
Github user chemikadze commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230578497 --- Diff: service/src/test/java/org/apache/griffin/core/job/EventServiceTest.java --- @@ -0,0 +1,69 @@ +package org.apache.griffin.core.job

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

2018-11-04 Thread chemikadze
Github user chemikadze commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230578447 --- Diff: service/src/main/resources/application.properties --- @@ -62,3 +62,5 @@ elasticsearch.scheme=http livy.uri=http://localhost:8998/

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

2018-11-01 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230049250 --- Diff: service/src/main/java/org/apache/griffin/core/job/JobServiceImpl.java --- @@ -168,7 +175,14 @@ public AbstractJob addJob(AbstractJob j

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

2018-11-01 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230047870 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEvent.java --- @@ -0,0 +1,44 @@ +/* +Licensed to the Apache S

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

2018-11-01 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230047399 --- Diff: service/src/test/java/org/apache/griffin/core/job/EventServiceTest.java --- @@ -0,0 +1,69 @@ +package org.apache.griffin.core.job;

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

2018-11-01 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230047076 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEventListeners.java --- @@ -0,0 +1,61 @@ +package org.apache.gri

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

2018-11-01 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230046860 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEventListeners.java --- @@ -0,0 +1,61 @@ +package org.apache.gri

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

2018-11-01 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230046728 --- Diff: service/src/main/java/org/apache/griffin/core/integration/JobEventHook.java --- @@ -0,0 +1,11 @@ +package org.apache.griffin.core.

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

2018-11-01 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r230045266 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEventListeners.java --- @@ -0,0 +1,61 @@ +package org.apache.gri

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

2018-10-31 Thread guoyuepeng
Github user guoyuepeng commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r229714976 --- Diff: service/src/main/java/org/apache/griffin/core/integration/JobEvent.java --- @@ -0,0 +1,21 @@ +package org.apache.griffin.core.int

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

2018-10-31 Thread guoyuepeng
Github user guoyuepeng commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r229715591 --- Diff: service/src/main/java/org/apache/griffin/core/integration/JobEventHook.java --- @@ -0,0 +1,11 @@ +package org.apache.griffin.core

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

2018-10-31 Thread guoyuepeng
Github user guoyuepeng commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r229720415 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEventListeners.java --- @@ -0,0 +1,61 @@ +package org.apache.gr

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

2018-10-31 Thread guoyuepeng
Github user guoyuepeng commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r229713768 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEventListeners.java --- @@ -0,0 +1,61 @@ +package org.apache.gr

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

2018-10-31 Thread guoyuepeng
Github user guoyuepeng commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r229713456 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEventListeners.java --- @@ -0,0 +1,61 @@ +package org.apache.gr

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

2018-10-31 Thread guoyuepeng
Github user guoyuepeng commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r229708121 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEvent.java --- @@ -0,0 +1,44 @@ +/* +Licensed to the Apache

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

2018-10-30 Thread chemikadze
Github user chemikadze commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r229565301 --- Diff: service/src/test/java/org/apache/griffin/core/job/EventServiceTest.java --- @@ -0,0 +1,69 @@ +package org.apache.griffin.core.job

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

2018-10-30 Thread chemikadze
Github user chemikadze commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r229564055 --- Diff: service/src/main/java/org/apache/griffin/core/integration/JobEvent.java --- @@ -0,0 +1,21 @@ +package org.apache.griffin.core.int

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

2018-10-30 Thread chemikadze
Github user chemikadze commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r229565395 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEvent.java --- @@ -0,0 +1,44 @@ +/* +Licensed to the Apache

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

2018-10-30 Thread chemikadze
Github user chemikadze commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r229565134 --- Diff: service/src/main/java/org/apache/griffin/core/job/JobServiceImpl.java --- @@ -168,7 +175,14 @@ public AbstractJob addJob(AbstractJob

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

2018-10-24 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r228040937 --- Diff: service/src/main/java/org/apache/griffin/core/job/JobServiceImpl.java --- @@ -128,10 +129,22 @@ Licensed to the Apache Software Founda

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

2018-10-24 Thread toyboxman
Github user toyboxman commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r228040806 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEvent.java --- @@ -0,0 +1,44 @@ +/* +Licensed to the Apache S

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

2018-10-24 Thread chemikadze
Github user chemikadze commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r227897445 --- Diff: service/src/main/java/org/apache/griffin/core/integration/GriffinEvent.java --- @@ -0,0 +1,44 @@ +/* +Licensed to the Apache

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

2018-10-24 Thread chemikadze
Github user chemikadze commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/444#discussion_r227900111 --- Diff: service/src/main/java/org/apache/griffin/core/job/JobServiceImpl.java --- @@ -128,10 +129,22 @@ Licensed to the Apache Software Found

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

2018-10-24 Thread toyboxman
GitHub user toyboxman opened a pull request: https://github.com/apache/incubator-griffin/pull/444 Define griffin plain-vanilla hook. the purpose of hook is for integration with components outside. Griffin would offer information about internal task status. Task: GRIFFIN-200