[GitHub] storm pull request: STORM-433: Give users visibility to the depth ...

2015-02-05 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/236#issuecomment-73051461 > Do we still want this? Yes, I think this would be a tremendous help to users while debugging "slowness" issues. --- If your project is set up for it, you can reply to t

[jira] [Commented] (STORM-433) Give users visibility to the depth of queues at each bolt

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-433?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307242#comment-14307242 ] ASF GitHub Bot commented on STORM-433: -- Github user d2r commented on the pull request:

[GitHub] storm pull request: STORM-441: Remove bootstrap macro from Clojure...

2015-02-05 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/237#discussion_r24165018 --- Diff: storm-core/src/clj/backtype/storm/daemon/nimbus.clj --- @@ -14,25 +14,36 @@ ;; See the License for the specific language governing permissions and

[jira] [Commented] (STORM-441) Remove bootstrap macro from Clojure codebase

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307246#comment-14307246 ] ASF GitHub Bot commented on STORM-441: -- Github user d2r commented on a diff in the pul

[GitHub] storm pull request: STORM-441: Remove bootstrap macro from Clojure...

2015-02-05 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/237#discussion_r24165194 --- Diff: storm-core/src/clj/backtype/storm/daemon/executor.clj --- @@ -52,7 +60,7 @@ )) (defn- mk-grouper - "Returns a function that retur

[jira] [Commented] (STORM-441) Remove bootstrap macro from Clojure codebase

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307248#comment-14307248 ] ASF GitHub Bot commented on STORM-441: -- Github user d2r commented on a diff in the pul

[GitHub] storm pull request: STORM-441: Remove bootstrap macro from Clojure...

2015-02-05 Thread knusbaum
Github user knusbaum commented on a diff in the pull request: https://github.com/apache/storm/pull/237#discussion_r24169370 --- Diff: storm-core/src/clj/backtype/storm/daemon/nimbus.clj --- @@ -14,25 +14,36 @@ ;; See the License for the specific language governing permissions a

[jira] [Commented] (STORM-441) Remove bootstrap macro from Clojure codebase

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307373#comment-14307373 ] ASF GitHub Bot commented on STORM-441: -- Github user knusbaum commented on a diff in th

[GitHub] storm pull request: STORM-441: Remove bootstrap macro from Clojure...

2015-02-05 Thread knusbaum
Github user knusbaum commented on a diff in the pull request: https://github.com/apache/storm/pull/237#discussion_r24169449 --- Diff: storm-core/src/clj/backtype/storm/daemon/executor.clj --- @@ -52,7 +60,7 @@ )) (defn- mk-grouper - "Returns a function that

[jira] [Commented] (STORM-441) Remove bootstrap macro from Clojure codebase

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307374#comment-14307374 ] ASF GitHub Bot commented on STORM-441: -- Github user knusbaum commented on a diff in th

[GitHub] storm pull request: STORM-658:when config topology.acker.executors...

2015-02-05 Thread harshach
Github user harshach commented on a diff in the pull request: https://github.com/apache/storm/pull/417#discussion_r24178428 --- Diff: storm-core/src/clj/backtype/storm/daemon/common.clj --- @@ -312,7 +312,7 @@ )) (defn has-ackers? [storm-conf] - (or (nil? (s

[jira] [Commented] (STORM-658) config topology.acker.executors default value is null and then should not start acker bolts

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-658?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307546#comment-14307546 ] ASF GitHub Bot commented on STORM-658: -- Github user harshach commented on a diff in th

[GitHub] storm pull request: STORM-658:when config topology.acker.executors...

2015-02-05 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/417#discussion_r24178855 --- Diff: storm-core/src/clj/backtype/storm/daemon/common.clj --- @@ -312,7 +312,7 @@ )) (defn has-ackers? [storm-conf] - (or (nil? (storm-

[jira] [Commented] (STORM-658) config topology.acker.executors default value is null and then should not start acker bolts

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-658?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307555#comment-14307555 ] ASF GitHub Bot commented on STORM-658: -- Github user d2r commented on a diff in the pul

