[jira] [Created] (STORM-1688) provide ParallismKillWorkerManager to shutdown workers in parallel

2016-04-05 Thread caofangkun (JIRA)
caofangkun created STORM-1688: - Summary: provide ParallismKillWorkerManager to shutdown workers in parallel Key: STORM-1688 URL: https://issues.apache.org/jira/browse/STORM-1688 Project: Apache Storm

[GitHub] storm pull request: [STORM-1567] in defaults.yaml 'topology.disabl...

2016-02-22 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/1135 [STORM-1567] in defaults.yaml 'topology.disable.loadaware' should be 'topology.disable.loadaware.messaging' You can merge this pull request into a Git repository by running: $ git pull

[jira] [Updated] (STORM-1567) in defaults.yaml 'topology.disable.loadaware' should be 'topology.disable.loadaware.messaging'

2016-02-22 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1567?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun updated STORM-1567: -- Description: {code:title=defaults.yaml|borderStyle=solid} diff --git a/conf/defaults.yaml b/conf

[jira] [Created] (STORM-1567) in defaults.yaml 'topology.disable.loadaware' should be 'topology.disable.loadaware.messaging'

2016-02-22 Thread caofangkun (JIRA)
caofangkun created STORM-1567: - Summary: in defaults.yaml 'topology.disable.loadaware' should be 'topology.disable.loadaware.messaging' Key: STORM-1567 URL: https://issues.apache.org/jira/browse/STORM-1567

[GitHub] storm pull request: [STORM-1501] launch worker process exception w...

2016-01-26 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/1046 [STORM-1501] launch worker process exception will cause supervisor process exited [util.clj/async-loop](https://github.com/apache/storm/blob/master/storm-core/src/clj/org/apache/storm/util.clj

[jira] [Created] (STORM-1501) launch worker process exception will cause supervisor process exited

2016-01-25 Thread caofangkun (JIRA)
caofangkun created STORM-1501: - Summary: launch worker process exception will cause supervisor process exited Key: STORM-1501 URL: https://issues.apache.org/jira/browse/STORM-1501 Project: Apache Storm

[GitHub] storm pull request: [STORM-1453] nimbus.clj/wait-for-desired-code-...

2016-01-11 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/1003 [STORM-1453] nimbus.clj/wait-for-desired-code-replication print wrong log message You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] storm pull request: [STORM-1453] nimbus.clj/wait-for-desired-code-...

2016-01-11 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/996#issuecomment-170755067 Updated as PR #1003 Close this PR . --- 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

[GitHub] storm pull request: [STORM-1453] nimbus.clj/wait-for-desired-code-...

2016-01-11 Thread caofangkun
Github user caofangkun closed the pull request at: https://github.com/apache/storm/pull/996 --- 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, or if the feature

[jira] [Created] (STORM-1453) nimbus.clj/wait-for-desired-code-replication print wrong log message

2016-01-08 Thread caofangkun (JIRA)
caofangkun created STORM-1453: - Summary: nimbus.clj/wait-for-desired-code-replication print wrong log message Key: STORM-1453 URL: https://issues.apache.org/jira/browse/STORM-1453 Project: Apache Storm

[GitHub] storm pull request: [STORM-1453] nimbus.clj/wait-for-desired-code-...

2016-01-08 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/996 [STORM-1453] nimbus.clj/wait-for-desired-code-replication print wrong log message You can merge this pull request into a Git repository by running: $ git pull https://github.com/caofangkun

[GitHub] storm pull request: [STORM-668] add external project storm-ui and ...

2016-01-05 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/992 [STORM-668] add external project storm-ui and generate storm-ui.war and could be deployed on TomCat You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] storm pull request: STORM-668:Generate storm-ui.war and could be d...

2016-01-05 Thread caofangkun
Github user caofangkun closed the pull request at: https://github.com/apache/storm/pull/423 --- 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, or if the feature

[jira] [Created] (STORM-1427) add TupleUtils/listHashCode method and delete tuple.clj

2015-12-30 Thread caofangkun (JIRA)
caofangkun created STORM-1427: - Summary: add TupleUtils/listHashCode method and delete tuple.clj Key: STORM-1427 URL: https://issues.apache.org/jira/browse/STORM-1427 Project: Apache Storm

