[jira] [Created] (STORM-316) Added registerMetric validation to have timeBucketSizeInSecs = 1

2014-05-15 Thread Francois Dang Ngoc (JIRA)
Francois Dang Ngoc created STORM-316: Summary: Added registerMetric validation to have timeBucketSizeInSecs = 1 Key: STORM-316 URL: https://issues.apache.org/jira/browse/STORM-316 Project: Apache

[jira] [Commented] (STORM-233) Avoid worker killed on startup because of heavy ZK load

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-233?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13998134#comment-13998134 ] ASF GitHub Bot commented on STORM-233: -- Github user nathanmarz commented on the pull

[jira] [Commented] (STORM-138) Pluggable serialization for multilang

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13998143#comment-13998143 ] ASF GitHub Bot commented on STORM-138: -- Github user d2r commented on the pull request:

[GitHub] incubator-storm pull request: STORM-245: implement Stream.localOrS...

2014-05-15 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/incubator-storm/pull/43#issuecomment-43141466 +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

[jira] [Updated] (STORM-320) Support STORM_CONF_DIR environment variable for alternate conf dir

2014-05-15 Thread Albert Chu (JIRA)
[ https://issues.apache.org/jira/browse/STORM-320?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Albert Chu updated STORM-320: - Attachment: STORM-320.patch Support STORM_CONF_DIR environment variable for alternate conf dir

[GitHub] incubator-storm pull request: Change Twitter authentication

2014-05-15 Thread prabeesh
Github user prabeesh commented on the pull request: https://github.com/apache/incubator-storm/pull/62#issuecomment-42385017 Added new JIRA https://issues.apache.org/jira/browse/STORM-310 I can make a new request. --- If your project is set up for it, you can reply to this

[GitHub] incubator-storm pull request: STORM-297 Storm Performance cannot b...

2014-05-15 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/103#discussion_r12362591 --- Diff: storm-core/src/jvm/backtype/storm/messaging/netty/Context.java --- @@ -47,12 +48,14 @@ public void prepare(Map storm_conf) {

[jira] [Commented] (STORM-310) Change Twitter authentication stream using the new Twitter4J. An optional set of filters can be used to restricted tweets.

2014-05-15 Thread Prabeesh K (JIRA)
[ https://issues.apache.org/jira/browse/STORM-310?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13997795#comment-13997795 ] Prabeesh K commented on STORM-310: -- Thank you Change Twitter authentication stream using

[GitHub] incubator-storm pull request: STORM-320: Support STORM_CONF_DIR en...

2014-05-15 Thread chu11
GitHub user chu11 opened a pull request: https://github.com/apache/incubator-storm/pull/114 STORM-320: Support STORM_CONF_DIR environment variable to support Support STORM_CONF_DIR environment variable to support alternate location of storm.yaml file. You can merge this pull

[GitHub] incubator-storm pull request: curator-1.3.3 - apache curator-2.4....

2014-05-15 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/incubator-storm/pull/113#issuecomment-43090639 Please place the JIRA number in the title of the pull request so the two are linked together. --- If your project is set up for it, you can reply to this email

[jira] [Commented] (STORM-280) storm unit tests are failing on windows

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-280?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13997637#comment-13997637 ] ASF GitHub Bot commented on STORM-280: -- Github user revans2 commented on the pull

[jira] [Commented] (STORM-317) Add SECURITY.md to release binaries

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13997668#comment-13997668 ] ASF GitHub Bot commented on STORM-317: -- Github user asfgit closed the pull request at:

[GitHub] incubator-storm pull request: [STORM-233] Removed inline heartbeat...

2014-05-15 Thread nathanmarz
Github user nathanmarz commented on the pull request: https://github.com/apache/incubator-storm/pull/98#issuecomment-43145410 Why does this avoid being killed when under heavy ZK load? --- If your project is set up for it, you can reply to this email and have your reply appear on

[jira] [Commented] (STORM-298) Logback file does not include full path for metrics appender fileNamePattern

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13997628#comment-13997628 ] ASF GitHub Bot commented on STORM-298: -- Github user asfgit closed the pull request at:

[GitHub] incubator-storm pull request: STORM-63 remove timeout drpc request...

2014-05-15 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/incubator-storm/pull/33#issuecomment-43140198 @xiaokang, Looks good. I wrote a quick unit test for this, and sent a [pull request](https://github.com/xiaokang/incubator-storm/pull/1) to you. Could you take a

[GitHub] incubator-storm pull request: Added progress bar when submitting t...

2014-05-15 Thread darthbear
Github user darthbear commented on the pull request: https://github.com/apache/incubator-storm/pull/100#issuecomment-42563750 Thank you. I'll do that. I also updated the code a little bit (conditions in submitJar() to be exactly the same as what it was before). Thanks. --- If your

[GitHub] incubator-storm pull request: [STORM-138]: Pluggable serialization...

2014-05-15 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/84#discussion_r12665268 --- Diff: storm-core/src/jvm/backtype/storm/spout/ShellSpout.java --- @@ -35,91 +35,84 @@ private SpoutOutputCollector _collector;

[GitHub] incubator-storm pull request: STORM-279. cluster.xml doesn't take ...

2014-05-15 Thread harshach
GitHub user harshach opened a pull request: https://github.com/apache/incubator-storm/pull/108 STORM-279. cluster.xml doesn't take in STORM_LOG_DIR values. changed from using env var to read from conf file user can add storm.log.dir to storm.yaml to specify a different location

[jira] [Commented] (STORM-305) Create developer documentation

2014-05-15 Thread Michael Noll (JIRA)
[ https://issues.apache.org/jira/browse/STORM-305?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13997959#comment-13997959 ] Michael Noll commented on STORM-305: Thanks for merging, Bobby. I closed the old pull

[GitHub] incubator-storm pull request: STORM-295. Add storm configuration t...

2014-05-15 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/incubator-storm/pull/87#issuecomment-42443868 @revans2 this pull request now launches workers with java from JAVA_HOME if set. It uses a slightly different approach than #97 and updates the unit tests. ---

[jira] [Resolved] (STORM-310) Change Twitter authentication stream using the new Twitter4J. An optional set of filters can be used to restricted tweets.

2014-05-15 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-310?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans resolved STORM-310. --- Resolution: Fixed Fix Version/s: 0.9.2-incubating Thanks Prabeesh, I pulled

[jira] [Updated] (STORM-297) Storm Performance cannot be scaled up by adding more CPU cores

2014-05-15 Thread Sean Zhong (JIRA)
[ https://issues.apache.org/jira/browse/STORM-297?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sean Zhong updated STORM-297: - Attachment: storm_Netty_receiver_diagram.png show how the message order is reserved when using multiple

[jira] [Commented] (STORM-310) Change Twitter authentication stream using the new Twitter4J. An optional set of filters can be used to restricted tweets.

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-310?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13996500#comment-13996500 ] ASF GitHub Bot commented on STORM-310: -- Github user d2r commented on the pull request:

[GitHub] incubator-storm pull request: [STORM-234] Thrift purpose conf to l...

2014-05-15 Thread revans2
GitHub user revans2 opened a pull request: https://github.com/apache/incubator-storm/pull/109 [STORM-234] Thrift purpose conf to let DRPC support authentication You can merge this pull request into a Git repository by running: $ git pull

Re: [jira] [Created] (STORM-319) mvn build storm failed for that ../../multilang/rb/storm.rb have syntax error.

2014-05-15 Thread Sean Zhong
Are you using Windows OS? If so, please try to use git to download fresh code on Linux like system. This is expected on windows, because storm.rb is a symbol link, it will be wrongly treated as a file on windows. Sean On Thu, May 15, 2014 at 12:25 AM, MoonGod (JIRA) j...@apache.org wrote:

[GitHub] incubator-storm pull request: Sliding window counter wipe zero cou...

2014-05-15 Thread Gvain
Github user Gvain closed the pull request at: https://github.com/apache/incubator-storm/pull/105 --- 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

[jira] [Updated] (STORM-315) Added progress bar when submitting topology

2014-05-15 Thread Francois Dang Ngoc (JIRA)
[ https://issues.apache.org/jira/browse/STORM-315?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Francois Dang Ngoc updated STORM-315: - Priority: Trivial (was: Major) Added progress bar when submitting topology

[GitHub] incubator-storm pull request: STORM-305: Create developer document...

2014-05-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-storm/pull/112 --- 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

[GitHub] incubator-storm pull request: STORM-297 Storm Performance cannot b...

2014-05-15 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/103#discussion_r12362656 --- Diff: storm-core/src/jvm/backtype/storm/messaging/netty/Server.java --- @@ -31,35 +31,65 @@ import org.slf4j.LoggerFactory;

[jira] [Commented] (STORM-264) Remove deprecated topology.optimize config?

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13998133#comment-13998133 ] ASF GitHub Bot commented on STORM-264: -- Github user nathanmarz commented on the pull

[GitHub] incubator-storm pull request: STORM-297 Storm Performance cannot b...

2014-05-15 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/103#discussion_r12362464 --- Diff: storm-core/src/clj/backtype/storm/daemon/executor.clj --- @@ -204,6 +204,7 @@ storm-conf (normalized-component-conf

[GitHub] incubator-storm pull request: Sliding window counter wipe zero cou...

2014-05-15 Thread Gvain
GitHub user Gvain opened a pull request: https://github.com/apache/incubator-storm/pull/105 Sliding window counter wipe zero counts Do not emit zero counts and computeTotalCount() only once per tick in SlidingWindowCounter You can merge this pull request into a Git repository by

About Tick Tuples

2014-05-15 Thread Srinath C
Hi, I'm seeing a lot of tick tuples emitted when I enable debug on my topology. None of my spouts/bolts have enabled it. Is this normal? Are tick tuples being used internally by storm for something? Also, I'm not able to relate the number of tick tuples to either my spout/bolt counts.

[GitHub] incubator-storm pull request: [STORM-310] Change Twitter authentic...

2014-05-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-storm/pull/62 --- 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

Re: Regarding: NullPointerException from deep inside Storm

2014-05-15 Thread Srinath C
Thanks for the reply Bobby. What are the symptoms that may arise as a result of this? On Wed, May 14, 2014 at 8:20 PM, Bobby Evans ev...@yahoo-inc.com wrote: The output collector is not thread safe. As such some odd things can happen if it is called from multiple threads. The reality is

[jira] [Commented] (STORM-310) Change Twitter authentication stream using the new Twitter4J. An optional set of filters can be used to restricted tweets.

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-310?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13997658#comment-13997658 ] ASF GitHub Bot commented on STORM-310: -- Github user asfgit closed the pull request at:

[jira] [Resolved] (STORM-317) Add SECURITY.md to release binaries

2014-05-15 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-317?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans resolved STORM-317. --- Resolution: Fixed Fix Version/s: 0.9.2-incubating Thanks Michael, I pull this

[jira] [Commented] (STORM-138) Pluggable serialization for multilang

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13998137#comment-13998137 ] ASF GitHub Bot commented on STORM-138: -- Github user d2r commented on a diff in the

[jira] [Commented] (STORM-138) Pluggable serialization for multilang

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13998136#comment-13998136 ] ASF GitHub Bot commented on STORM-138: -- Github user d2r commented on a diff in the

[GitHub] incubator-storm pull request: [STORM-138]: Pluggable serialization...

2014-05-15 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/84#discussion_r12665267 --- Diff: storm-core/src/jvm/backtype/storm/spout/ShellSpout.java --- @@ -35,91 +35,84 @@ private SpoutOutputCollector _collector;

[jira] [Created] (STORM-320) Support STORM_CONF_DIR environment variable for alternate conf dir

2014-05-15 Thread Albert Chu (JIRA)
Albert Chu created STORM-320: Summary: Support STORM_CONF_DIR environment variable for alternate conf dir Key: STORM-320 URL: https://issues.apache.org/jira/browse/STORM-320 Project: Apache Storm

[GitHub] incubator-storm pull request: [STORM-138]: Pluggable serialization...

2014-05-15 Thread nathanmarz
Github user nathanmarz commented on the pull request: https://github.com/apache/incubator-storm/pull/84#issuecomment-43145721 +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

[GitHub] incubator-storm pull request: [STORM-138]: Pluggable serialization...

2014-05-15 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/84#discussion_r12665246 --- Diff: storm-core/src/jvm/backtype/storm/multilang/JsonSerializer.java --- @@ -0,0 +1,164 @@ +package backtype.storm.multilang; + +import

[jira] [Commented] (STORM-138) Pluggable serialization for multilang

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13998140#comment-13998140 ] ASF GitHub Bot commented on STORM-138: -- Github user d2r commented on a diff in the

[jira] [Commented] (STORM-138) Pluggable serialization for multilang

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13998139#comment-13998139 ] ASF GitHub Bot commented on STORM-138: -- Github user d2r commented on a diff in the

[GitHub] incubator-storm pull request: STORM-305: Create developer document...

2014-05-15 Thread miguno
Github user miguno closed the pull request at: https://github.com/apache/incubator-storm/pull/104 --- 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

[GitHub] incubator-storm pull request: [STORM-138]: Pluggable serialization...

2014-05-15 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/84#discussion_r12665255 --- Diff: storm-core/src/jvm/backtype/storm/spout/ShellSpout.java --- @@ -35,91 +35,84 @@ private SpoutOutputCollector _collector;

Re: [jira] [Created] (STORM-319) mvn build storm failed for that ../../multilang/rb/storm.rb have syntax error.

2014-05-15 Thread Sean Zhong
To verify this, please check whether file storm-core/src/dev/resources/storm.rb is a symbol link or not. On Thu, May 15, 2014 at 8:59 AM, Sean Zhong clock...@gmail.com wrote: Are you using Windows OS? If so, please try to use git to download fresh code on Linux like system. This is expected

[jira] [Commented] (STORM-320) Support STORM_CONF_DIR environment variable for alternate conf dir

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-320?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13997832#comment-13997832 ] ASF GitHub Bot commented on STORM-320: -- GitHub user chu11 opened a pull request:

[GitHub] incubator-storm pull request: STORM-295. Add storm configuration t...

2014-05-15 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/incubator-storm/pull/87#issuecomment-42445852 +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

Re: Peeking into storm's internal buffers

2014-05-15 Thread Srinath C
Hi Sean, Could you figure out anything from these thread dumps? I analysed worker heap dumps and found a lot of __tick tuples getting accumulated over a period of time. But thats how far I got. I wasn't able to figure out to which bolt they was intended to and why they are not getting

[GitHub] incubator-storm pull request: added validation to registermetrics ...

2014-05-15 Thread darthbear
GitHub user darthbear opened a pull request: https://github.com/apache/incubator-storm/pull/110 added validation to registermetrics to have timebucketSizeInSecs = 1 added validation so one cannot register a metric with timeBucketSizeInSecs = 0 (that crashes the topology) You can

[jira] [Commented] (STORM-280) storm unit tests are failing on windows

2014-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-280?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13997642#comment-13997642 ] ASF GitHub Bot commented on STORM-280: -- Github user asfgit closed the pull request at:

[GitHub] incubator-storm pull request: STORM-297 Storm Performance cannot b...

2014-05-15 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/103#discussion_r12410447 --- Diff: storm-core/src/jvm/backtype/storm/messaging/netty/Client.java --- @@ -84,43 +93,87 @@ // Start the connection attempt.