[GitHub] storm pull request: STORM-994 Close download clients and channels ...

2015-08-25 Thread fmazoyer
GitHub user fmazoyer opened a pull request: https://github.com/apache/storm/pull/697 STORM-994 Close download clients and channels to avoid resource leaks This pull request to fix STORM-994 to avoid resource leaks between nimbus and supervisors. Could you please review

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

2015-01-22 Thread fmazoyer
Github user fmazoyer commented on the pull request: https://github.com/apache/storm/pull/268#issuecomment-71063105 Hello, we recently stumbled upon the issue STORM-404 in storm 0.9.3. I was just curious if some more work was planned to fix that guy? Or just, could the work

[GitHub] storm pull request: STORM-248 STORM-322 are fixed and upmerged

2015-01-15 Thread fmazoyer
Github user fmazoyer commented on the pull request: https://github.com/apache/storm/pull/373#issuecomment-70099486 Hi Robert, do you think there are still some things to do on my side ? Thanks for everything :-) --- If your project is set up for it, you can reply

[GitHub] storm pull request: STORM-248 STORM-322 are fixed and upmerged

2015-01-07 Thread fmazoyer
Github user fmazoyer commented on the pull request: https://github.com/apache/storm/pull/373#issuecomment-69002802 Thank you for your comment, this is done :-) Could you please review ? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] storm pull request: STORM-248 STORM-322 proposal for fix

2015-01-06 Thread fmazoyer
Github user fmazoyer commented on the pull request: https://github.com/apache/storm/pull/353#issuecomment-68863466 Rebased and resolved conflicts from upstream/master. Unfortunately, it also took two files I did not intend to pull: changelog.md and SequenceFormat.java. Sorry

[GitHub] storm pull request: STORM-248 STORM-322 are fixed and upmerged

2015-01-06 Thread fmazoyer
GitHub user fmazoyer opened a pull request: https://github.com/apache/storm/pull/373 STORM-248 STORM-322 are fixed and upmerged Details are in JIRA https://issues.apache.org/jira/browse/STORM-248. As a collateral benefit, this also addresses JIRA https://issues.apache.org/jira

[GitHub] storm pull request: STORM-248 STORM-322 proposal for fix

2014-12-19 Thread fmazoyer
GitHub user fmazoyer opened a pull request: https://github.com/apache/storm/pull/353 STORM-248 STORM-322 proposal for fix Details are in JIRA https://issues.apache.org/jira/browse/STORM-248. As a collateral benefit, this also addresses JIRA https://issues.apache.org/jira/browse

[GitHub] storm pull request: Performance improvement: Fix for https://issue...

2014-12-09 Thread fmazoyer
GitHub user fmazoyer opened a pull request: https://github.com/apache/storm/pull/343 Performance improvement: Fix for https://issues.apache.org/jira/browse/STORM-585 Hello, this fix helped us gain a sizable amount of performances in none:grouping. Details are in JIRA https

[GitHub] storm pull request: Performance improvement: Fix for https://issue...

2014-12-09 Thread fmazoyer
Github user fmazoyer commented on the pull request: https://github.com/apache/storm/pull/343#issuecomment-66322250 Could you help me with a simple way to do that, please? Thanks for your help :-) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] storm pull request: Storm 585

2014-12-08 Thread fmazoyer
Github user fmazoyer closed the pull request at: https://github.com/apache/storm/pull/342 --- 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