[jira] [Created] (STORM-1424) Removed unused topology-path variable in config.clj/read-supervisor-storm-conf

2015-12-28 Thread caofangkun (JIRA)
caofangkun created STORM-1424: - Summary: Removed unused topology-path variable in config.clj/read-supervisor-storm-conf Key: STORM-1424 URL: https://issues.apache.org/jira/browse/STORM-1424 Project

[GitHub] storm pull request: [STORM-1424] Removed unused topology-path vari...

2015-12-28 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/981 [STORM-1424] Removed unused topology-path variable You can merge this pull request into a Git repository by running: $ git pull https://github.com/caofangkun/apache-storm storm-1424

[GitHub] storm pull request: fastfix-01:Removed unused topology-path variab...

2015-12-28 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/469#issuecomment-167691552 @d2r this branch is outdated, I fired a new PR #981 and created jira [STORM-1424](https://issues.apache.org/jira/browse/STORM-1424) for it . So close

[GitHub] storm pull request: fastfix-01:Removed unused topology-path variab...

2015-12-28 Thread caofangkun
Github user caofangkun closed the pull request at: https://github.com/apache/storm/pull/469 --- 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, or if the feature

[GitHub] storm pull request: [STORM-1426] keep backtype.storm.tuple.Address...

2015-12-28 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/983 [STORM-1426] keep backtype.storm.tuple.AddressedTuple and delete duplicated backtype.storm.messaging.AddressedTuple You can merge this pull request into a Git repository by running: $ git

[jira] [Created] (STORM-1426) keep backtype.storm.tuple.AddressedTuple and delete duplicated backtype.storm.messaging.AddressedTuple

2015-12-28 Thread caofangkun (JIRA)
caofangkun created STORM-1426: - Summary: keep backtype.storm.tuple.AddressedTuple and delete duplicated backtype.storm.messaging.AddressedTuple Key: STORM-1426 URL: https://issues.apache.org/jira/browse/STORM

[GitHub] storm pull request: Update timer.clj

2015-08-12 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/680#discussion_r36937217 --- Diff: storm-core/src/clj/backtype/storm/timer.clj --- @@ -53,8 +53,11 @@ ;; event generation. If any recurring

[GitHub] storm pull request: STORM-974 Introduces Tuple - ES document mapp...

2015-08-12 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/679#discussion_r36938144 --- Diff: external/storm-elasticsearch/src/main/java/org/apache/storm/elasticsearch/trident/EsStateFactory.java --- @@ -19,6 +19,7 @@ import

[GitHub] storm pull request: STORM-974 Introduces Tuple - ES document mapp...

2015-08-12 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/679#discussion_r36938292 --- Diff: external/storm-elasticsearch/src/test/java/org/apache/storm/elasticsearch/bolt/EsPercolateBoltTest.java --- @@ -21,6 +21,7 @@ import

[jira] [Created] (STORM-988) supervisor.slots.ports should not allow duplicate element values

2015-08-12 Thread caofangkun (JIRA)
caofangkun created STORM-988: Summary: supervisor.slots.ports should not allow duplicate element values Key: STORM-988 URL: https://issues.apache.org/jira/browse/STORM-988 Project: Apache Storm

[GitHub] storm pull request: STORM-974 Introduces Tuple - ES document mapp...

2015-08-12 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/679#discussion_r36938034 --- Diff: external/storm-elasticsearch/src/main/java/org/apache/storm/elasticsearch/bolt/EsPercolateBolt.java --- @@ -31,14 +32,21 @@ --- End

[GitHub] storm pull request: STORM-974 Introduces Tuple - ES document mapp...

2015-08-12 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/679#discussion_r36938233 --- Diff: external/storm-elasticsearch/src/test/java/org/apache/storm/elasticsearch/bolt/EsIndexBoltTest.java --- @@ -20,6 +20,7 @@ import

[GitHub] storm pull request: STORM-988: supervisor.slots.ports should not a...

2015-08-12 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/681 STORM-988: supervisor.slots.ports should not allow duplicate element values You can merge this pull request into a Git repository by running: $ git pull https://github.com/caofangkun/apache

[GitHub] storm pull request: [STORM-949] On the topology summary UI page, a...

