[GitHub] storm pull request: STORM-558 change swap! to reset! to fix as...

2014-11-17 Thread xiaokang
GitHub user xiaokang opened a pull request: https://github.com/apache/storm/pull/315 STORM-558 change swap! to reset! to fix assignment-versions always empty in supervisor This PR is for JIRA STORM-558 to fix #167 . You can merge this pull request into a Git repository by running:

[jira] [Commented] (STORM-558) supervisor's assignment-versions memory cache for ZK is always empty

2014-11-17 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14214413#comment-14214413 ] ASF GitHub Bot commented on STORM-558: -- GitHub user xiaokang opened a pull request:

[GitHub] storm pull request: [STORM-375] Smarter downloading of assignments...

2014-11-17 Thread xiaokang
Github user xiaokang commented on the pull request: https://github.com/apache/storm/pull/167#issuecomment-63273071 Thanks @nathanmarz for verification. Another PR #315 is opened. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[jira] [Commented] (STORM-375) Smarter downloading of assignments by supervisors and workers

2014-11-17 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14214414#comment-14214414 ] ASF GitHub Bot commented on STORM-375: -- Github user xiaokang commented on the pull

[GitHub] storm pull request: STORM-558 change swap! to reset! to fix as...

2014-11-17 Thread xiaokang
Github user xiaokang commented on the pull request: https://github.com/apache/storm/pull/315#issuecomment-63273340 The unit test is failed BOTH before and after this code change. I'll try to find the problem. --- If your project is set up for it, you can reply to this email and have

[GitHub] storm pull request: STORM-558 change swap! to reset! to fix as...

2014-11-17 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/315#issuecomment-63318696 +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 and

[GitHub] storm pull request: STORM-558 change swap! to reset! to fix as...

2014-11-17 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/315#issuecomment-63320364 +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 and

[jira] [Commented] (STORM-558) supervisor's assignment-versions memory cache for ZK is always empty

2014-11-17 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14214729#comment-14214729 ] ASF GitHub Bot commented on STORM-558: -- Github user harshach commented on the pull

Re: [DISCUSS] Release Apache Storm 0.9.3

2014-11-17 Thread Bobby Evans
Even if we cannot reproduce it in all cases, the change was rather small not totally necessary.  +1 for reverting.  - Bobby On Sunday, November 16, 2014 2:23 PM, Harsha st...@harsha.io wrote: I am able to reproduce this using Sean's topology. I am +1 on reverting this. Thanks,

[GitHub] storm pull request: [STORM-557] Created docs directory and added i...

2014-11-17 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/314#issuecomment-63332256 I was curious if we wanted to keep a 5MB file in our github repo that everyone must download to get storm? As it stands now the git repo is about 11MB total. This could

[jira] [Commented] (STORM-557) High Quality Images for presentations, etc.

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

[GitHub] storm pull request: [STORM-557] Created docs directory and added i...

2014-11-17 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/314#issuecomment-63336734 @revans2 There is another JIRA asking for the current storm.apache.org docs to be moved onto github. https://issues.apache.org/jira/browse/STORM-544. As

[jira] [Commented] (STORM-557) High Quality Images for presentations, etc.

2014-11-17 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-557?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14214838#comment-14214838 ] ASF GitHub Bot commented on STORM-557: -- Github user harshach commented on the pull

[GitHub] storm pull request: [STORM-557] Created docs directory and added i...

2014-11-17 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/314#issuecomment-63337668 OK so the next question I have is, should we split the one big SVG file up along side rasterized PNG equivalents for use in the docs? --- If your project is set up for

[jira] [Commented] (STORM-557) High Quality Images for presentations, etc.

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

[GitHub] storm pull request: [STORM-557] Created docs directory and added i...

2014-11-17 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/314#issuecomment-63338507 @revans2 sounds like a good idea to provide PNG files. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[jira] [Commented] (STORM-558) supervisor's assignment-versions memory cache for ZK is always empty

2014-11-17 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14214898#comment-14214898 ] ASF GitHub Bot commented on STORM-558: -- Github user Parth-Brahmbhatt commented on the

[jira] [Commented] (STORM-512) KafkaBolt doesn't handle ticks properly

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

[jira] [Commented] (STORM-558) supervisor's assignment-versions memory cache for ZK is always empty

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

[jira] [Commented] (STORM-512) KafkaBolt doesn't handle ticks properly

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

acking and offsets

