[jira] [Commented] (STORM-1483) add storm-mongodb connector

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196896#comment-15196896 ] ASF GitHub Bot commented on STORM-1483: --- Github user vesense commented on the pull r

[GitHub] storm pull request: [STORM-1483] add storm-mongodb connector

2016-03-15 Thread vesense
Github user vesense commented on the pull request: https://github.com/apache/storm/pull/1038#issuecomment-197184879 @harshach Done. -> https://github.com/apache/storm/pull/1219 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[jira] [Commented] (STORM-1483) add storm-mongodb connector

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196895#comment-15196895 ] ASF GitHub Bot commented on STORM-1483: --- GitHub user vesense opened a pull request:

[GitHub] storm pull request: STORM-1483: add storm-mongodb connector

2016-03-15 Thread vesense
GitHub user vesense opened a pull request: https://github.com/apache/storm/pull/1219 STORM-1483: add storm-mongodb connector for 1.x-branch You can merge this pull request into a Git repository by running: $ git pull https://github.com/vesense/storm STORM-1483-1.x Alternativel

[jira] [Commented] (STORM-676) Storm Trident support for sliding/tumbling windows

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-676?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196873#comment-15196873 ] ASF GitHub Bot commented on STORM-676: -- Github user arunmahadevan commented on the pul

[GitHub] storm pull request: STORM-676 Trident implementation for sliding a...

2016-03-15 Thread arunmahadevan
Github user arunmahadevan commented on the pull request: https://github.com/apache/storm/pull/1072#issuecomment-197175210 @harshach I haven't reviewed the code yet, plan to do that within a couple of days. --- If your project is set up for it, you can reply to this email and have you

[jira] [Commented] (STORM-1268) port backtype.storm.daemon.builtin-metrics to java

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196835#comment-15196835 ] ASF GitHub Bot commented on STORM-1268: --- GitHub user unsleepy22 opened a pull reques

[GitHub] storm pull request: [STORM-1268] port builtin-metrics to java

2016-03-15 Thread unsleepy22
GitHub user unsleepy22 opened a pull request: https://github.com/apache/storm/pull/1218 [STORM-1268] port builtin-metrics to java You can merge this pull request into a Git repository by running: $ git pull https://github.com/unsleepy22/storm STORM-1268 Alternatively you can

[jira] [Commented] (STORM-1232) port backtype.storm.scheduler.DefaultScheduler to java

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196830#comment-15196830 ] ASF GitHub Bot commented on STORM-1232: --- Github user vesense commented on the pull r

[GitHub] storm pull request: [STORM-1232] port backtype.storm.scheduler.Def...

2016-03-15 Thread vesense
Github user vesense commented on the pull request: https://github.com/apache/storm/pull/1108#issuecomment-197168827 @revans2 Thanks for your review. I think I have addressed all of your comments. --- If your project is set up for it, you can reply to this email and have your reply ap

[jira] [Commented] (STORM-1232) port backtype.storm.scheduler.DefaultScheduler to java

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196825#comment-15196825 ] ASF GitHub Bot commented on STORM-1232: --- Github user vesense commented on a diff in

[GitHub] storm pull request: [STORM-1232] port backtype.storm.scheduler.Def...

2016-03-15 Thread vesense
Github user vesense commented on a diff in the pull request: https://github.com/apache/storm/pull/1108#discussion_r56285598 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/EvenScheduler.java --- @@ -0,0 +1,168 @@ +/** + * Licensed to the Apache Software Foundation (

[GitHub] storm pull request: [STORM-1232] port backtype.storm.scheduler.Def...

2016-03-15 Thread vesense
Github user vesense commented on a diff in the pull request: https://github.com/apache/storm/pull/1108#discussion_r56285608 --- Diff: storm-core/test/clj/org/apache/storm/scheduler_test.clj --- @@ -262,20 +262,15 @@ (deftest test-sort-slots ;; test supervisor2 has

[jira] [Commented] (STORM-1232) port backtype.storm.scheduler.DefaultScheduler to java

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196824#comment-15196824 ] ASF GitHub Bot commented on STORM-1232: --- Github user vesense commented on a diff in

[jira] [Commented] (STORM-1232) port backtype.storm.scheduler.DefaultScheduler to java

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196826#comment-15196826 ] ASF GitHub Bot commented on STORM-1232: --- Github user vesense commented on a diff in

[jira] [Commented] (STORM-1232) port backtype.storm.scheduler.DefaultScheduler to java

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196823#comment-15196823 ] ASF GitHub Bot commented on STORM-1232: --- Github user vesense commented on a diff in

[GitHub] storm pull request: [STORM-1232] port backtype.storm.scheduler.Def...

2016-03-15 Thread vesense
Github user vesense commented on a diff in the pull request: https://github.com/apache/storm/pull/1108#discussion_r56285602 --- Diff: storm-core/test/clj/org/apache/storm/scheduler_test.clj --- @@ -262,20 +262,15 @@ (deftest test-sort-slots ;; test supervisor2 has

[jira] [Commented] (STORM-1630) "create symbolic link" needs elevation or setting privilege about creating symbolic link

2016-03-15 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1630?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196822#comment-15196822 ] Jungtaek Lim commented on STORM-1630: - I found related doc from Hadoop. https://github

[GitHub] storm pull request: [STORM-1232] port backtype.storm.scheduler.Def...

2016-03-15 Thread vesense
Github user vesense commented on a diff in the pull request: https://github.com/apache/storm/pull/1108#discussion_r56285568 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/DefaultScheduler.java --- @@ -0,0 +1,111 @@ +/** + * Licensed to the Apache Software Foundatio

[jira] [Commented] (STORM-1602) Blobstore UTs are failed on Windows

2016-03-15 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1602?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196794#comment-15196794 ] Jungtaek Lim commented on STORM-1602: - I found other reason which makes BlobStore UTs

[jira] [Commented] (STORM-1279) port backtype.storm.daemon.supervisor to java

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1279?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196778#comment-15196778 ] ASF GitHub Bot commented on STORM-1279: --- Github user hustfxj commented on the pull r

[GitHub] storm pull request: [STORM-1279] port backtype.storm.daemon.superv...

2016-03-15 Thread hustfxj
Github user hustfxj commented on the pull request: https://github.com/apache/storm/pull/1184#issuecomment-197155837 @jerrypeng Of course --- 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

[jira] [Commented] (STORM-956) When the execute() or nextTuple() hang on external resources, stop the Worker's heartbeat

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196773#comment-15196773 ] ASF GitHub Bot commented on STORM-956: -- Github user hustfxj commented on the pull requ

[GitHub] storm pull request: STORM-956: When the execute() or nextTuple() h...

2016-03-15 Thread hustfxj
Github user hustfxj commented on the pull request: https://github.com/apache/storm/pull/1209#issuecomment-197155218 @srdo I mean that we can report errors into Zookeeper whether the option is enabled. And the metrisc is not only record the counter, but also the timeout time. --- I

[jira] [Commented] (STORM-1629) Files/move doesn't work properly with non-empty directory in Windows

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196757#comment-15196757 ] ASF GitHub Bot commented on STORM-1629: --- Github user hustfxj commented on the pull r

[GitHub] storm pull request: STORM-1629 (For 1.x) Files/move doesn't work p...

2016-03-15 Thread hustfxj
Github user hustfxj commented on the pull request: https://github.com/apache/storm/pull/1215#issuecomment-197152524 +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] [Updated] (STORM-1631) torm CGroup bugs 1) when launching workers as the user that submitted the topology 2) when initial cleanup of cgroup fails

2016-03-15 Thread Boyang Jerry Peng (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1631?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Boyang Jerry Peng updated STORM-1631: - Description: In secure multitenant storm, topology workers are launched with permission of

[jira] [Updated] (STORM-1631) torm CGroup bugs 1) when launching workers as the user that submitted the topology 2) when initial cleanup of cgroup fails

2016-03-15 Thread Boyang Jerry Peng (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1631?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Boyang Jerry Peng updated STORM-1631: - Summary: torm CGroup bugs 1) when launching workers as the user that submitted the topolog

[jira] [Updated] (STORM-1632) Disable event logging by default

2016-03-15 Thread Roshan Naik (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1632?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Roshan Naik updated STORM-1632: --- Description: EventLogging has performance penalty. For a simple speed of light topology with a singl

[jira] [Updated] (STORM-1632) Disable event logging by default

2016-03-15 Thread Roshan Naik (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1632?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Roshan Naik updated STORM-1632: --- Priority: Blocker (was: Major) > Disable event logging by default >

[GitHub] storm pull request: STORM-1632 Disable event logging by default

2016-03-15 Thread roshannaik
GitHub user roshannaik opened a pull request: https://github.com/apache/storm/pull/1217 STORM-1632 Disable event logging by default updating setting in defaults.yaml You can merge this pull request into a Git repository by running: $ git pull https://github.com/roshannaik/sto

[jira] [Commented] (STORM-1632) Disable event logging by default

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196744#comment-15196744 ] ASF GitHub Bot commented on STORM-1632: --- GitHub user roshannaik opened a pull reques

[jira] [Created] (STORM-1632) Disable event logging by default

2016-03-15 Thread Roshan Naik (JIRA)
Roshan Naik created STORM-1632: -- Summary: Disable event logging by default Key: STORM-1632 URL: https://issues.apache.org/jira/browse/STORM-1632 Project: Apache Storm Issue Type: Bug C

[jira] [Commented] (STORM-1602) Blobstore UTs are failed on Windows

2016-03-15 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1602?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=1519#comment-1519 ] Jungtaek Lim commented on STORM-1602: - I guess we should modify localizer tests to not

[jira] [Commented] (STORM-1057) Add throughput metric to spout/bolt and display them on web ui

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196645#comment-15196645 ] ASF GitHub Bot commented on STORM-1057: --- Github user harshach commented on the pull

[GitHub] storm pull request: [STORM-1057] Add throughput metrics to spouts/...

2016-03-15 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/753#issuecomment-197118972 @revans2 @d2r @redsanket can you comment. thanks. --- 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

[jira] [Commented] (STORM-1057) Add throughput metric to spout/bolt and display them on web ui

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196639#comment-15196639 ] ASF GitHub Bot commented on STORM-1057: --- Github user wangli1426 commented on the pul

[GitHub] storm pull request: [STORM-1057] Add throughput metrics to spouts/...

2016-03-15 Thread wangli1426
Github user wangli1426 commented on the pull request: https://github.com/apache/storm/pull/753#issuecomment-197118115 @harshach I can up-merge this commit. But before I start, I would like to make sure that more committers want this PR get merged. I really spent a lot of time and eff

Re: [DISCUSS] Handling symbolic link to be compatible with Windows

2016-03-15 Thread Jungtaek Lim
Thanks Bobby for sharing opinion. I was curious that what other projects handle it, and if Hadoop on Windows also requires manual privilege, I'm also OK to just document how to set up. Then release note for 1.0.0 should mention this change. Maybe on-topic or off-topic, I'm also investigating Blob

[jira] [Commented] (STORM-1581) Repair github links in the storm documentation

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1581?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196596#comment-15196596 ] ASF GitHub Bot commented on STORM-1581: --- Github user dossett commented on the pull r

[GitHub] storm pull request: STORM-1581: Repair github links in storm docum...

2016-03-15 Thread dossett
Github user dossett commented on the pull request: https://github.com/apache/storm/pull/1157#issuecomment-197103542 +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-1624) add maven central status in README

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196594#comment-15196594 ] ASF GitHub Bot commented on STORM-1624: --- Github user dossett commented on the pull r

[GitHub] storm pull request: [STORM-1624] add maven central status in READM...

2016-03-15 Thread dossett
Github user dossett commented on the pull request: https://github.com/apache/storm/pull/1212#issuecomment-197102565 +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: Rename README.markdown to README.md.

2016-03-15 Thread dossett
Github user dossett commented on the pull request: https://github.com/apache/storm/pull/1213#issuecomment-197100612 Seems like a reasonable change. Perhaps leave a README.markdown with a pointer to README.md though? --- If your project is set up for it, you can reply to this email a

[GitHub] storm pull request: Update doc for rebalance command

2016-03-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1130 --- 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 enab

[GitHub] storm pull request: Add a version to snakeyaml dependency

2016-03-15 Thread dossett
Github user dossett closed the pull request at: https://github.com/apache/storm/pull/1126 --- 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 ena

[GitHub] storm pull request: Add a version to snakeyaml dependency

2016-03-15 Thread dossett
Github user dossett commented on the pull request: https://github.com/apache/storm/pull/1126#issuecomment-197095654 You are correct. --- 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 featu

[GitHub] storm pull request: fix storm-starter README.markdown links

2016-03-15 Thread dossett
Github user dossett commented on the pull request: https://github.com/apache/storm/pull/1154#issuecomment-197095427 +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: Fixed incorrect storm-kafka documentation.

2016-03-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1168 --- 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 enab

[GitHub] storm pull request: Fixed incorrect storm-kafka documentation.

2016-03-15 Thread dossett
Github user dossett commented on the pull request: https://github.com/apache/storm/pull/1168#issuecomment-197092133 +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-1464) storm-hdfs should support writing to multiple files

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196557#comment-15196557 ] ASF GitHub Bot commented on STORM-1464: --- Github user dossett commented on the pull r

[GitHub] storm pull request: STORM-1464: Support multiple file outputs

2016-03-15 Thread dossett
Github user dossett commented on the pull request: https://github.com/apache/storm/pull/1044#issuecomment-197087032 @arunmahadevan I have addressed several of your comments -- thanks for the review, it was helpful. Per my other comments, the key can't be based only on the partition in

[jira] [Commented] (STORM-1464) storm-hdfs should support writing to multiple files

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196542#comment-15196542 ] ASF GitHub Bot commented on STORM-1464: --- Github user dossett commented on a diff in

[GitHub] storm pull request: STORM-1464: Support multiple file outputs

2016-03-15 Thread dossett
Github user dossett commented on a diff in the pull request: https://github.com/apache/storm/pull/1044#discussion_r56264969 --- Diff: external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/AbstractHdfsBolt.java --- @@ -52,9 +54,10 @@ * Half of the default Config.TO

[jira] [Commented] (STORM-1057) Add throughput metric to spout/bolt and display them on web ui

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196515#comment-15196515 ] ASF GitHub Bot commented on STORM-1057: --- Github user harshach commented on the pull

[GitHub] storm pull request: [STORM-1057] Add throughput metrics to spouts/...

2016-03-15 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/753#issuecomment-197074316 @wangli1426 This is patch very important. I would like to see this get merged in for 1.x-branch. Can you please upmerge this PR. --- If your project is set up for it,

[jira] [Commented] (STORM-1623) nimbus.clj's minor bug

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196400#comment-15196400 ] ASF GitHub Bot commented on STORM-1623: --- Github user zhuoliu commented on the pull r

[GitHub] storm pull request: [STORM-1623] fix bug about nimbus.clj

2016-03-15 Thread zhuoliu
Github user zhuoliu commented on the pull request: https://github.com/apache/storm/pull/1211#issuecomment-197056859 Good catch. +1. Checked, it was introduced only to master. 1.x is OK. --- If your project is set up for it, you can reply to this email and have your reply appear o

[jira] [Commented] (STORM-1624) add maven central status in README

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196394#comment-15196394 ] ASF GitHub Bot commented on STORM-1624: --- Github user zhuoliu commented on the pull r

[GitHub] storm pull request: [STORM-1624] add maven central status in READM...

2016-03-15 Thread zhuoliu
Github user zhuoliu commented on the pull request: https://github.com/apache/storm/pull/1212#issuecomment-197055834 +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-1464) storm-hdfs should support writing to multiple files

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196371#comment-15196371 ] ASF GitHub Bot commented on STORM-1464: --- Github user rajkonda commented on a diff in

[GitHub] storm pull request: STORM-1464: Support multiple file outputs

2016-03-15 Thread rajkonda
Github user rajkonda commented on a diff in the pull request: https://github.com/apache/storm/pull/1044#discussion_r56251428 --- Diff: external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/AbstractHdfsBolt.java --- @@ -52,9 +54,10 @@ * Half of the default Config.T

[jira] [Commented] (STORM-1464) storm-hdfs should support writing to multiple files

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196354#comment-15196354 ] ASF GitHub Bot commented on STORM-1464: --- Github user dossett commented on a diff in

[GitHub] storm pull request: STORM-1464: Support multiple file outputs

2016-03-15 Thread dossett
Github user dossett commented on a diff in the pull request: https://github.com/apache/storm/pull/1044#discussion_r56249577 --- Diff: external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/AbstractHdfsBolt.java --- @@ -223,29 +259,57 @@ public final void execute(Tuple tuple)

[jira] [Commented] (STORM-1279) port backtype.storm.daemon.supervisor to java

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1279?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196341#comment-15196341 ] ASF GitHub Bot commented on STORM-1279: --- Github user jerrypeng commented on the pull

[GitHub] storm pull request: [STORM-1279] port backtype.storm.daemon.superv...

2016-03-15 Thread jerrypeng
Github user jerrypeng commented on the pull request: https://github.com/apache/storm/pull/1184#issuecomment-197042280 I have a bug fix for cgroups that involve a couple lines of modification in supervisor.clj. https://github.com/apache/storm/pull/1216 If that pull r

[jira] [Commented] (STORM-1464) storm-hdfs should support writing to multiple files

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196339#comment-15196339 ] ASF GitHub Bot commented on STORM-1464: --- Github user dossett commented on a diff in

[jira] [Commented] (STORM-1631) Storm CGroup bug when launching workers as the user that submitted the topology

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1631?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196337#comment-15196337 ] ASF GitHub Bot commented on STORM-1631: --- GitHub user jerrypeng opened a pull request

[GitHub] storm pull request: STORM-1464: Support multiple file outputs

2016-03-15 Thread dossett
Github user dossett commented on a diff in the pull request: https://github.com/apache/storm/pull/1044#discussion_r56247649 --- Diff: external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/AbstractHdfsBolt.java --- @@ -223,29 +259,57 @@ public final void execute(Tuple tuple)

[GitHub] storm pull request: [STORM-1631] - Storm CGroup bug when launching...

2016-03-15 Thread jerrypeng
GitHub user jerrypeng opened a pull request: https://github.com/apache/storm/pull/1216 [STORM-1631] - Storm CGroup bug when launching workers as the user that submitted the topology You can merge this pull request into a Git repository by running: $ git pull https://github.co

[jira] [Commented] (STORM-1464) storm-hdfs should support writing to multiple files

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196274#comment-15196274 ] ASF GitHub Bot commented on STORM-1464: --- Github user dossett commented on a diff in

[GitHub] storm pull request: STORM-1464: Support multiple file outputs

2016-03-15 Thread dossett
Github user dossett commented on a diff in the pull request: https://github.com/apache/storm/pull/1044#discussion_r56241745 --- Diff: external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/AbstractHdfsBolt.java --- @@ -198,22 +194,62 @@ public final void execute(Tuple tuple)

[jira] [Commented] (STORM-1464) storm-hdfs should support writing to multiple files

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196265#comment-15196265 ] ASF GitHub Bot commented on STORM-1464: --- Github user dossett commented on a diff in

[GitHub] storm pull request: STORM-1464: Support multiple file outputs

2016-03-15 Thread dossett
Github user dossett commented on a diff in the pull request: https://github.com/apache/storm/pull/1044#discussion_r56241176 --- Diff: external/storm-hdfs/README.md --- @@ -240,6 +240,23 @@ If you are using Trident and sequence files you can do something like this:

[jira] [Commented] (STORM-1614) Clean backpressure zk node in do-cleanup

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196263#comment-15196263 ] ASF GitHub Bot commented on STORM-1614: --- Github user zhuoliu commented on the pull r

[GitHub] storm pull request: STORM-1614: backpressure init and cleanup chan...

2016-03-15 Thread zhuoliu
Github user zhuoliu commented on the pull request: https://github.com/apache/storm/pull/1206#issuecomment-197019532 Looks good. +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] storm pull request: STORM-1614: backpressure init and cleanup chan...

2016-03-15 Thread zhuoliu
Github user zhuoliu commented on a diff in the pull request: https://github.com/apache/storm/pull/1206#discussion_r56240881 --- Diff: storm-core/src/clj/org/apache/storm/cluster.clj --- @@ -73,6 +73,7 @@ (teardown-topology-errors! [this storm-id]) (heartbeat-storms [th

[jira] [Commented] (STORM-1614) Clean backpressure zk node in do-cleanup

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196262#comment-15196262 ] ASF GitHub Bot commented on STORM-1614: --- Github user zhuoliu commented on a diff in

[jira] [Commented] (STORM-1464) storm-hdfs should support writing to multiple files

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196261#comment-15196261 ] ASF GitHub Bot commented on STORM-1464: --- Github user dossett commented on a diff in

[GitHub] storm pull request: STORM-1464: Support multiple file outputs

2016-03-15 Thread dossett
Github user dossett commented on a diff in the pull request: https://github.com/apache/storm/pull/1044#discussion_r56240831 --- Diff: external/storm-hdfs/README.md --- @@ -240,6 +240,23 @@ If you are using Trident and sequence files you can do something like this:

[GitHub] storm pull request: Update doc for rebalance command

2016-03-15 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/1130#issuecomment-197018229 +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-1464) storm-hdfs should support writing to multiple files

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196253#comment-15196253 ] ASF GitHub Bot commented on STORM-1464: --- Github user dossett commented on a diff in

[GitHub] storm pull request: STORM-1464: Support multiple file outputs

2016-03-15 Thread dossett
Github user dossett commented on a diff in the pull request: https://github.com/apache/storm/pull/1044#discussion_r56240076 --- Diff: external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/AbstractHdfsBolt.java --- @@ -52,9 +54,10 @@ * Half of the default Config.TO

[jira] [Commented] (STORM-1549) Add support for extending tuple tree timeout

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196246#comment-15196246 ] ASF GitHub Bot commented on STORM-1549: --- Github user srdo commented on the pull requ

[GitHub] storm pull request: STORM-1549: Add support for resetting tuple ti...

2016-03-15 Thread srdo
Github user srdo commented on the pull request: https://github.com/apache/storm/pull/1107#issuecomment-197016989 The test failure looks unrelated to this PR, was also happening on 1.x-branch https://travis-ci.org/apache/storm/jobs/115948898 --- If your project is set up for it, you c

[GitHub] storm pull request: STORM-822: Kafka Spout New Consumer API

2016-03-15 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1131#discussion_r56239392 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/spout/KafkaSpout.java --- @@ -0,0 +1,517 @@ +/* + * Licensed to the Apache

[jira] [Commented] (STORM-822) As a storm developer I’d like to use the new kafka consumer API (0.8.3) to reduce dependencies and use long term supported kafka apis

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-822?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196242#comment-15196242 ] ASF GitHub Bot commented on STORM-822: -- Github user revans2 commented on a diff in the

[GitHub] storm pull request: STORM-822: Kafka Spout New Consumer API

2016-03-15 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1131#discussion_r56239098 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/spout/KafkaSpout.java --- @@ -0,0 +1,517 @@ +/* + * Licensed to the Apache

[jira] [Commented] (STORM-822) As a storm developer I’d like to use the new kafka consumer API (0.8.3) to reduce dependencies and use long term supported kafka apis

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-822?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196241#comment-15196241 ] ASF GitHub Bot commented on STORM-822: -- Github user revans2 commented on a diff in the

[jira] [Commented] (STORM-822) As a storm developer I’d like to use the new kafka consumer API (0.8.3) to reduce dependencies and use long term supported kafka apis

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-822?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196240#comment-15196240 ] ASF GitHub Bot commented on STORM-822: -- Github user revans2 commented on a diff in the

[GitHub] storm pull request: STORM-822: Kafka Spout New Consumer API

2016-03-15 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1131#discussion_r56239019 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/spout/KafkaSpout.java --- @@ -0,0 +1,517 @@ +/* + * Licensed to the Apache

[jira] [Commented] (STORM-1614) Clean backpressure zk node in do-cleanup

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196230#comment-15196230 ] ASF GitHub Bot commented on STORM-1614: --- Github user ptgoetz commented on the pull r

[GitHub] storm pull request: STORM-1614: backpressure init and cleanup chan...

2016-03-15 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/1206#issuecomment-197014240 +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-822: Kafka Spout New Consumer API

2016-03-15 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1131#discussion_r56238105 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/spout/KafkaSpout.java --- @@ -0,0 +1,517 @@ +/* + * Licensed to the Apache

[jira] [Commented] (STORM-822) As a storm developer I’d like to use the new kafka consumer API (0.8.3) to reduce dependencies and use long term supported kafka apis

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-822?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196227#comment-15196227 ] ASF GitHub Bot commented on STORM-822: -- Github user revans2 commented on a diff in the

[jira] [Commented] (STORM-822) As a storm developer I’d like to use the new kafka consumer API (0.8.3) to reduce dependencies and use long term supported kafka apis

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-822?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196225#comment-15196225 ] ASF GitHub Bot commented on STORM-822: -- Github user revans2 commented on a diff in the

[GitHub] storm pull request: STORM-822: Kafka Spout New Consumer API

2016-03-15 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1131#discussion_r56237896 --- Diff: external/storm-kafka-client/src/main/java/org/apache/storm/kafka/spout/KafkaSpout.java --- @@ -0,0 +1,517 @@ +/* + * Licensed to the Apache

[jira] [Commented] (STORM-1615) Update state checkpointing doc with bolt's acking contract

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1615?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196222#comment-15196222 ] ASF GitHub Bot commented on STORM-1615: --- Github user ptgoetz commented on the pull r

[GitHub] storm pull request: [STORM-1615] Update state checkpointing doc wi...

2016-03-15 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/1210#issuecomment-197013148 +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-822) As a storm developer I’d like to use the new kafka consumer API (0.8.3) to reduce dependencies and use long term supported kafka apis

2016-03-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-822?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196219#comment-15196219 ] ASF GitHub Bot commented on STORM-822: -- Github user revans2 commented on a diff in the

  1   2   >