[GitHub] storm pull request: Fix docs incorrect number of stream groupings

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/636#issuecomment-121877328 Thanks @vesense! --- 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 fe

[GitHub] storm pull request: Fix docs incorrect number of stream groupings

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/636 --- 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 enabl

[jira] [Commented] (STORM-837) HdfsState ignores commits

2015-07-16 Thread Arun Mahadevan (JIRA)
[ https://issues.apache.org/jira/browse/STORM-837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629736#comment-14629736 ] Arun Mahadevan commented on STORM-837: -- Agree with the approach of keeping data and tx

[jira] [Commented] (STORM-837) HdfsState ignores commits

2015-07-16 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629747#comment-14629747 ] Robert Joseph Evans commented on STORM-837: --- I agree that we need to overwrite, b

[GitHub] storm pull request: Fix links to documentation in Javadoc

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/626#issuecomment-121963469 @alexshadow007 thanks for the fix. I pulled this into master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] storm pull request: Fix links to documentation in Javadoc

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/626 --- 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 enabl

[GitHub] storm pull request: Fix a few links in the documentation

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/622#issuecomment-121964485 +1 looks good to me. --- 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 fe

[GitHub] storm pull request: Fix a few links in the documentation

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/622#issuecomment-121964658 Thanks @thilinarmtb I merged this into master. --- 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 pro

[GitHub] storm pull request: Fix a few links in the documentation

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/622 --- 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 enabl

[GitHub] storm pull request: Fix links to documentation in Javadoc

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/626#issuecomment-121963708 I forgot to formally say +1 so here it is. --- 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

[GitHub] storm pull request: STORM-924:Set the file mode of the files inclu...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/614#issuecomment-121966256 Hmm, when I build it on my mac everything shows up fine, either with or without this fix. +1 --- If your project is set up for it, you can reply to this email and have y

[jira] [Commented] (STORM-924) Set the file mode of the files included when packaging release packages

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629772#comment-14629772 ] ASF GitHub Bot commented on STORM-924: -- Github user revans2 commented on the pull requ

[GitHub] storm pull request: STORM-924:Set the file mode of the files inclu...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/614 --- 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 enabl

[jira] [Commented] (STORM-924) Set the file mode of the files included when packaging release packages

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629774#comment-14629774 ] ASF GitHub Bot commented on STORM-924: -- Github user asfgit closed the pull request at:

[GitHub] storm pull request: STORM-924:Set the file mode of the files inclu...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/614#issuecomment-121967209 @HeartSaVioR I'm not sure exactly what is happening here either but the patch does not hurt anything. If you want to dig more into exactly what is happening we can file

[jira] [Resolved] (STORM-924) Set the file mode of the files included when packaging release packages

2015-07-16 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-924?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans resolved STORM-924. --- Resolution: Fixed Fix Version/s: 0.11.0 Thanks [~caofangkun], I merged this int

[GitHub] storm pull request: STORM-913:Use Curator's delete().deletingChild...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/615#issuecomment-121968459 @caofangkun I am a bit nervous about this change. We are changing code that had the option to auto create parents or auto-delete parents to not have that option any more

[jira] [Commented] (STORM-913) Use Curator's delete().deletingChildrenIfNeeded() instead of zk/delete-recursive

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-913?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629792#comment-14629792 ] ASF GitHub Bot commented on STORM-913: -- Github user revans2 commented on the pull requ

[GitHub] storm pull request: Update Maven documentation to storm-core 0.9.5...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/618#issuecomment-121968859 +1 looks good to me --- 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 fea

[jira] [Commented] (STORM-924) Set the file mode of the files included when packaging release packages

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629781#comment-14629781 ] ASF GitHub Bot commented on STORM-924: -- Github user revans2 commented on the pull requ

[GitHub] storm pull request: Update Maven documentation to storm-core 0.9.5...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/618 --- 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 enabl

[GitHub] storm pull request: Update Maven documentation to storm-core 0.9.5...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/618#issuecomment-121969197 Thanks @charleswhchan I merged this into master. --- 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 p

STORM-799 PR available, feedback requested.

2015-07-16 Thread Aaron . Dossett
Hi, I took a look at this issue as a learning exercise. The change was pretty trivial, but one unexpected benefit of the change was removing a minor, but potentially risky, runtime type-check from MetricsConsumerBolt.java. _metricsConsumer.prepare(stormConf, _registrationArgument, context, (I

[GitHub] storm pull request: STORM-799 Use IErrorReport interface more broa...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/607#issuecomment-122006252 +1. The only possible issue is if someone is implementing their own OutputCollector, possibly for running storm topologies on a different runtime. And I am not too conce

[jira] [Commented] (STORM-799) Consolidate IErrorReporter and reportError methods

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629944#comment-14629944 ] ASF GitHub Bot commented on STORM-799: -- Github user revans2 commented on the pull requ

[GitHub] storm pull request: STORM-799 Use IErrorReport interface more broa...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/607 --- 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 enabl

[jira] [Commented] (STORM-799) Consolidate IErrorReporter and reportError methods

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629960#comment-14629960 ] ASF GitHub Bot commented on STORM-799: -- Github user asfgit closed the pull request at:

[jira] [Resolved] (STORM-799) Consolidate IErrorReporter and reportError methods

2015-07-16 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-799?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans resolved STORM-799. --- Resolution: Fixed Fix Version/s: 0.11.0 Thanks [~doss...@gmail.com], I just merg

[GitHub] storm pull request: Cleanup extra ']' bracket in documentation mar...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/619 --- 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 enabl

[GitHub] storm pull request: Cleanup extra ']' bracket in documentation mar...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/619#issuecomment-122010877 +1 looks great to me @charleswhchan I merged this into master keep up the good work. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] storm pull request: Added @Override annotation in sample code

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/620#issuecomment-122011966 +1 looks great to me. --- 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 f

[GitHub] storm pull request: Added @Override annotation in sample code

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/620 --- 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 enabl

[GitHub] storm pull request: Added @Override annotation in sample code

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/620#issuecomment-122012144 @charleswhchan I merged this into master thanks for the update. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] storm pull request: STORM-926:change pom to use maven-shade-plugin...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/621#issuecomment-122013414 The changes look fine to me. I am a bit concerned about minimizing the shaded jars. Have you tested this in a stand alone cluster? My understanding is that it removes c

[jira] [Commented] (STORM-926) change pom to use maven-shade-plugin:2.2 and minimize the size of shade jar and fix duplicated classes warnings of flue-examples

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-926?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629974#comment-14629974 ] ASF GitHub Bot commented on STORM-926: -- Github user revans2 commented on the pull requ

[GitHub] storm pull request: STORM-926:change pom to use maven-shade-plugin...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/621#issuecomment-122027544 I ran my own cluster and it looks good so I am inclined to check it in +1. --- If your project is set up for it, you can reply to this email and have your reply appear on

[jira] [Commented] (STORM-926) change pom to use maven-shade-plugin:2.2 and minimize the size of shade jar and fix duplicated classes warnings of flue-examples

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-926?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630032#comment-14630032 ] ASF GitHub Bot commented on STORM-926: -- Github user revans2 commented on the pull requ

[GitHub] storm pull request: STORM-926:change pom to use maven-shade-plugin...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/621 --- 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 enabl

[jira] [Commented] (STORM-926) change pom to use maven-shade-plugin:2.2 and minimize the size of shade jar and fix duplicated classes warnings of flue-examples

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-926?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630036#comment-14630036 ] ASF GitHub Bot commented on STORM-926: -- Github user asfgit closed the pull request at:

[jira] [Resolved] (STORM-926) change pom to use maven-shade-plugin:2.2 and minimize the size of shade jar and fix duplicated classes warnings of flue-examples

2015-07-16 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-926?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans resolved STORM-926. --- Resolution: Fixed Fix Version/s: 0.11.0 Thanks [~caofangkun], I merged this int

[GitHub] storm pull request: Add FluxParser method parseInputStream() to el...

2015-07-16 Thread lyonbrw
GitHub user lyonbrw opened a pull request: https://github.com/apache/storm/pull/637 Add FluxParser method parseInputStream() to eliminate disk usage. You can merge this pull request into a Git repository by running: $ git pull https://github.com/lyonbrw/storm master Alternati

[GitHub] storm pull request: Add FluxParser method parseInputStream() to el...

2015-07-16 Thread lyonbrw
Github user lyonbrw commented on the pull request: https://github.com/apache/storm/pull/637#issuecomment-122034395 Included reusable parseInputStream for use by parseFile and parseResource. Also allows a path for programatically creating topologies without the need to write to / read

[GitHub] storm pull request: STORM-793 Made change to logviewer.clj in orde...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/634 --- 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 enabl

[jira] [Resolved] (STORM-793) Logviewer 500 response when metadata has not yet been written (with auth enabled)

2015-07-16 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-793?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans resolved STORM-793. --- Resolution: Fixed Fix Version/s: 0.11.0 Thanks [~sanket991], I merged this into

[GitHub] storm pull request: Add FluxParser method parseInputStream() to el...

2015-07-16 Thread lyonbrw
Github user lyonbrw closed the pull request at: https://github.com/apache/storm/pull/637 --- 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

[jira] [Commented] (STORM-793) Logviewer 500 response when metadata has not yet been written (with auth enabled)

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-793?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630083#comment-14630083 ] ASF GitHub Bot commented on STORM-793: -- Github user asfgit closed the pull request at:

[GitHub] storm pull request: Add FluxParser method parseInputStream() to el...

2015-07-16 Thread lyonbrw
GitHub user lyonbrw reopened a pull request: https://github.com/apache/storm/pull/637 Add FluxParser method parseInputStream() to eliminate disk usage. You can merge this pull request into a Git repository by running: $ git pull https://github.com/lyonbrw/storm master Alterna

[jira] [Created] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

2015-07-16 Thread Brendan W Lyon (JIRA)
Brendan W Lyon created STORM-942: Summary: Add FluxParser method parseInputStream() to eliminate disk usage. Key: STORM-942 URL: https://issues.apache.org/jira/browse/STORM-942 Project: Apache Storm

[jira] [Updated] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

2015-07-16 Thread Brendan W Lyon (JIRA)
[ https://issues.apache.org/jira/browse/STORM-942?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Brendan W Lyon updated STORM-942: - Affects Version/s: 0.10.0 Fix Version/s: 0.11.0 > Add FluxParser method parseInputStream()

[GitHub] storm pull request: fix the dismatch between doc and code for read...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/570#issuecomment-122071976 @chuanlei Sorry this has taken so long. The change looks good to me. +1 @HeartSaVioR I don't think we need a JIRA for this, it is a minor documentation change.

[GitHub] storm pull request: fix the dismatch between doc and code for read...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/570 --- 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 enabl

[GitHub] storm pull request: fix the dismatch between doc and code for read...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/570#issuecomment-122078266 @revans2 I didn't mean this PR needs a JIRA issue. I just guided @chuanlei to get familiar with contributing rule while waiting for committers to review. I w

[GitHub] storm pull request: STORM-845 Storm ElasticSearch connector

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/573#issuecomment-122078315 @sweetest The code looks great to me. I would like to see some more javadocs explaining what each of the bolts/state do/need as input. I would also like to see some jav

[jira] [Commented] (STORM-845) Storm ElasticSearch connector

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630264#comment-14630264 ] ASF GitHub Bot commented on STORM-845: -- Github user revans2 commented on the pull requ

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/637#discussion_r34834409 --- Diff: external/flux/flux-core/src/main/java/org/apache/storm/flux/parser/FluxParser.java --- @@ -44,40 +44,47 @@ private FluxParser(){} // TO

[GitHub] storm pull request: Feature autotuning for max spout pending

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/582#issuecomment-122081360 @chuanlei I like auto-tuning max spout pending, but I am confused what we are tuning it for. The current code seems to increase the size of max-spout-pending if the numb

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

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

[GitHub] storm pull request: In TestHiveBolt make the collector mock slight...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/583#issuecomment-122082381 +1 @dossett do you want to file a JIRA for this? It is not a big deal because it is such a small change, but it does impact what type of credit you get for your chan

[GitHub] storm pull request: Feature add topology based scheduler

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/584#issuecomment-122083375 @chuanlei This is a rather interesting feature. It would be nice if you could file a JIRA for this. --- If your project is set up for it, you can reply to this email an

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/637#discussion_r34835109 --- Diff: external/flux/flux-core/src/main/java/org/apache/storm/flux/parser/FluxParser.java --- @@ -44,40 +44,47 @@ private FluxParser(){} // TO

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

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

[GitHub] storm pull request: fix the dismatch between doc and code for read...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/570#issuecomment-122084941 @HeartSaVioR Sounds good. Thanks by the way for all of the reviews before becoming a committer. We need more people to keep up with all of the pull requests. --- If y

[GitHub] storm pull request: [STORM-857] create logs metadata dir when runn...

2015-07-16 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/586#issuecomment-122085853 +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-857) Supervisor process fails to write log metadata to YAML file when "supervisor.run.worker.as.user" is enabled

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-857?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630293#comment-14630293 ] ASF GitHub Bot commented on STORM-857: -- Github user revans2 commented on the pull requ

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread lyonbrw
Github user lyonbrw commented on the pull request: https://github.com/apache/storm/pull/637#issuecomment-122088647 Just saw that redundant close() function before you commented. My bad. ;) Thanks. Corrected both, build in progress. --- If your project is set up for it, you can reply

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630300#comment-14630300 ] ASF GitHub Bot commented on STORM-942: -- Github user lyonbrw commented on the pull requ

[GitHub] storm pull request: [STORM-857] create logs metadata dir when runn...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/586 --- 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 enabl

[jira] [Commented] (STORM-857) Supervisor process fails to write log metadata to YAML file when "supervisor.run.worker.as.user" is enabled

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-857?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630311#comment-14630311 ] ASF GitHub Bot commented on STORM-857: -- Github user asfgit closed the pull request at:

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

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

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/637#issuecomment-122090455 @lyonbrw Indentation is still broken. Please compare parseInputStream()'s indent level to other methods. Btw, could you get rid of merge commit from PR i

[jira] [Resolved] (STORM-857) Supervisor process fails to write log metadata to YAML file when "supervisor.run.worker.as.user" is enabled

2015-07-16 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-857?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans resolved STORM-857. --- Resolution: Fixed Fix Version/s: 0.11.0 Thanks [~dagit], I merged this into mas

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread lyonbrw
Github user lyonbrw commented on the pull request: https://github.com/apache/storm/pull/637#issuecomment-122097348 Apologize for the merge commit. What is the proper way to remove it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630336#comment-14630336 ] ASF GitHub Bot commented on STORM-942: -- Github user lyonbrw commented on the pull requ

[GitHub] storm pull request: Feature autotuning for max spout pending

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/582#issuecomment-122097464 @chuanlei We can get a hint from Heron paper that Twitter don't catch up latest Storm version. Paper mentions ZeroMQ, but Storm changes default transport laye

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/637#issuecomment-122099913 @lyonbrw Actually I recommended you to make new branch before do some works. I can cherry-pick your commits into master, so let me handle it for now. --- If

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread lyonbrw
Github user lyonbrw commented on the pull request: https://github.com/apache/storm/pull/637#issuecomment-122099783 Indentation fixed. --- 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 feat

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630345#comment-14630345 ] ASF GitHub Bot commented on STORM-942: -- Github user lyonbrw commented on the pull requ

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

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

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/637#issuecomment-122100673 OK, confirmed that all things fixed. +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 proje

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

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

[GitHub] storm pull request: fix the dismatch between doc and code for read...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/570#issuecomment-122113786 @revans2 Sure. The more, the better. We(PMC members) are responsible for considering how to make Storm community more active. --- If your project is set up

[GitHub] storm pull request: [STORM-935] Update Disruptor queue version to ...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/630#issuecomment-122113950 +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] [Commented] (STORM-935) Update Disruptor queue version to 2.10.4

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

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/637#issuecomment-122117993 @lyonbrw I cherry-picked your efforts into master now. Would you close this PR? --- If your project is set up for it, you can reply to this email and have your r

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

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

[jira] [Resolved] (STORM-943) In TestHiveBolt make the collector mock slightly more precise

2015-07-16 Thread Aaron Dossett (JIRA)
[ https://issues.apache.org/jira/browse/STORM-943?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aaron Dossett resolved STORM-943. - Resolution: Fixed > In TestHiveBolt make the collector mock slightly more precise > ---

[jira] [Created] (STORM-943) In TestHiveBolt make the collector mock slightly more precise

2015-07-16 Thread Aaron Dossett (JIRA)
Aaron Dossett created STORM-943: --- Summary: In TestHiveBolt make the collector mock slightly more precise Key: STORM-943 URL: https://issues.apache.org/jira/browse/STORM-943 Project: Apache Storm

[GitHub] storm pull request: STORM-943: In TestHiveBolt make the collector ...

2015-07-16 Thread dossett
Github user dossett commented on the pull request: https://github.com/apache/storm/pull/583#issuecomment-122118472 Thanks for the feedback @revans2. STORM-943 created and associated to this JIRA. --- If your project is set up for it, you can reply to this email and have your reply a

[jira] [Commented] (STORM-943) In TestHiveBolt make the collector mock slightly more precise

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630429#comment-14630429 ] ASF GitHub Bot commented on STORM-943: -- Github user dossett commented on the pull requ

[jira] [Resolved] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

2015-07-16 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-942?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-942. Resolution: Fixed Assignee: Brendan W Lyon Thanks [~lyonbrw] for the great work. I merged into

[GitHub] storm pull request: [STORM-935] Update Disruptor queue version to ...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/630#issuecomment-122119993 Since many users are already affected this situation, it should be included to 0.9.x line. --- If your project is set up for it, you can reply to this email and have

[jira] [Commented] (STORM-935) Update Disruptor queue version to 2.10.4

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

[GitHub] storm pull request: [STORM-935] Update Disruptor queue version to ...

2015-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/630 --- 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 enabl

[jira] [Commented] (STORM-935) Update Disruptor queue version to 2.10.4

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-935?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630468#comment-14630468 ] ASF GitHub Bot commented on STORM-935: -- Github user asfgit closed the pull request at:

[GitHub] storm pull request: [STORM-935] Update Disruptor queue version to ...

2015-07-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/630#issuecomment-122124259 It is merged to master, and I'm working on backport to 0.10.x and 0.9.x. --- If your project is set up for it, you can reply to this email and have your reply appear

[jira] [Commented] (STORM-935) Update Disruptor queue version to 2.10.4

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

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630519#comment-14630519 ] ASF GitHub Bot commented on STORM-942: -- Github user lyonbrw closed the pull request at

[jira] [Commented] (STORM-942) Add FluxParser method parseInputStream() to eliminate disk usage.

2015-07-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14630518#comment-14630518 ] ASF GitHub Bot commented on STORM-942: -- Github user lyonbrw commented on the pull requ

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread lyonbrw
Github user lyonbrw commented on the pull request: https://github.com/apache/storm/pull/637#issuecomment-122130478 Will do. 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 project does not have this featur

[GitHub] storm pull request: [STORM-942] Add FluxParser method parseInputSt...

2015-07-16 Thread lyonbrw
Github user lyonbrw closed the pull request at: https://github.com/apache/storm/pull/637 --- 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

  1   2   >