2015-08-11 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/675#discussion_r36819256 --- Diff: storm-core/src/ui/public/templates/topology-page-template.html --- @@ -313,6 +320,10 @@ /th th class=headerLast error

[jira] [Created] (STORM-985) provide .editorconfig with git pre-commit hooks to check code style before commit

2015-08-11 Thread caofangkun (JIRA)
caofangkun created STORM-985: Summary: provide .editorconfig with git pre-commit hooks to check code style before commit Key: STORM-985 URL: https://issues.apache.org/jira/browse/STORM-985 Project

[jira] [Created] (STORM-986) Caching maven dependencies to speedup CI builds

2015-08-11 Thread caofangkun (JIRA)
caofangkun created STORM-986: Summary: Caching maven dependencies to speedup CI builds Key: STORM-986 URL: https://issues.apache.org/jira/browse/STORM-986 Project: Apache Storm Issue Type

[GitHub] storm pull request: STORM-986: Caching maven dependencies to speed...

2015-08-11 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/677 STORM-986: Caching maven dependencies to speedup CI builds You can merge this pull request into a Git repository by running: $ git pull https://github.com/caofangkun/apache-storm storm-986

[GitHub] storm pull request: STORM-977 Correct log prefix (-15) when sendin...

2015-08-10 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/672#discussion_r36708996 --- Diff: storm-core/src/clj/backtype/storm/daemon/supervisor.clj --- @@ -262,7 +262,7 @@ (psim/kill-process thread-pid)) (doseq [pid

[GitHub] storm pull request: STORM-977 Correct log prefix (-15) when sendin...

2015-08-10 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/672#issuecomment-129691333 The travis ci build failed does not make sense , Does ``` export STORM_TEST_TIMEOUT_MS=10``` not long enough ? --- If your project is set up for it, you can

[GitHub] storm pull request: STORM-959:remove unnecessary dependency from s...

2015-08-10 Thread caofangkun
Github user caofangkun closed the pull request at: https://github.com/apache/storm/pull/652 --- 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, or if the feature

[GitHub] storm pull request: STORM-904: Move bin/storm command line to java...

2015-08-05 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/662#discussion_r36284000 --- Diff: storm-core/src/jvm/backtype/storm/utils/StormCommandExecutor.java --- @@ -0,0 +1,785 @@ +package backtype.storm.utils; + +import

[jira] [Closed] (STORM-970) UT messaging_test.clj#test-receiver-message-order build failed

2015-08-05 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-970?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun closed STORM-970. Resolution: Not A Problem When I set STORM_TEST_TIMEOUT_MS larger , the UT passed. {code} ➜ storm git

[GitHub] storm pull request: Storm-Kafka trident topology example

2015-08-04 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/666#discussion_r36169516 --- Diff: examples/storm-starter/pom.xml --- @@ -96,6 +96,12 @@ groupIdcom.google.guava/groupId artifactIdguava/artifactId

[GitHub] storm pull request: STORM-966 ConfigValidation.DoubleValidator is ...

2015-08-04 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/658#issuecomment-127811188 I checked the CI build log and find https://travis-ci.org/apache/storm/jobs/74073153#L4132 and there's a PR on this , @HeartSaVioR Could you please

[GitHub] storm pull request: STORM-966 ConfigValidation.DoubleValidator is ...

2015-08-03 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/658#discussion_r36149821 --- Diff: storm-core/src/jvm/backtype/storm/ConfigValidation.java --- @@ -28,7 +28,6 @@ /** * Declares methods for validating

[jira] [Closed] (STORM-751) Move javadoc aggregate to site phase.

2015-08-02 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-751?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun closed STORM-751. Resolution: Fixed Fixed in https://github.com/apache/storm/commit/e2e17e1c7899fb918562bbad21a7e68394b5f285

[GitHub] storm pull request: STORM-913:Use Curator's delete().deletingChild...

2015-08-02 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/615#issuecomment-127095071 @revans2 Please don't forget check this PR in :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] storm pull request: STORM-933:NullPointerException during KafkaSpo...

2015-07-30 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/660 STORM-933:NullPointerException during KafkaSpout deactivation You can merge this pull request into a Git repository by running: $ git pull https://github.com/caofangkun/apache-storm storm

[jira] [Assigned] (STORM-933) NullPointerException during KafkaSpout deactivation

2015-07-30 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-933?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun reassigned STORM-933: Assignee: caofangkun NullPointerException during KafkaSpout deactivation

[GitHub] storm pull request: STORM-966 ConfigValidation.DoubleValidator is ...

2015-07-30 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/658#issuecomment-126202559 +1 --- 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] storm pull request: STORM-964, add config for worker logwriter to ...

2015-07-29 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/657#discussion_r35833865 --- Diff: storm-core/src/clj/backtype/storm/daemon/supervisor.clj --- @@ -675,6 +675,7 @@ (add-to-classpath topo-classpath

[jira] [Commented] (STORM-838) Upgrade Jackson

2015-07-27 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14642369#comment-14642369 ] caofangkun commented on STORM-838: -- It may be not necessary to update jackson version . I

[jira] [Created] (STORM-962) travis user's $SHELL env is /bin/sh but should be /bin/bash

2015-07-27 Thread caofangkun (JIRA)
caofangkun created STORM-962: Summary: travis user's $SHELL env is /bin/sh but should be /bin/bash Key: STORM-962 URL: https://issues.apache.org/jira/browse/STORM-962 Project: Apache Storm

[jira] [Closed] (STORM-551) Will be better if Use storm.conf.dir instead of storm.conf.file

2015-07-24 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-551?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun closed STORM-551. Resolution: Duplicate Will be better if Use storm.conf.dir instead of storm.conf.file

[GitHub] storm pull request: STORM-951. Storm Hive connector leaking connec...

2015-07-24 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/645#discussion_r35401647 --- Diff: external/storm-hive/src/main/java/org/apache/storm/hive/bolt/HiveBolt.java --- @@ -170,10 +170,10 @@ public void run

[GitHub] storm pull request: STORM-927: make storm jar wait for deployment ...

2015-07-24 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/623#issuecomment-124425109 @DevInsanity Do your topology use [StormSubmitter.submitTopologyWithProgressBar](https://github.com/apache/storm/blob/master/storm-core/src/jvm/backtype/storm

[GitHub] storm pull request: [STORM-440] completed exposing the drpcclient ...

2015-07-24 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/648#discussion_r35406122 --- Diff: storm-core/src/jvm/backtype/storm/utils/DRPCClient.java --- @@ -46,7 +46,22 @@ public DRPCClient(Map conf, String host, int port, Integer timeout

[GitHub] storm pull request: STORM-914: AutoHDFS should also update hdfs.co...

2015-07-24 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/606#discussion_r35409819 --- Diff: external/storm-hdfs/src/test/java/org/apache/storm/hdfs/trident/TridentSequenceTopology.java --- @@ -60,7 +61,7 @@ public static StormTopology

[GitHub] storm pull request: [STORM-440] completed exposing the drpcclient ...

2015-07-24 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/648#discussion_r35406230 --- Diff: storm-core/src/jvm/backtype/storm/utils/NimbusClient.java --- @@ -30,11 +30,18 @@ private Nimbus.Client _client; private static

[GitHub] storm pull request: STORM-959:remove unnecessary dependency from s...

2015-07-23 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/652#issuecomment-124295316 duplicates #609 --- 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

[jira] [Created] (STORM-959) remove unnecessary dependency from storm-hive/pom.xml

2015-07-23 Thread caofangkun (JIRA)
caofangkun created STORM-959: Summary: remove unnecessary dependency from storm-hive/pom.xml Key: STORM-959 URL: https://issues.apache.org/jira/browse/STORM-959 Project: Apache Storm Issue Type

[GitHub] storm pull request: STORM-959:remove unnecessary dependency from s...

2015-07-23 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/652#issuecomment-124034146 @harshach Could you please have a look at this PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] storm pull request: STORM-959:remove unnecessary dependency from s...

2015-07-23 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/652 STORM-959:remove unnecessary dependency from storm-hive/pom.xml org.apache.calcite:calcite-core:0.9.2-incubating does not take affect at all. You can merge this pull request into a Git

[GitHub] storm pull request: STORM-947: replace all `backtype.storm.schedul...

2015-07-21 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/640#discussion_r35073778 --- Diff: storm-core/src/clj/backtype/storm/scheduler/DefaultScheduler.clj --- @@ -18,7 +18,7 @@ (:require [backtype.storm.scheduler.EvenScheduler

[jira] [Created] (STORM-955) replace all backtype.storm.daemon.common.clj#StormBase with backtype.storm.generated.StormBase

2015-07-21 Thread caofangkun (JIRA)
caofangkun created STORM-955: Summary: replace all backtype.storm.daemon.common.clj#StormBase with backtype.storm.generated.StormBase Key: STORM-955 URL: https://issues.apache.org/jira/browse/STORM-955

[GitHub] storm pull request: STORM-946: We should remove Closed Client form...

2015-07-17 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/639#issuecomment-122203636 @tedxia I could only find two situations ( am I right?) will change ```client.status``` to ```closed``` by call ```(.close socket)```. and do not need reconnect

[GitHub] storm pull request: STORM-946: We should remove Closed Client form...

2015-07-17 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/639#discussion_r34865678 --- Diff: storm-core/src/clj/backtype/storm/daemon/worker.clj --- @@ -288,7 +288,18 @@ (filter-key (complement

[GitHub] storm pull request: STORM-947: replace all `backtype.storm.schedul...

2015-07-17 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/640 STORM-947: replace all `backtype.storm.scheduler.ExecutorDetails` with `backtype.storm.generated.ExecutorInfo ` You can merge this pull request into a Git repository by running: $ git

[jira] [Commented] (STORM-930) Nimbus server shutting down

2015-07-16 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-930?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14630676#comment-14630676 ] caofangkun commented on STORM-930: -- See: https://github.com/apache/storm/blob/v0.9.3-rc1

[GitHub] storm pull request: STORM-913:Use Curator's delete().deletingChild...

2015-07-16 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/615#issuecomment-122156319 @revans2 The whole usages call delete-node with ```:force true```, that is the ```:force ``` parameter is unneeded at all. --- If your project is set up

[GitHub] storm pull request: STORM-945:DefaultRolloverStrategy element is...

