Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2367#discussion_r143914514
--- Diff:
external/storm-kafka-client/src/main/java/org/apache/storm/kafka/spout/internal/OffsetManager.java
---
@@ -77,57 +78,58 @@ public void
Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2366#discussion_r143865842
--- Diff:
storm-client/src/jvm/org/apache/storm/grouping/LoadAwareShuffleGrouping.java ---
@@ -50,10 +69,28 @@
volatile int[] choices;
GitHub user revans2 opened a pull request:
https://github.com/apache/storm/pull/2368
STORM-2771: By default don't run any tests as integration tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/revans2/incubator-storm
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2366#discussion_r143862027
--- Diff:
storm-client/src/jvm/org/apache/storm/grouping/LoadAwareShuffleGrouping.java ---
@@ -50,10 +69,28 @@
volatile int[] choices;
Github user Ethanlm commented on the issue:
https://github.com/apache/storm/pull/2366
Thanks! Will do
---
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2366
@Ethanlm Everything looks good I am +1 on this change. I would love to see
more numbers to back this up, but as of right now it looks really good.
---
Github user kishorvpatil commented on a diff in the pull request:
https://github.com/apache/storm/pull/2363#discussion_r143844543
--- Diff:
storm-server/src/test/java/org/apache/storm/localizer/AsyncLocalizerTest.java
---
@@ -286,29 +297,80 @@ public void
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2366
I ran some small performance tests and things are looking really good.
Have you run any tests? How did you come up with the 80% config for growing in
size, and 20% for shrinking?
I would
Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2366#discussion_r143837500
--- Diff:
storm-client/src/jvm/org/apache/storm/grouping/LoadAwareShuffleGrouping.java ---
@@ -50,10 +61,28 @@
volatile int[] choices;
Thanks Jungtaek, that sounds like a good plan. Here's the new PR for 2607
https://github.com/apache/storm/pull/2367.
Beginning release next week sounds good to me.
2017-10-10 17:42 GMT+02:00 Arun Mahadevan :
> +1 for addressing the pending reviews and getting 1.2.0 out soon.
>
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2181
@tiodollar Since there is still a conflict here, and we'd like this to go
in 1.1.2 which we hope to release before too long, I've resolved the conflicts
and addressed the review comments. There's a PR
GitHub user srdo opened a pull request:
https://github.com/apache/storm/pull/2367
STORM-2607: Storm-kafka-client never commits the last message on a
partition.
This finishes up https://github.com/apache/storm/pull/2181, which has been
blocked on a rebase for a while. It was
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2366#discussion_r143829089
--- Diff:
storm-client/src/jvm/org/apache/storm/grouping/LoadAwareShuffleGrouping.java ---
@@ -50,10 +61,28 @@
volatile int[] choices;
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2366#discussion_r143828158
--- Diff:
storm-client/src/jvm/org/apache/storm/task/WorkerTopologyContext.java ---
@@ -34,6 +36,8 @@
private String _pidDir;
Map
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2366#discussion_r143826667
--- Diff:
storm-client/src/jvm/org/apache/storm/grouping/LoadAwareShuffleGrouping.java ---
@@ -20,14 +20,20 @@
import
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2366
@Ethanlm You have some NPEs in the LoadAwareShuffleGroupingTest
---
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2363
#2345 was merged so I rebased to make it more clear the new changes. I
actually delete code now :).
---
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2345
---
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2365#discussion_r143807488
--- Diff: storm-client/src/jvm/org/apache/storm/drpc/DRPCSpout.java ---
@@ -105,13 +105,19 @@ public Adder(String server, int port, Map
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2365
You are also going to need to fix the checkstyle errors.
---
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2365#discussion_r143807656
--- Diff: storm-client/src/jvm/org/apache/storm/drpc/DRPCSpout.java ---
@@ -105,13 +105,19 @@ public Adder(String server, int port, Map
Github user kishorvpatil commented on the issue:
https://github.com/apache/storm/pull/2345
@revans2 LGTM.
---
+1 for addressing the pending reviews and getting 1.2.0 out soon.
On 10/10/17, 6:14 AM, "Jungtaek Lim" wrote:
>Stig,
>
>Let's just handle all the issues pending Storm 1.1.2. For pending issues on
>Storm 1.2.0, I already handled all the things.
>
>For STORM-2607, could you
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/2203
@revans2 I need to recheck which thing (meter or counter) they are using,
but JStorm also doesnât sample metrics other than histogram. Unless they did
custom optimization, IMHO it should be
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2203
@HeartSaVioR I agree that if we cannot find a better way we may not want to
support a smooth transition for the metrics, but that also means that we cannot
put it into 1.x.
But I think the
Github user Ethanlm closed the pull request at:
https://github.com/apache/storm/pull/2270
---
Github user Ethanlm commented on the issue:
https://github.com/apache/storm/pull/2270
This https://github.com/apache/storm/pull/2366 addes locality awareness to
LoadAwareShuffleGrouping. Closing this one.
---
GitHub user Ethanlm opened a pull request:
https://github.com/apache/storm/pull/2366
[STORM-2686] Add locality awareness to LoadAwareShuffleGrouping
A redesign of https://github.com/apache/storm/pull/2270 . This adds the
locality awareness to LoadAwareShuffleGrouping. It applies
28 matches
Mail list logo