[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

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread ???? Sheng Wu
+1 Be better. -- Sheng Wu Apache SkyWalking -- Original -- From: "ShaoFeng Shi"; Date: Thu, Nov 1, 2018 04:48 PM To: "general"; Cc: "dev"; Subject: Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP +1 (non-binding) Gook

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread Alex Lv
+1 Alex Lv From: 吴晟 Sheng Wu Sent: Thursday, November 1, 2018 16:52 To: general Cc: dev Subject: Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP +1 Be better. -- Sheng Wu Apache SkyWalking -- Original

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread Eugene Liu
It's time to graduate now! totally agree +1 From: jenny li Sent: Thursday, November 1, 2018 3:53 PM To: dev@griffin.incubator.apache.org Cc: gene...@incubator.apache.org Subject: Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP +1 Can‘t wait to see this

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread ShaoFeng Shi
+1 (non-binding) Gook luck Griffin! Eugene Liu 于2018年11月1日周四 下午3:57写道: > It's time to graduate now! > > totally agree +1 > > From: jenny li > Sent: Thursday, November 1, 2018 3:53 PM > To: dev@griffin.incubator.apache.org > Cc: gene...@incubator.apache.org >

Fwd: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread William Guo
+dev -- Forwarded message - From: William Guo Date: Thu, Nov 1, 2018 at 11:21 AM Subject: [VOTE] Graduate Apache Griffin (incubating) as a TLP To: Hi all, Given that we've got positive feedback on the DISCUSS thread, after we had made some changes based on DISCUSS feedback.

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread jenny li
+1 Can‘t wait to see this great project Apache Griffin from the Incubator to Graduation. Best regards, Jenny On Thu, Nov 1, 2018 at 3:31 PM William Guo wrote: > +dev > > -- Forwarded message - > From: William Guo > Date: Thu, Nov 1, 2018 at 11:21 AM > Subject: [VOTE] Graduate

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread 亮 邵
+1 From: William Guo Sent: Thursday, November 1, 2018 7:31 AM To: gene...@incubator.apache.org; dev@griffin.incubator.apache.org Subject: Fwd: [VOTE] Graduate Apache Griffin (incubating) as a TLP +dev -- Forwarded message - From: William Guo

[GitHub] incubator-griffin-site pull request #13: Fix layout issue when on mobile

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

Re: [DISCUSS] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread Kevin Yao
+1 Thanks, Kevin Yao William Guo 于2018年11月1日 周四上午10:24写道: > Will go to vote after some changes upon DISCUSS feedback. > > Thanks, > William > > On Thu, Nov 1, 2018 at 10:04 AM William Guo wrote: > > > Thanks all. > > > > One following update: Apache Griffin Name Search resolved. > > > >

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread Jean-Baptiste Onofré
+1 (binding) Regards JB Le 1 nov. 2018 à 14:03, à 14:03, Willem Jiang a écrit: >+1 binding. >It's good to see griffin is ready to graduate. > >Willem Jiang > >Twitter: willemjiang >Weibo: 姜宁willem > >On Thu, Nov 1, 2018 at 3:31 PM William Guo wrote: >> >> +dev >> >> -- Forwarded

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread Willem Jiang
+1 binding. It's good to see griffin is ready to graduate. Willem Jiang Twitter: willemjiang Weibo: 姜宁willem On Thu, Nov 1, 2018 at 3:31 PM William Guo wrote: > > +dev > > -- Forwarded message - > From: William Guo > Date: Thu, Nov 1, 2018 at 11:21 AM > Subject: [VOTE]

[GitHub] incubator-griffin-site pull request #13: Fix layout issue when on mobile

2018-11-01 Thread icesmartjuan
GitHub user icesmartjuan opened a pull request: https://github.com/apache/incubator-griffin-site/pull/13 Fix layout issue when on mobile You can merge this pull request into a Git repository by running: $ git pull https://github.com/icesmartjuan/incubator-griffin-site

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread Yin Wan
+1 On Thu, Nov 1, 2018 at 3:31 PM William Guo wrote: > +dev > > -- Forwarded message - > From: William Guo > Date: Thu, Nov 1, 2018 at 11:21 AM > Subject: [VOTE] Graduate Apache Griffin (incubating) as a TLP > To: > > > Hi all, > > Given that we've got positive feedback on the

[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

[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

[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

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread john@apache
+1 On Thu, Nov 1, 2018 at 6:03 PM Jean-Baptiste Onofré wrote: > +1 (binding) > > Regards > JB > > Le 1 nov. 2018 à 14:03, à 14:03, Willem Jiang a > écrit: > >+1 binding. > >It's good to see griffin is ready to graduate. > > > >Willem Jiang > > > >Twitter: willemjiang > >Weibo: 姜宁willem > > >

[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

[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

[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

Re: [VOTE] Graduate Apache Griffin (incubating) as a TLP

2018-11-01 Thread Dave Fisher
+1 (binding) Regards, Dave Sent from my iPhone > On Nov 1, 2018, at 12:31 AM, William Guo wrote: > > +dev > > -- Forwarded message - > From: William Guo > Date: Thu, Nov 1, 2018 at 11:21 AM > Subject: [VOTE] Graduate Apache Griffin (incubating) as a TLP > To: > > > Hi