I forgot the one, but this is great time to revisit this, since we got
resolved many storm-kafka-client issues hence feeling it as fairly stable.
I'm +1 on deprecate storm-kafka at 1.x version lines and remove it at
2.0.0. Do we want to have explicit VOTE to receive more opinions here?
Thanks,
Ju
Hi devs,
I've noticed that we are providing 4 different version lines (1.1.x, 1.0.x,
0.10.x, 0.9.x) in download page, and I expect we will add one more for
1.2.0. Moreover, we have one more develop version line (2.0.0 - master)
which most of development happens there.
Recently we're releasing 3 v
Github user agresch commented on the issue:
https://github.com/apache/storm/pull/2554
That is correct. With the current default RocksDB implementation, we would
have workers forward to the Supervisor, and the Supervisor metric processor
would be NimbusMetricProcessor - sending them t
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/2554
@agresch
> We're planning on getting the metrics forwarded from the workers to the
Supervisor and inserting from HBase there (or forwarding to Nimbus for
RocksDB). In the non-HBase impl
Github user asfgit closed the pull request at:
https://github.com/apache/storm-site/pull/2
---
This vote is now closed and passes with 3 binding +1 votes, and no 0 or -1
votes.
Vote tally:
+1:
- Bobby Evans
- Jungtaek Lim
- P. Taylor Goetz
I will release the staged artifacts and announce the release after 24 hours.
-Taylor
> On Feb 8, 2018, at 2:22 PM, P. Taylor Goetz wrote:
>
> This
This vote is now closed and passes with 3 binding +1 votes and no -1 or 0 votes.
Vote tally:
+1:
- Bobby Evans
- Jungtaek Lim
- P. Taylor Goetz
I will release the staged artifacts and announce the release after 24 hours.
-Taylor
> On Feb 8, 2018, at 12:48 PM, P. Taylor Goetz wrote:
>
> This
+1 (binding)
-Taylor
> On Feb 8, 2018, at 12:48 PM, P. Taylor Goetz wrote:
>
> This is a call to vote on releasing Apache Storm 1.1.2 (rc3)
>
> Full list of changes in this release:
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.1.2-rc3/RELEASE_NOTES.html
>
> The tag/commit
+1 (binding)
-Taylor
> On Feb 8, 2018, at 2:22 PM, P. Taylor Goetz wrote:
>
> This is a call to vote on releasing Apache Storm 1.0.6 (rc3)
>
> Full list of changes in this release:
>
> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.0.6-rc3/RELEASE_NOTES.html
>
> The tag/commit t
This vote is now closed and passes with 4 binding +1 votes, 1 non-binding +1,
and no 0 or -1 votes.
Vote tally (* indicates a binding vote):
+1:
- Jungtaek Lim*
- Satish Duggana*
- Arun Iyer*
- Alexandre Vermeerbergen
- Bobby Evans*
I will release the staged artifacts and announce the release a
Github user ptgoetz commented on the issue:
https://github.com/apache/storm-site/pull/2
+1
---
Github user kishorvpatil closed the pull request at:
https://github.com/apache/storm/pull/2487
---
Github user kishorvpatil commented on the issue:
https://github.com/apache/storm/pull/2487
closing the PR as no longer needed for master branch.
---
+1
I built from source all of the unit tests passed and some basic integration
tests passed too.
On Sat, Feb 10, 2018 at 12:00 PM Alexandre Vermeerbergen <
avermeerber...@gmail.com> wrote:
> Hello,
>
> +1 (non binding)
>
> Details: I almost got nuts when applying storm-kafka-client-1.2.0 RC4
> t
Happy to do the check. This is something we don't want to get wrong.
On Fri, Feb 9, 2018 at 5:38 PM P. Taylor Goetz wrote:
> Doh! I didn’t even think consider that it was likely autogenerated by an
> IDE and likely identical to what the JDK would generate.
>
> I agree with Bobby, let’s leave it
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2502#discussion_r167625247
--- Diff:
storm-client/src/jvm/org/apache/storm/cluster/IStormClusterState.java ---
@@ -40,109 +40,117 @@
import org.apache.storm.nimbus.NimbusInfo;
Github user arunmahadevan commented on the issue:
https://github.com/apache/storm/pull/2555
+1
---
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2553
---
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2553
Thanks @agresch I just merged this into master.
---
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2554#discussion_r167608762
--- Diff:
storm-server/src/main/java/org/apache/storm/metricstore/WorkerMetricsProcessor.java
---
@@ -0,0 +1,40 @@
+/**
+ * Licensed to the Apache S
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2554#discussion_r167608546
--- Diff:
storm-server/src/main/java/org/apache/storm/metricstore/WorkerMetricsProcessor.java
---
@@ -0,0 +1,40 @@
+/**
+ * Licensed to the Apache S
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2554#discussion_r167607557
--- Diff: storm-server/src/main/java/org/apache/storm/DaemonConfig.java ---
@@ -1040,6 +1039,13 @@
// and log an error message, so just validating th
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2554#discussion_r167608078
--- Diff:
storm-server/src/main/java/org/apache/storm/metricstore/MetricStoreConfig.java
---
@@ -41,5 +41,23 @@ public static MetricStore configure(Map conf
Github user agresch commented on the issue:
https://github.com/apache/storm/pull/2553
Can this be merged?
---
GitHub user HeartSaVioR opened a pull request:
https://github.com/apache/storm/pull/2555
STORM-2841 Use extended class instead of partial mock for TestHiveBolt
* while mocking partially, oddly it calls real method intermittently which
breaks test
Please refer https://issues
25 matches
Mail list logo
