[
https://issues.apache.org/jira/browse/STORM-1038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942559#comment-14942559
]
ASF GitHub Bot commented on STORM-1038:
---
Github user hsun-cnnxty commented on the pu
Github user hsun-cnnxty commented on the pull request:
https://github.com/apache/storm/pull/728#issuecomment-145318156
@revans2, just merged with the latest master. I don't have a decent storm
cluster for performance test. With a small local cluster on single machine.
I had tried
[
https://issues.apache.org/jira/browse/STORM-1015?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942549#comment-14942549
]
ASF GitHub Bot commented on STORM-1015:
---
Github user hsun-cnnxty commented on the pu
Github user hsun-cnnxty commented on the pull request:
https://github.com/apache/storm/pull/705#issuecomment-145317404
Hi Chi,
Storm stores more than just offset/partition data in the "state", would it
be necessary to declare?
public interface StateStore {
publ
GitHub user drorweiss opened a pull request:
https://github.com/apache/storm/pull/781
Add codota search for apache/stoem to Readme
The Storm codebase was semantically indexed in Codota - available at
https://www.codota.com/xref/#/github_apache_storm_560da9ada8cb8703008bbfdc/find
Github user mjtieman commented on the pull request:
https://github.com/apache/storm/pull/454#issuecomment-145277282
Conflicts resolved.
---
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 user harshach commented on the pull request:
https://github.com/apache/storm/pull/778#issuecomment-145275960
+1. LGTM. I am not sure why the build is failing though.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/774#issuecomment-145275933
+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
[
https://issues.apache.org/jira/browse/STORM-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942405#comment-14942405
]
ASF GitHub Bot commented on STORM-1081:
---
Github user harshach commented on the pull
[
https://issues.apache.org/jira/browse/STORM-1080?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942407#comment-14942407
]
ASF GitHub Bot commented on STORM-1080:
---
Github user harshach commented on the pull
[
https://issues.apache.org/jira/browse/STORM-412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942381#comment-14942381
]
ASF GitHub Bot commented on STORM-412:
--
Github user Parth-Brahmbhatt commented on the
Github user Parth-Brahmbhatt commented on the pull request:
https://github.com/apache/storm/pull/766#issuecomment-145263234
+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
en
[
https://issues.apache.org/jira/browse/STORM-893?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942378#comment-14942378
]
ASF GitHub Bot commented on STORM-893:
--
Github user jerrypeng commented on the pull re
Github user jerrypeng commented on the pull request:
https://github.com/apache/storm/pull/746#issuecomment-145261204
@revans2 Thanks for your review again! I just push a commit that contains
those revisions you suggested
---
If your project is set up for it, you can reply to this e
[
https://issues.apache.org/jira/browse/STORM-828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942377#comment-14942377
]
ASF GitHub Bot commented on STORM-828:
--
Github user revans2 commented on the pull requ
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/668#issuecomment-145261119
Sorry it took me so long to review this. For the most part things look
good. I made a few comments on the HdfBolt that I would like to see reflected
in the SequenceFile
[
https://issues.apache.org/jira/browse/STORM-828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942376#comment-14942376
]
ASF GitHub Bot commented on STORM-828:
--
Github user revans2 commented on a diff in the
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/668#discussion_r41088521
--- Diff:
external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/HdfsBolt.java ---
@@ -44,6 +49,25 @@
private transient FSDataOutputStream out;
[
https://issues.apache.org/jira/browse/STORM-828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942375#comment-14942375
]
ASF GitHub Bot commented on STORM-828:
--
Github user revans2 commented on a diff in the
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/668#discussion_r41088509
--- Diff:
external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/HdfsBolt.java ---
@@ -44,6 +49,25 @@
private transient FSDataOutputStream out;
[
https://issues.apache.org/jira/browse/STORM-828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942374#comment-14942374
]
ASF GitHub Bot commented on STORM-828:
--
Github user revans2 commented on a diff in the
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/668#discussion_r41088495
--- Diff:
external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/HdfsBolt.java ---
@@ -44,6 +49,25 @@
private transient FSDataOutputStream out;
[
https://issues.apache.org/jira/browse/STORM-828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942373#comment-14942373
]
ASF GitHub Bot commented on STORM-828:
--
Github user revans2 commented on a diff in the
[
https://issues.apache.org/jira/browse/STORM-828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942372#comment-14942372
]
ASF GitHub Bot commented on STORM-828:
--
Github user revans2 commented on a diff in the
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/668#discussion_r41088489
--- Diff:
external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/HdfsBolt.java ---
@@ -44,6 +49,25 @@
private transient FSDataOutputStream out;
Hi Parth,
Unfortunately I really don't have additional info to provide, but your
response was helpful.
Thanks,
Abe
On Fri, Oct 2, 2015 at 1:32 PM, Parth Brahmbhatt <
pbrahmbh...@hortonworks.com> wrote:
> Can you elaborate/send logs to explain what you mean by ³bad state².
> Restarting nimbus
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/668#discussion_r41088474
--- Diff:
external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/HdfsBolt.java ---
@@ -44,6 +49,25 @@
private transient FSDataOutputStream out;
[
https://issues.apache.org/jira/browse/STORM-828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942370#comment-14942370
]
ASF GitHub Bot commented on STORM-828:
--
Github user revans2 commented on a diff in the
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/668#discussion_r41088459
--- Diff:
external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/CSVFileBolt.java
---
@@ -0,0 +1,32 @@
+/**
+ * Licensed to the Apache Software
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/776
---
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
[
https://issues.apache.org/jira/browse/STORM-1057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942368#comment-14942368
]
ASF GitHub Bot commented on STORM-1057:
---
Github user wangli1426 commented on the pul
Github user wangli1426 commented on the pull request:
https://github.com/apache/storm/pull/753#issuecomment-145260599
@revans2 ,
I have addressed all your concerns in d552a99. The most important
modification is that instead of employing RateTracker, I reuse the stats of
```emitted
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/668#discussion_r41088401
--- Diff:
external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/HdfsBolt.java ---
@@ -44,6 +49,25 @@
private transient FSDataOutputStream out;
[
https://issues.apache.org/jira/browse/STORM-828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942366#comment-14942366
]
ASF GitHub Bot commented on STORM-828:
--
Github user revans2 commented on a diff in the
[
https://issues.apache.org/jira/browse/STORM-997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942364#comment-14942364
]
ASF GitHub Bot commented on STORM-997:
--
Github user revans2 commented on the pull requ
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/692#issuecomment-145260515
@priyank5485 and others I am not totally sure that this is a good idea. If
HDFS is running secure and Storm is running insecure, with this feature enabled
you have now d
[
https://issues.apache.org/jira/browse/STORM-1073?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942361#comment-14942361
]
ASF GitHub Bot commented on STORM-1073:
---
Github user harshach commented on the pull
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/767#issuecomment-145260302
@arunmahadevan please review this
---
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 no
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/667#issuecomment-145260043
@caishize Any update on this? I really would like to have cgroup support,
and this should work really well with the resource aware scheduling we are
working on, on anoth
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/777
---
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 user revans2 commented on the pull request:
https://github.com/apache/storm/pull/728#issuecomment-145253788
@hsun-cnnxty, The changes look good to me. Please upmerge to the latest
code. Then I would like to run some performance tests on it to see how it
compares to the curren
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/758#issuecomment-145252882
@prokopowicz I am +1 for merging this in. Is there a JIRA associated with
this change?
---
If your project is set up for it, you can reply to this email and have your
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/772#issuecomment-145254567
@ptgoetz @HeartSaVioR @revans2 I really don't want to make batching
optional. Without batching in my tests I am able to insert 2500 tuples per sec.
Thats abysmal perfor
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/746#issuecomment-145253628
@jerrypeng Just two more issues, both of which re relatively minor. After
them I am +1 for merging this in. Then hopefully development work on RAS can
move forward in o
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/766#issuecomment-145252754
I am still +1 on this. @kishorvpatil The merge conflict is very minor in
the commands list in storm.py, please upmerge.
@Parth-Brahmbhatt I really would like to
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/746#discussion_r41087631
--- Diff:
storm-core/src/jvm/backtype/storm/scheduler/resource/strategies/ResourceAwareStrategy.java
---
@@ -0,0 +1,478 @@
+/**
+ * Licensed to the Ap
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/746#discussion_r41087614
--- Diff:
storm-core/src/jvm/backtype/storm/networktopography/DefaultRackDNSToSwitchMapping.java
---
@@ -0,0 +1,39 @@
+package backtype.storm.networktopog
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/554#issuecomment-145251891
@d2r please upmerge. The code still looks great but it looks like upSecs
and a debug were added into the results. I really want to get this in, as it
will speed up page
[
https://issues.apache.org/jira/browse/STORM-1079?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942338#comment-14942338
]
ASF GitHub Bot commented on STORM-1079:
---
Github user harshach commented on the pull
[
https://issues.apache.org/jira/browse/STORM-1038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942333#comment-14942333
]
ASF GitHub Bot commented on STORM-1038:
---
Github user revans2 commented on the pull r
[
https://issues.apache.org/jira/browse/STORM-893?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942330#comment-14942330
]
ASF GitHub Bot commented on STORM-893:
--
Github user revans2 commented on the pull requ
[
https://issues.apache.org/jira/browse/STORM-893?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942329#comment-14942329
]
ASF GitHub Bot commented on STORM-893:
--
Github user revans2 commented on a diff in the
[
https://issues.apache.org/jira/browse/STORM-893?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942326#comment-14942326
]
ASF GitHub Bot commented on STORM-893:
--
Github user revans2 commented on a diff in the
[
https://issues.apache.org/jira/browse/STORM-412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942318#comment-14942318
]
ASF GitHub Bot commented on STORM-412:
--
Github user revans2 commented on the pull requ
[
https://issues.apache.org/jira/browse/STORM-820?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942313#comment-14942313
]
ASF GitHub Bot commented on STORM-820:
--
Github user revans2 commented on the pull requ
[
https://issues.apache.org/jira/browse/STORM-1078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942294#comment-14942294
]
ASF GitHub Bot commented on STORM-1078:
---
Github user asfgit closed the pull request
[
https://issues.apache.org/jira/browse/STORM-1078?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Robert Joseph Evans resolved STORM-1078.
Resolution: Fixed
Fix Version/s: 0.11.0
> RateTracker.java is not thread safe
[
https://issues.apache.org/jira/browse/STORM-1082?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Robert Joseph Evans resolved STORM-1082.
Resolution: Fixed
Fix Version/s: 0.11.0
Thanks [~zhuoliu],
I merged this int
[
https://issues.apache.org/jira/browse/STORM-1082?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942288#comment-14942288
]
ASF GitHub Bot commented on STORM-1082:
---
Github user asfgit closed the pull request
[
https://issues.apache.org/jira/browse/STORM-1079?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942159#comment-14942159
]
ASF GitHub Bot commented on STORM-1079:
---
Github user HeartSaVioR commented on the pu
Github user HeartSaVioR commented on the pull request:
https://github.com/apache/storm/pull/772#issuecomment-145211043
Maybe it also needs to include default tick interval as @dossett stated.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user HeartSaVioR commented on the pull request:
https://github.com/apache/storm/pull/772#issuecomment-145210986
I missed something before voting: seems like we should think about meaning
about default values.
If user takes default configuration, tuples could be timed ou
[
https://issues.apache.org/jira/browse/STORM-1079?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14942158#comment-14942158
]
ASF GitHub Bot commented on STORM-1079:
---
Github user HeartSaVioR commented on the pu
63 matches
Mail list logo