2015-07-16 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/638 STORM-945:DefaultRolloverStrategy element is not a policy,and should not be putted in the Policies element ``` - storm-current bin/storm nimbus Running: /usr/share/jdk-current/bin/java

[jira] [Created] (STORM-945) DefaultRolloverStrategy element is not a policy,and should not be putted in the Policies element.

2015-07-16 Thread caofangkun (JIRA)
caofangkun created STORM-945: Summary: DefaultRolloverStrategy element is not a policy,and should not be putted in the Policies element. Key: STORM-945 URL: https://issues.apache.org/jira/browse/STORM-945

[GitHub] storm pull request: STORM-924:Set the file mode of the files inclu...

2015-07-05 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/614#issuecomment-118686693 Thanks @HeartSaVioR . I have fixed as you mentioned. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[jira] [Created] (STORM-926) change pom to use maven-shade-plugin:1.6 to minimize the size of shade jar

2015-07-05 Thread caofangkun (JIRA)
caofangkun created STORM-926: Summary: change pom to use maven-shade-plugin:1.6 to minimize the size of shade jar Key: STORM-926 URL: https://issues.apache.org/jira/browse/STORM-926 Project: Apache

[jira] [Updated] (STORM-926) change pom to use maven-shade-plugin:2.2 and minimize the size of shade jar and fix duplicated classes warnings of flue-examples

2015-07-05 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-926?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun updated STORM-926: - Summary: change pom to use maven-shade-plugin:2.2 and minimize the size of shade jar and fix duplicated

[GitHub] storm pull request: STORM-926:change pom to use maven-shade-plugin...

2015-07-05 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/621 STORM-926:change pom to use maven-shade-plugin:2.2 and minimize the size of shade jar and fix duplicated classes warnings of flue-examples 1: fix external/flux/flux-examples duplicated classes

[jira] [Created] (STORM-924) Set the file mode of the files included when packaging release packages

2015-07-02 Thread caofangkun (JIRA)
caofangkun created STORM-924: Summary: Set the file mode of the files included when packaging release packages Key: STORM-924 URL: https://issues.apache.org/jira/browse/STORM-924 Project: Apache Storm

[jira] [Created] (STORM-920) Should we remove duplicated classes/code ?

2015-06-30 Thread caofangkun (JIRA)
caofangkun created STORM-920: Summary: Should we remove duplicated classes/code ? Key: STORM-920 URL: https://issues.apache.org/jira/browse/STORM-920 Project: Apache Storm Issue Type

[GitHub] storm pull request: STORM-913:Use Curator's delete().deletingChild...

2015-06-25 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/605#discussion_r33326733 --- Diff: pom.xml --- @@ -211,7 +211,7 @@ guava.version16.0.1/guava.version netty.version3.9.0.Final/netty.version

[jira] [Created] (STORM-913) Use Curator's delete().deletingChildrenIfNeeded() instead of zk/delete-recursive

2015-06-25 Thread caofangkun (JIRA)
caofangkun created STORM-913: Summary: Use Curator's delete().deletingChildrenIfNeeded() instead of zk/delete-recursive Key: STORM-913 URL: https://issues.apache.org/jira/browse/STORM-913 Project

[GitHub] storm pull request: STORM-913:Use Curator's delete().deletingChild...

2015-06-25 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/605 STORM-913:Use Curator's delete().deletingChildrenIfNeeded() instead of zk/delete-recursive You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] storm pull request: STORM-668:Generate storm-ui.war and could be d...

2015-06-17 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/423#issuecomment-112737243 @d2r Thank you for your attention of this PR. In my opinion , 1, storm ui is more like a RESTful APIs server than a UI server. 2, We can use jetty

[jira] [Updated] (STORM-736) Add a RESTful API to print all of the thread's information and stack traces of Nimbus/Supervisor/Worker Process

2015-03-29 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-736?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun updated STORM-736: - Summary: Add a RESTful API to print all of the thread's information and stack traces of Nimbus/Supervisor

[jira] [Created] (STORM-736) Private a RESTful API to print all of the thread's information and stack traces of Nimbus/Supervisor/Worker Process

2015-03-29 Thread caofangkun (JIRA)
caofangkun created STORM-736: Summary: Private a RESTful API to print all of the thread's information and stack traces of Nimbus/Supervisor/Worker Process Key: STORM-736 URL: https://issues.apache.org/jira/browse

[GitHub] storm pull request: STORM-587. trident transactional state in zk s...

2015-03-25 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/475#discussion_r27186979 --- Diff: storm-core/src/jvm/backtype/storm/transactional/state/TransactionalState.java --- @@ -59,7 +59,8 @@ protected TransactionalState(Map conf

[GitHub] storm pull request: STORM-721. Storm UI server should support SSL.

2015-03-24 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/479#discussion_r27090305 --- Diff: storm-core/src/jvm/backtype/storm/Config.java --- @@ -567,6 +567,39 @@ public static final Object UI_HEADER_BUFFER_BYTES_SCHEMA

[GitHub] storm pull request: STORM-694: java.lang.ClassNotFoundException: b...

2015-03-24 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/449#issuecomment-85783849 @duke-bartholomew It works very well when I try to reproduce this issue: ``` git clone https://github.com/apache/storm.git cd storm mvn

[GitHub] storm pull request: STORM-553:add eclipse java formatter file for ...

2015-03-23 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/478 STORM-553:add eclipse java formatter file for Apache Storm Java Code You can merge this pull request into a Git repository by running: $ git pull https://github.com/caofangkun/apache-storm

[GitHub] storm pull request: STORM-694: java.lang.ClassNotFoundException: b...

2015-03-23 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/449#issuecomment-85329513 @duke-bartholomew Does this still a issue? See SupervisorInfo has already be imported at : https://github.com/apache/storm/blob/master/storm-core/src

[GitHub] storm pull request: fastfix-01:modify config.read-supervisor-storm...

2015-03-18 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/469 fastfix-01:modify config.read-supervisor-storm-conf function remove unused topology-path You can merge this pull request into a Git repository by running: $ git pull https://github.com

[jira] [Updated] (STORM-701) provide eclipse plugin for monitoring/debugging user topology

2015-03-17 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-701?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun updated STORM-701: - Attachment: New Storm Topology Project-03.jpg New Storm Topology Project-02.jpg

