[GitHub] incubator-gearpump pull request #228: [GEARPUMP-351] Support executor number...

2017-09-29 Thread darionyaphet
Github user darionyaphet closed the pull request at: https://github.com/apache/incubator-gearpump/pull/228 ---

[GitHub] incubator-gearpump pull request #228: [GEARPUMP-351] Support executor number...

2017-09-29 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/228 [GEARPUMP-351] Support executor number in submit When `ClientContext` submit application to cluster and must add a argument executors to specifies the executor number use

[GitHub] incubator-gearpump issue #200: [GEARPUMP-330] Allow examples to run in `sbt ...

2017-07-25 Thread darionyaphet
Github user darionyaphet commented on the issue: https://github.com/apache/incubator-gearpump/pull/200 maybe not only *gearpump-core* and *gearpump-streaming* should change to **compile** . --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-gearpump pull request #199: [GEARPUMP-328] FetchThread fetchSleepM...

2017-07-24 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/199 [GEARPUMP-328] FetchThread fetchSleepMS from Int to Long currently *KafkaConfig.FETCH_SLEEP_MS_CONFIG* is a INT parameter , move it into a LONG parameter is better . You can

[GitHub] incubator-gearpump issue #199: [GEARPUMP-328] FetchThread fetchSleepMS from ...

2017-07-24 Thread darionyaphet
Github user darionyaphet commented on the issue: https://github.com/apache/incubator-gearpump/pull/199 cc @manuzhang Could you make a review when you have time. thanks ~~ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-gearpump pull request #191: [GEARPUMP-319] Support Sorted Set in R...

2017-06-22 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/191 [GEARPUMP-319] Support Sorted Set in Redis Sorted Set is similar with Set in Redis with unique values . Each element storage in Sorted Set is associated with a score

[GitHub] incubator-gearpump pull request #125: [GEARPUMP-256] Camel-Case as variable ...

2016-12-23 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/125 [GEARPUMP-256] Camel-Case as variable name [Camel-Case as variable name](https://issues.apache.org/jira/browse/GEARPUMP-256) HBaseSink's naming and using Camel-Case

[GitHub] incubator-gearpump issue #102: [GEARPUMP-228] Message's timestamp should be ...

2016-10-31 Thread darionyaphet
Github user darionyaphet commented on the issue: https://github.com/apache/incubator-gearpump/pull/102 thanks @huafengw --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] incubator-gearpump pull request #102: [GEARPUMP-228] Message's timestamp sho...

2016-10-29 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/102 [GEARPUMP-228] Message's timestamp should be current time stamp [GEARPUMP-228 Message's timestamp should be current time stamp](https://issues.apache.org/jira/browse/GEARPUMP-228

[GitHub] incubator-gearpump pull request #89: GEARPUMP-216 Support Delete and Increme...

2016-09-25 Thread darionyaphet
Github user darionyaphet closed the pull request at: https://github.com/apache/incubator-gearpump/pull/89 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-gearpump pull request #89: GEARPUMP-216 Support Delete and Increme...

2016-09-25 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/89 GEARPUMP-216 Support Delete and Increment in HBaseSink [GEARPUMP-216](https://issues.apache.org/jira/browse/GEARPUMP-216) Support Delete and Increment in HBase Sink. You can

[GitHub] incubator-gearpump issue #11: fix #106 Gearpump Redis Integration

2016-07-01 Thread darionyaphet
Github user darionyaphet commented on the issue: https://github.com/apache/incubator-gearpump/pull/11 @kkasravi @manuzhang `RedisStorage` maybe need more discuss so I have remove it . After we have a discuss , I will repush it :) --- If your project is set up for it, you

[GitHub] incubator-gearpump issue #52: [GEARPUMP-152] Upgrade Storm support to 1.0.x

2016-06-28 Thread darionyaphet
Github user darionyaphet commented on the issue: https://github.com/apache/incubator-gearpump/pull/52 @manuzhang I found storm 1.X have add some function about Blob and LogConfig . I'm miss it and working . --- If your project is set up for it, you can reply to this email

[GitHub] incubator-gearpump issue #52: [GEARPUMP-152] Upgrade Storm support to 1.0.x

2016-06-28 Thread darionyaphet
Github user darionyaphet commented on the issue: https://github.com/apache/incubator-gearpump/pull/52 @manuzhang when I try to implement Nimbus method , I found some of them not implement will throw a UnsupportedOperationException . I want to know which command we are not support

[GitHub] incubator-gearpump pull request #52: [GEARPUMP-152] Upgrade Storm support to...

2016-06-28 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/52 [GEARPUMP-152] Upgrade Storm support to 1.0.x [GEARPUMP-152](https://issues.apache.org/jira/browse/GEARPUMP-152) Storm 1.0.0 has been released with a lot of changes. We need

[GitHub] incubator-gearpump pull request #51: [GEARPUMP-166] Rename sumByValue to sum...

2016-06-26 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/51 [GEARPUMP-166] Rename sumByValue to sumByKey [GEARPUMP-166](https://issues.apache.org/jira/browse/GEARPUMP-166) Streaming org.apache.gearpump.streaming.dsl.Stream have

[GitHub] incubator-gearpump pull request #11: fix #106 Gearpump Redis Integration

2016-06-02 Thread darionyaphet
Github user darionyaphet commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/11#discussion_r65502913 --- Diff: experiments/redis/src/main/scala/org/apache/gearpump/streaming/redis/RedisMessage.scala --- @@ -0,0 +1,65

[GitHub] incubator-gearpump pull request: fix GEARPUMP-141 Remove useless m...

2016-05-30 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/incubator-gearpump/pull/20#issuecomment-222503486 @huafengw @manuzhang Could add some comment about this functions ? This functions is very confuse to others . --- If your project is set up

[GitHub] incubator-gearpump pull request: fix GEARPUMP-141 Remove useless m...

2016-05-30 Thread darionyaphet
Github user darionyaphet closed the pull request at: https://github.com/apache/incubator-gearpump/pull/20 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-gearpump pull request: fix #106 Gearpump Redis Integrati...

2016-05-18 Thread darionyaphet
Github user darionyaphet commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/11#discussion_r63819150 --- Diff: project/BuildExample.scala --- @@ -229,4 +229,20 @@ object BuildExample extends sbt.Build

[GitHub] incubator-gearpump pull request: fix GEARPUMP-141 Remove useless m...

2016-05-17 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/incubator-gearpump/pull/20#issuecomment-219914766 @manuzhang but the method was decorate with `private` . --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-gearpump pull request: fix GEARPUMP-141 Remove useless m...

2016-05-17 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/incubator-gearpump/pull/20#issuecomment-219773664 conflicts ?? I can't found any conflicts ! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-gearpump pull request: fix GEARPUMP-141 , remove useless...

2016-05-17 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/20 fix GEARPUMP-141 , remove useless method [GEARPUMP-141](https://issues.apache.org/jira/browse/GEARPUMP-141) In external-hbase module , writeObject and readObject seems never used

[GitHub] incubator-gearpump pull request: fix GEARPUMP-141 , remove useless...

2016-05-17 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/19 fix GEARPUMP-141 , remove useless method [GEARPUMP-141](https://issues.apache.org/jira/browse/GEARPUMP-141) In external-hbase module , writeObject and readObject seems

[GitHub] incubator-gearpump pull request: fix #106 Gearpump Redis Integrati...

2016-05-14 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/incubator-gearpump/pull/11#issuecomment-219225271 # Sorry to reply it with so long . @manuzhang @huafengw redis examples class name have rename to `RedisSourceSinkExample

[GitHub] incubator-gearpump pull request: fix #106 Gearpump Redis Integrati...

2016-05-14 Thread darionyaphet
Github user darionyaphet commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/11#discussion_r63278015 --- Diff: examples/streaming/redis/src/main/scala/org/apache/gearpump/streaming/example/redis/RedisSourceSink.scala --- @@ -0,0 +1,67

[GitHub] incubator-gearpump pull request: fix #106 Gearpump Redis Integrati...

2016-05-11 Thread darionyaphet
Github user darionyaphet commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/11#discussion_r62852676 --- Diff: examples/streaming/redis/src/main/scala/org/apache/gearpump/streaming/example/redis/RedisSourceSink.scala --- @@ -0,0 +1,67

[GitHub] incubator-gearpump pull request: fix #106 Gearpump Redis Integrati...

2016-05-10 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/incubator-gearpump/pull/11#issuecomment-218355383 Hi @huafengw sure I will rename packages from io.gearpump to org.apache.gearpump :D --- If your project is set up for it, you can reply to this email

[GitHub] incubator-gearpump pull request: GEARPUMP-24, refactor DataSource ...

2016-05-09 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/incubator-gearpump/pull/7#issuecomment-217859410 Hi @manuzhang why change datasource API from `List[Message]` to `Message` ? --- If your project is set up for it, you can reply to this email

[GitHub] incubator-gearpump pull request: fix #121 DataSourceProcessor and ...

2016-05-09 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/incubator-gearpump/pull/13#issuecomment-217900186 @manuzhang When I using some processors to build a DAG , it's just to describe how to process the input stream , parallelism is the attribute

[GitHub] incubator-gearpump pull request: fix #106 Gearpump Redis Integrati...

2016-05-08 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/incubator-gearpump/pull/11#issuecomment-217726672 @manuzhang I have add license headers . @kkasravi I will add some unit tests . thank you :) --- If your project is set up

[GitHub] incubator-gearpump pull request: fix #121 DataSourceProcessor and ...

2016-05-07 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/incubator-gearpump/pull/12 fix #121 DataSourceProcessor and DataSinkProcessor default parallelism Set DataSourceProcessor and DataSinkProcessor's default parallelism value to 1. When user startup

[GitHub] incubator-gearpump pull request: gearpump redis

2016-05-06 Thread darionyaphet
Github user darionyaphet closed the pull request at: https://github.com/apache/incubator-gearpump/pull/10 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so