[GitHub] storm pull request: STORM-130: Supervisor getting killed due to ja...

2015-02-05 Thread harshach
GitHub user harshach opened a pull request: https://github.com/apache/storm/pull/418 STORM-130: Supervisor getting killed due to java.io.FileNotFoundException: File '../stormconf.ser' does not exist. You can merge this pull request into a Git repository by running: $ git pull

[jira] [Commented] (STORM-130) [Storm 0.8.2]: java.io.FileNotFoundException: File '../stormconf.ser' does not exist

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307673#comment-14307673 ] ASF GitHub Bot commented on STORM-130: -- GitHub user harshach opened a pull request:

[GitHub] storm pull request: STORM-130: Supervisor getting killed due to ja...

2015-02-05 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/401#issuecomment-73105790 @revans2 @ptgoetz opened a PR against 0.9.3-branch https://github.com/apache/storm/pull/418 . Also updated the current PR to handle another case. This issue can happ

[jira] [Commented] (STORM-130) [Storm 0.8.2]: java.io.FileNotFoundException: File '../stormconf.ser' does not exist

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307781#comment-14307781 ] ASF GitHub Bot commented on STORM-130: -- Github user harshach commented on the pull req

[GitHub] storm pull request: update bylaws for adoption discussion

2015-02-05 Thread ptgoetz
GitHub user ptgoetz opened a pull request: https://github.com/apache/storm/pull/419 update bylaws for adoption discussion Please see [DISCUSS] thread on d...@storm.apache.com NOTE: This pull request should not be merged until a successful VOTE for adoption has taken place.

[DISCUSS] Adopt Apache Storm Bylaws

2015-02-05 Thread P. Taylor Goetz
Associated pull request can be found here: https://github.com/apache/storm/pull/419 This is another attempt at gaining consensus regarding adopting official bylaws for the Apache Storm project. The changes are minor and should be apparent in the pull request diff. In earlier discussions, ther

[jira] [Assigned] (STORM-617) In Storm secure mode re-deploying trident topology causes zookeeper ACL issue

2015-02-05 Thread Sriharsha Chintalapani (JIRA)
[ https://issues.apache.org/jira/browse/STORM-617?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sriharsha Chintalapani reassigned STORM-617: Assignee: Sriharsha Chintalapani > In Storm secure mode re-deploying trident

[GitHub] storm pull request: STORM-658:when config topology.acker.executors...

2015-02-05 Thread caofangkun
Github user caofangkun commented on a diff in the pull request: https://github.com/apache/storm/pull/417#discussion_r24216861 --- Diff: storm-core/src/clj/backtype/storm/daemon/common.clj --- @@ -312,7 +312,7 @@ )) (defn has-ackers? [storm-conf] - (or (nil?

[jira] [Commented] (STORM-658) config topology.acker.executors default value is null and then should not start acker bolts

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-658?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14308415#comment-14308415 ] ASF GitHub Bot commented on STORM-658: -- Github user caofangkun commented on a diff in

[GitHub] storm pull request: [STORM-609] Add storm-redis to storm external

2015-02-05 Thread dashengju
Github user dashengju commented on the pull request: https://github.com/apache/storm/pull/365#issuecomment-73170761 @HeartSaVioR , @ptgoetz I agree with "committer sponsors" only committers can join external's "committer sponsors". I have changed the committer sponsors membe

[jira] [Commented] (STORM-609) add storm-redis to storm external

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-609?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14308466#comment-14308466 ] ASF GitHub Bot commented on STORM-609: -- Github user dashengju commented on the pull re

[jira] [Commented] (STORM-400) Thrift upgrade to 0.9.1

2015-02-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-400?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14308496#comment-14308496 ] ASF GitHub Bot commented on STORM-400: -- Github user kishorvpatil commented on the pull

[GitHub] storm pull request: [STORM-400] Thrift upgrade to thrift-0.9.2

2015-02-05 Thread kishorvpatil
Github user kishorvpatil commented on the pull request: https://github.com/apache/storm/pull/410#issuecomment-73174063 @ptgoetz, Thanks for reviewing this. I have up merged the changes. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi