Github user satishd commented on the issue:
https://github.com/apache/storm/pull/2053
+1 LGTM
---
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 fea
GitHub user vesense opened a pull request:
https://github.com/apache/storm/pull/2061
STORM-2469: fix integration-test dependencies issue
This is the hot fix for integration-test dependencies issue
* add storm-server. since __the ExclamationTopology depends on
LocalCluster__
Hi Taylor,
It is glad to see your opinion.
After the open source of Beam, there are a lot of interests in Beam from our
internal users in Alibaba and other companies in China, which promotes us to
provide the support of JStorm runner. But since the implementation of Storm
runner is out of date
-- Forwarded message --
From: Pracheer Agarwal
Date: Tue, Apr 11, 2017 at 9:07 AM
Subject: Distribution of workers across cluster with Resource Aware
Scheduler
To: u...@storm.apache.org
Hi All,
We are in the process of migrating to storm version 1.0.3. The pattern that
we observ
Github user liu-zhaokun commented on the issue:
https://github.com/apache/storm/pull/2058
@vesense Yes,you are right.
---
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 user vesense commented on the issue:
https://github.com/apache/storm/pull/2058
I guess the change is a copy from
http://storm.apache.org/releases/1.1.0/SECURITY.html "Storm Kerberos
Configuration" section.
Overall looks good to me.
---
If your project is set up for it, yo
Github user liu-zhaokun commented on the issue:
https://github.com/apache/storm/pull/2057
Hi, @vesense how are you?
I am sorry to bother you again.Would you help me to merge it?Thanks for
your help these days.
---
If your project is set up for it, you can reply to this email and
Github user liu-zhaokun commented on the issue:
https://github.com/apache/storm/pull/2056
Hi, @vesense how are you?
I am sorry to bother you again.Would you help me to merge it?Thanks for
your help these days.
---
If your project is set up for it, you can reply to this email and
Github user vesense commented on the issue:
https://github.com/apache/storm/pull/2056
+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 wishes so, or if the feature
Github user vesense commented on the issue:
https://github.com/apache/storm/pull/2057
+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 wishes so, or if the feature
Github user vesense commented on the issue:
https://github.com/apache/storm/pull/2059
@revans2 100ms dosen't work, but 1s works well. 500ms sometimes still get
failure.
I changed the value to 1s.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user harshach commented on the issue:
https://github.com/apache/storm/pull/2053
+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 wishes so, or if the feature
Github user harshach commented on the issue:
https://github.com/apache/storm/pull/2026
+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 wishes so, or if the feature
Github user ambud commented on a diff in the pull request:
https://github.com/apache/storm/pull/2054#discussion_r110977694
--- Diff:
storm-client/src/jvm/org/apache/storm/security/auth/RegexKerberosPrincipalToLocal.java
---
@@ -0,0 +1,56 @@
+/**
+ * Licensed to the Apache
Github user ambud commented on a diff in the pull request:
https://github.com/apache/storm/pull/2054#discussion_r110976923
--- Diff:
storm-client/src/jvm/org/apache/storm/security/auth/RegexKerberosPrincipalToLocal.java
---
@@ -0,0 +1,56 @@
+/**
+ * Licensed to the Apache
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2028
+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 wishes so, or if the feature
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2060
Only the last commit is the one to look at.
---
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 user revans2 opened a pull request:
https://github.com/apache/storm/pull/2060
STORM-2468: Remove clojure from storm-client
This is based off STORM-2447
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/revans2/incubator-storm
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2032
@ppoulosk Sadly with the latest refactoring of the code
`SupervisorUtils.processLauncherAndWait` is not on the classpath for storm-core
any more. It is a part of storm-server. You may need to do so
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2032
+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 wishes so, or if the feature
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2054#discussion_r110968429
--- Diff:
storm-client/src/jvm/org/apache/storm/security/auth/RegexKerberosPrincipalToLocal.java
---
@@ -0,0 +1,56 @@
+/**
+ * Licensed to the Apach
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2054#discussion_r110968735
--- Diff:
storm-client/src/jvm/org/apache/storm/security/auth/RegexKerberosPrincipalToLocal.java
---
@@ -0,0 +1,56 @@
+/**
+ * Licensed to the Apach
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2054#discussion_r110967560
--- Diff:
storm-client/src/jvm/org/apache/storm/security/auth/RegexKerberosPrincipalToLocal.java
---
@@ -0,0 +1,56 @@
+/**
+ * Licensed to the Apach
Github user harshach commented on the issue:
https://github.com/apache/storm/pull/2029
Thanks @hmcc. +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 wishes so, or
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2047
@HeartSaVioR could you please take a look at 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 not have this f
Github user harshach commented on the issue:
https://github.com/apache/storm/pull/2056
+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 wishes so, or if the feature
Github user marcust commented on the issue:
https://github.com/apache/storm/pull/2055
@srdo Done
---
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
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2055
Nice find. The change LGTM. You should create an issue for this at
https://issues.apache.org/jira/browse/STORM so the bugfix gets tracked. Then
rename this PR and the commit message to include the issue
Hi Taylor,
Thanks immensely for taking the time to write such rich detail. I have a
lot to learn about the relationship between Storm and JStorm as software
and as communities.
Your final note I can immediately agree with and reinforce. The fruits of
this endeavor should reside in the Beam reposi
Hi Taylor,
I am very glad to see the interests in pushing forward Beam Storm runner.
However, I cannot convince myself the benefits of having one runner to
support all.
Beam have three types of users: pipeline writers, library writers, and
runner implementers.
I can see pros vs cons as following
GitHub user vesense opened a pull request:
https://github.com/apache/storm/pull/2059
STORM-2463: fix DRPCTest.testDequeueAfterTimeout test failure
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vesense/storm STORM-2463
Alternat
GitHub user liu-zhaokun opened a pull request:
https://github.com/apache/storm/pull/2058
[STORM-2466] The example of jaas.conf in jaas_kerberos.conf should give
more details
[https://issues.apache.org/jira/browse/STORM-2466](url)
This documentation lacks several section such as
GitHub user liu-zhaokun opened a pull request:
https://github.com/apache/storm/pull/2057
[STORM-2465] modify storm-redis's READEME.md and update storm-redis.md
[https://issues.apache.org/jira/browse/STORM-2465](url)
There is a lack of "$" in the module of storm-redis's READEME.md
GitHub user liu-zhaokun opened a pull request:
https://github.com/apache/storm/pull/2056
[STORM-2464] update storm-mongodb.md
[https://issues.apache.org/jira/browse/STORM-2464](url)
storm-mongodb.md is out of date.For example,there is a new method in
MongoMapper,but there isn't
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2046
---
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 user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2048
---
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 user marcust opened a pull request:
https://github.com/apache/storm/pull/2055
Use explicit charset when decoding from array backed buffer
We had the case that this broke encoding for us because it used the default
system locale. Given the fact that this code uses an explic
Github user arunmahadevan commented on a diff in the pull request:
https://github.com/apache/storm/pull/2053#discussion_r110835582
--- Diff: storm-client/src/jvm/org/apache/storm/windowing/Window.java ---
@@ -47,9 +47,17 @@
List getExpired();
/**
- *
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2053#discussion_r110830274
--- Diff: storm-client/src/jvm/org/apache/storm/windowing/Window.java ---
@@ -47,9 +47,17 @@
List getExpired();
/**
- * If pro
39 matches
Mail list logo