[jira] [Updated] (STORM-701) provide eclipse plugin for monitoring/debugging user topology

2015-03-17 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-701?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun updated STORM-701: - Description: provide eclipse plugin for debugging user topology, submit topologies and monitor

[jira] [Updated] (STORM-701) provide eclipse plugin for monitoring/debugging user topology

2015-03-17 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-701?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun updated STORM-701: - Description: provide eclipse plugin for debugging user topology, submit topologies and monitor

[jira] [Updated] (STORM-701) provide eclipse plugin for creating/monitoring/debugging user topology

2015-03-17 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-701?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun updated STORM-701: - Summary: provide eclipse plugin for creating/monitoring/debugging user topology (was: provide eclipse

[GitHub] storm pull request: STORM-541:Clean duplicate dependences in poms

2015-03-12 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/422#discussion_r26361833 --- Diff: pom.xml --- @@ -521,12 +521,6 @@ scopetest/scope /dependency dependency

[GitHub] storm pull request: STORM-638:UI should show up process-id of the ...

2015-03-12 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/396#discussion_r26362269 --- Diff: storm-core/src/storm.thrift --- @@ -205,7 +205,8 @@ struct ExecutorSummary { 2: required string component_id; 3: required string

[jira] [Created] (STORM-701) provide eclipse plugin for monitoring/debugging user topology

2015-03-11 Thread caofangkun (JIRA)
caofangkun created STORM-701: Summary: provide eclipse plugin for monitoring/debugging user topology Key: STORM-701 URL: https://issues.apache.org/jira/browse/STORM-701 Project: Apache Storm

[GitHub] storm pull request: STORM-534:Store Nimbus Server Information in z...

2015-02-10 Thread caofangkun
Github user caofangkun closed the pull request at: https://github.com/apache/storm/pull/394 --- 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, or if the feature

[GitHub] storm pull request: STORM-534:Store Nimbus Server Information in z...

2015-02-10 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/394#issuecomment-73822455 Let's close it for now.More information will be discussed on [STORM-654](https://issues.apache.org/jira/browse/STORM-654) --- If your project is set up for it, you

[jira] [Closed] (STORM-534) Store Nimbus Server Information in zookeeper path {storm.zookeeper.root}/nimbus

2015-02-10 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-534?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun closed STORM-534. Resolution: Duplicate Store Nimbus Server Information in zookeeper path {storm.zookeeper.root}/nimbus

[jira] [Updated] (STORM-657) make the shutdown-worker sleep time before kill -9 configurable

2015-02-09 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-657?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun updated STORM-657: - Affects Version/s: 0.10.0 make the shutdown-worker sleep time before kill -9 configurable

[jira] [Created] (STORM-668) Generate storm-ui war using maven and deployed on TomCat

2015-02-09 Thread caofangkun (JIRA)
caofangkun created STORM-668: Summary: Generate storm-ui war using maven and deployed on TomCat Key: STORM-668 URL: https://issues.apache.org/jira/browse/STORM-668 Project: Apache Storm Issue

[GitHub] storm pull request: STORM-541:Clean duplicate dependences in poms

2015-02-09 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/422#issuecomment-73626073 @revans2 Could you please have a review on this PR? Thanks a lot . --- If your project is set up for it, you can reply to this email and have your reply appear

[jira] [Reopened] (STORM-534) Store Nimbus Server Information in zookeeper path {storm.zookeeper.root}/nimbus

2015-02-09 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-534?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun reopened STORM-534: -- Store Nimbus Server Information in zookeeper path {storm.zookeeper.root}/nimbus

[jira] [Updated] (STORM-657) make the shutdown-worker sleep time before kill -9 configurable

2015-02-09 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-657?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun updated STORM-657: - Fix Version/s: (was: 0.9.2-incubating) make the shutdown-worker sleep time before kill -9

[GitHub] storm pull request: STORM-668:Generate storm-ui.war and could be d...

2015-02-09 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/423 STORM-668:Generate storm-ui.war and could be deployed on TomCat ``` ➜ storm-core git:(storm-668) mvn clean mvn install -Dmaven.test.skip=true -Dwar ➜ storm-core git:(storm-668) cp

  1   2   >