2014-11-17 Thread clay teahouse
Hello All, I am using kafka spout that comes with storm 0.9.3 ( https://github.com/apache/storm). I am having several different bolts consuming the same tuples from the spout (in the same topology). These bolts process the tuples and send the output to different destinations. I have a couple of

[GitHub] storm pull request: [STORM-557] Created docs directory and added i...

2014-11-17 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/314#issuecomment-63354593 OK I split them apart, cleaned up one of the SVG images to make it smaller, and added in some PNG versions, along with updating the security document to use one of the

[jira] [Commented] (STORM-512) KafkaBolt doesn't handle ticks properly

2014-11-17 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14214986#comment-14214986 ] ASF GitHub Bot commented on STORM-512: -- Github user nielsbasjes commented on the pull

[jira] [Commented] (STORM-512) KafkaBolt doesn't handle ticks properly

2014-11-17 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14215010#comment-14215010 ] ASF GitHub Bot commented on STORM-512: -- Github user nielsbasjes commented on the pull

[GitHub] storm pull request: Storm555: Storm json response should set chars...

2014-11-17 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/313#issuecomment-63367628 Thanks @Parth-Brahmbhatt. I merged this into the master and 0.9.3 branches. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] storm pull request: STORM-558 change swap! to reset! to fix as...

2014-11-17 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/315#issuecomment-63373190 Thanks @xiaokang. I merged this into the master and 0.9.3 branches. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[jira] [Resolved] (STORM-558) supervisor's assignment-versions memory cache for ZK is always empty

2014-11-17 Thread P. Taylor Goetz (JIRA)
[ https://issues.apache.org/jira/browse/STORM-558?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] P. Taylor Goetz resolved STORM-558. --- Resolution: Fixed Fix Version/s: 0.9.3 supervisor's assignment-versions memory cache

[GitHub] storm pull request: STORM-558 change swap! to reset! to fix as...

2014-11-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/315 --- 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 is

Re: [DISCUSS] Release Apache Storm 0.9.3

2014-11-17 Thread P. Taylor Goetz
Okay, I’ve reverted the commit in the 0.9.3 branch. I did not revert it in master. For master we can either choose to revert it there as well, or leave it and continue to look for a root cause. I’m open to either and would like to hear what other people think. In addition to the revert of

[jira] [Commented] (STORM-558) supervisor's assignment-versions memory cache for ZK is always empty

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

Re: [DISCUSS] Release Apache Storm 0.9.3

2014-11-17 Thread P. Taylor Goetz
Thanks for the input Bobby and Nathan. I’ll revert it there as well unless anyone proposes a compelling reason not to. What are your thoughts on moving forward with the 0.9.3 release? -Taylor On Nov 17, 2014, at 4:45 PM, Nathan Marz nat...@nathanmarz.com wrote: I agree, it should be reverted

[jira] [Updated] (STORM-350) Update disruptor to latest version (3.2.1)

2014-11-17 Thread Sean Zhong (JIRA)
) -- Key: STORM-350 URL: https://issues.apache.org/jira/browse/STORM-350 Project: Apache Storm Issue Type: Dependency upgrade Reporter: Boris Aksenov Fix For: 0.10.0 Attachments: 20141117-0.9.3-rc1-3-worker

[jira] [Updated] (STORM-350) Update disruptor to latest version (3.2.1)

2014-11-17 Thread Sean Zhong (JIRA)
) -- Key: STORM-350 URL: https://issues.apache.org/jira/browse/STORM-350 Project: Apache Storm Issue Type: Dependency upgrade Reporter: Boris Aksenov Fix For: 0.10.0 Attachments: 20141117-0.9.3

[jira] [Updated] (STORM-350) Update disruptor to latest version (3.2.1)

2014-11-17 Thread Sean Zhong (JIRA)
Attachments: 20141117-0.9.3-rc1-3-worker-separate-1-spout-and-2-bolts-failing-tuples.png, 20141117-0.9.3-rc1-one-worker-failing-tuples.png, 20141117-0.9.3-rc1-three-workers-1-spout-3-bolts-failing-tuples.png, 20141118-0.9.3-branch-3-worker-separate-1-spout-and-2-bolts-ok.png, 20141118

[jira] [Reopened] (STORM-350) Update disruptor to latest version (3.2.1)

2014-11-17 Thread Sean Zhong (JIRA)
) -- Key: STORM-350 URL: https://issues.apache.org/jira/browse/STORM-350 Project: Apache Storm Issue Type: Dependency upgrade Reporter: Boris Aksenov Fix For: 0.9.3 Attachments: 20141117

[jira] [Updated] (STORM-350) Update disruptor to latest version (3.2.1)

2014-11-17 Thread Sean Zhong (JIRA)
Attachments: 20141117-0.9.3-rc1-3-worker-separate-1-spout-and-2-bolts-failing-tuples.png, 20141117-0.9.3-rc1-one-worker-failing-tuples.png, 20141117-0.9.3-rc1-three-workers-1-spout-3-bolts-failing-tuples.png, 20141118-0.9.3-branch-3-worker-separate-1-spout-and-2-bolts-ok.png, 20141118-0.9.3

[GitHub] storm pull request: STORM-329 : buffer message in client and recon...

2014-11-17 Thread tedxia
Github user tedxia commented on the pull request: https://github.com/apache/storm/pull/268#issuecomment-63424599 @ptgoetz I have merge storm/master to this patch and all tests pass except storm-core/test/clj/backtype/storm/multilang_test.clj, test results are(after I removed

[jira] [Commented] (STORM-329) Add Option to Config Message handling strategy when connection timeout

2014-11-17 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14215731#comment-14215731 ] ASF GitHub Bot commented on STORM-329: -- Github user tedxia commented on the pull

[GitHub] storm pull request: STORM-329 : buffer message in client and recon...

2014-11-17 Thread tedxia
Github user tedxia commented on the pull request: https://github.com/apache/storm/pull/268#issuecomment-63430061 ``` I tested this by applying it to the 0.9.3 branch and found problems with the unit tests (never-ending cycle of zookeeper reconnects, tests never complete). ```

[GitHub] storm pull request: STORM-188: Allow user to specifiy full configu...

2014-11-17 Thread clockfly
Github user clockfly commented on the pull request: https://github.com/apache/storm/pull/120#issuecomment-63431309 The original behavior of findAndReadConfigFile() is to locate config file on classpath. findResources(name) will not be empty when name exists on classpath.