[GitHub] [storm] LoggingResearch closed pull request #3553: Handle InterruptedException in Supervisor close() method.

2023-07-22 Thread via GitHub
LoggingResearch closed pull request #3553: Handle InterruptedException in Supervisor close() method. URL: https://github.com/apache/storm/pull/3553 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [storm] LoggingResearch opened a new pull request, #3553: Handle InterruptedException in Supervisor close() method.

2023-07-17 Thread via GitHub
LoggingResearch opened a new pull request, #3553: URL: https://github.com/apache/storm/pull/3553 ## What is the purpose of the change This PR addresses the handling of InterruptedException in the close() method of the Supervisor class. The changes include: 1. Adding a separate

Re: [DISCUSS] Call for PMC Members and Contributors

2023-07-15 Thread Aaron Niskode-Dossett
Hi everyone, I can confirm that voting has begun. In terms of timeline, the vote must stay open for one week and then this is a minimum waiting period of a few days after we forward results to the board to have new PMC members added. So please expect another update toward the last week of July.

Re: Storm 2.5.0 Release Candidate Vote

2023-07-08 Thread Alexandre Vermeerbergen
Great for jdk11/jdk17 ! Attached the mvn clean install's GZIP logs which I forgot to attach in previous post, in case that's of any help as a feedback on RC1 Le sam. 8 juil. 2023 à 18:54, Bipin Prasad a écrit : > > About jdk11,jdk17 this is coming shortly. Because of some downstream >

Re: Storm 2.5.0 Release Candidate Vote

2023-07-08 Thread Bipin Prasad
About jdk11,jdk17 this is coming shortly. Because of some downstream dependencies, there were some issues with tests. After this release, we need to add jdk11 and 17 to GitHub actions (and then at some point drop jdk8). Sent from Yahoo Mail for iPhone On Saturday, July 8, 2023, 11:44 AM,

Re: Storm 2.5.0 Release Candidate Vote

2023-07-08 Thread Alexandre Vermeerbergen
More tests on Storm 2.5.0-RC1: * Using the binary distribution, I upgrade my Storm cluster with 6 workers/supervisor dedicated machines (without changing the big JARs which are still compiled with Storm 2.4.0 dependencies) => everything work fine so far * I tried to compile from sources on a

Re: Storm 2.5.0 Release Candidate Vote

2023-07-08 Thread Bipin Prasad
I am working on release candidate 2, but will also look at this issue this weekend and get back to you.Thanks for checking. Sent from Yahoo Mail for iPhone On Saturday, July 8, 2023, 1:15 AM, Alexandre Vermeerbergen wrote: Hello, I tried to check apache-storm-2.5.0.tar.gz.sha512(from

Re: Storm 2.5.0 Release Candidate Vote

2023-07-08 Thread Alexandre Vermeerbergen
Hello, I tried to check apache-storm-2.5.0.tar.gz.sha512(from https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.5.0-rc1/), but I get this error: $ /usr/bin/sha512sum -c apache-storm-2.5.0.tar.gz.sha512 /usr/bin/sha512sum: apache-storm-2.5.0.tar.gz.sha512: no properly formatted SHA512

[GitHub] [storm] bipinprasad merged pull request #3552: [STORM-3930] Change the source code control URI

2023-07-06 Thread via GitHub
bipinprasad merged PR #3552: URL: https://github.com/apache/storm/pull/3552 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] bipinprasad opened a new pull request, #3552: [STORM-3930] Change the source code control URI

2023-07-06 Thread via GitHub
bipinprasad opened a new pull request, #3552: URL: https://github.com/apache/storm/pull/3552 …d update the uri from gitbox to github ## What is the purpose of the change *While doing "mvm perform", the checkout times out at the end when using the https connection. Also the url

[GitHub] [storm] bipinprasad merged pull request #3549: [STORM-3927] Change python to python3 in examples

2023-07-01 Thread via GitHub
bipinprasad merged PR #3549: URL: https://github.com/apache/storm/pull/3549 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] bipinprasad commented on pull request #3547: [STORM-3925] Allow user resources (in WorkerTopologyContext) to be set by Worker Hooks

2023-06-30 Thread via GitHub
bipinprasad commented on PR #3547: URL: https://github.com/apache/storm/pull/3547#issuecomment-1614815685 Triggered workflow. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [storm] bipinprasad merged pull request #3550: [STORM-3928] Use python3 in flux test, examples and wrappers

2023-06-30 Thread via GitHub
bipinprasad merged PR #3550: URL: https://github.com/apache/storm/pull/3550 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] bipinprasad merged pull request #3551: [STORM-3929] Update documentation to refer to python3

2023-06-30 Thread via GitHub
bipinprasad merged PR #3551: URL: https://github.com/apache/storm/pull/3551 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] bipinprasad merged pull request #3548: [STORM-3926] Require python3 for bin/storm

2023-06-30 Thread via GitHub
bipinprasad merged PR #3548: URL: https://github.com/apache/storm/pull/3548 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] bipinprasad opened a new pull request, #3551: [STORM-3929] Update documentation to refer to python3

2023-06-29 Thread via GitHub
bipinprasad opened a new pull request, #3551: URL: https://github.com/apache/storm/pull/3551 ## What is the purpose of the change *Python2 has been deprecated and removed Storm 2.5.0. Update documentation to reflect this change.* ## How was the change tested *Render

[GitHub] [storm] bipinprasad opened a new pull request, #3550: [STORM-3928] Use python3 in flux test, examples and wrappers

2023-06-29 Thread via GitHub
bipinprasad opened a new pull request, #3550: URL: https://github.com/apache/storm/pull/3550 ## What is the purpose of the change *Python2 has been deprecated. However many systems still activate python2 when only using python. Change it to python3 to ensure Python version 3 is

[GitHub] [storm] bipinprasad opened a new pull request, #3549: [STORM-3927] Change python to python3 in examples

2023-06-29 Thread via GitHub
bipinprasad opened a new pull request, #3549: URL: https://github.com/apache/storm/pull/3549 ## What is the purpose of the change *Python2 has been deprecated. However many systems still activate python2 when only using python. Change it to python3 to ensure Python version 3 is

Re: Storm 2.5.0 Release Candidate Vote

2023-06-29 Thread Aaron Gresch
I found some issues with python vs python3 with the release candidate that Bipin is addressing. On Mon, Jun 26, 2023 at 6:15 PM Bipin Prasad wrote: > Storm Release candidate version 2.5.0 rc1 is here: > https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.5.0-rc1 > Please vote on this

[GitHub] [storm] rzo1 commented on a diff in pull request #3546: [STORM-3924] Support for declaring WorkerHook in Flux topology definitions

2023-06-29 Thread via GitHub
rzo1 commented on code in PR #3546: URL: https://github.com/apache/storm/pull/3546#discussion_r1246591112 ## docs/flux.md: ## @@ -765,6 +770,16 @@ The `override` property controls how includes affect the values defined in the c **N.B.:** Includes are not yet recursive.

Final Reminder: Community Over Code call for presentations closing soon

2023-06-28 Thread Rich Bowen
[Note: You're receiving this email because you are subscribed to one or more project dev@ mailing lists at the Apache Software Foundation.] This is your final reminder that the Call for Presentations for Community Over Code (formerly known as ApacheCon) is closing soon - on Thursday, 13 July 2023

Re: Onboarding new Database Connectors to storm-external

2023-06-27 Thread Bipin Prasad
Stephen,    That will be great. Hope you get this message since the return email has INVALID at the end. Thanks—Bipin Sent from Yahoo Mail for iPhone On Tuesday, June 27, 2023, 8:21 AM, Stephen Powis wrote: I'm happy to take up the redis spout.  I think I have an outstanding PR (might be 3

Re: Onboarding new Database Connectors to storm-external

2023-06-27 Thread Stephen Powis
I'm happy to take up the redis spout. I think I have an outstanding PR (might be 3 or 4 years old at this point tho...) to improve the existing one, and also have this implementation I'd be willing to give to the project after bringing it up to date:

Re: Storm 2.5.0 Release Candidate Vote

2023-06-27 Thread Alexandre Vermeerbergen
Great news! I will start testing this Storm 2.5.0 RC later this week, as i do not have access to my machines before. Thanks! Alex Le mar. 27 juin 2023, 01:15, Bipin Prasad a écrit : > Storm Release candidate version 2.5.0 rc1 is here: >

Storm 2.5.0 Release Candidate Vote

2023-06-26 Thread Bipin Prasad
Storm Release candidate version 2.5.0 rc1 is here: https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.5.0-rc1 Please vote on this release. 1. How to vote is described here: https://github.com/apache/storm/blob/master/RELEASING.md#how-to-vote-on-a-release-candidate 2. Release

Onboarding new Database Connectors to storm-external

2023-06-25 Thread 6harat
Hey, I have been accumulating a list of popular/trending db for which storm connectors are still missing and preparing a case for getting them prioritized and added to the repo based on user interest. For now, we have the below list of potential candidates (based solely on my own experience in

Re: [DISCUSS] Call for PMC Members and Contributors

2023-06-25 Thread 6harat
Hey, I happened to come across the board meeting notes for this month (21st June 2023) which states that: "A report was expected, but not received" (ref: https://whimsy.apache.org/board/minutes/Storm.html) Humble request from PMC chair if we can get some traction here.  On Tue, Jun 20, 2023

[GitHub] [storm] rzo1 commented on pull request #3547: [STORM-3925] Allow user resources (in WorkerTopologyContext) to be set by Worker Hooks

2023-06-25 Thread via GitHub
rzo1 commented on PR #3547: URL: https://github.com/apache/storm/pull/3547#issuecomment-1605965102 @6harat i am no committer on Storm, so nö karma to do so ;-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [storm] 6harat commented on pull request #3547: [STORM-3925] Allow user resources (in WorkerTopologyContext) to be set by Worker Hooks

2023-06-25 Thread via GitHub
6harat commented on PR #3547: URL: https://github.com/apache/storm/pull/3547#issuecomment-1605948131 @bipinprasad @rzo1 could you please help trigger the workflow -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR Review] STORM-3925: Allow user resources (in WorkerTopologyContext) to be set by Worker Hooks

2023-06-25 Thread 6harat
Hi, The dev changes related to https://issues.apache.org/jira/browse/STORM-3925 are now completed and tested locally. Associated PR: https://github.com/apache/storm/pull/3547 Requesting the existing committers/PMC members with relevant permissions to: 1. trigger the github workflow for the above

[GitHub] [storm] 6harat commented on pull request #3546: [STORM-3924] Support for declaring WorkerHook in Flux topology definitions

2023-06-25 Thread via GitHub
6harat commented on PR #3546: URL: https://github.com/apache/storm/pull/3546#issuecomment-1605932509 > I was going through other parts of the code and found this TODO. Seems like it was intended to be connected to WorkerHooks when those were first introduced. > > ``` >

Re: [PR Review] Support for declaring WorkerHook in Flux topology definitions

2023-06-25 Thread 6harat
Hey, Bumping up this thread as I am still awaiting review comments on the aforementioned PR: https://github.com/apache/storm/pull/3546 Let me know if anything else is required from my end. On Tue, Jun 20, 2023 at 9:14 PM 6harat wrote: > Hey, > > Just an update on the Apache JIRA account. I

[GitHub] [storm] bipinprasad merged pull request #3540: [STORM-3916] Add Round Robin Scheduling strategy with optional node limit

2023-06-20 Thread via GitHub
bipinprasad merged PR #3540: URL: https://github.com/apache/storm/pull/3540 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [DISCUSS] Call for PMC Members and Contributors

2023-06-20 Thread Alexandre Vermeerbergen
Agree with Bipin Prasad, since it seems that we're close to a Storm 2.5.0 candidate release... Le mar. 20 juin 2023 à 17:38, Bipin Prasad a écrit : > > I would urge PMC Chair to give us some guidance on this. > > > Sent from Yahoo Mail for iPhone > > > On Tuesday, June 20, 2023, 8:32 AM, 6harat

Re: [PR Review] Support for declaring WorkerHook in Flux topology definitions

2023-06-20 Thread 6harat
Hey, Just an update on the Apache JIRA account. I was able to get the access with help from us...@infra.apache.org. Requesting existing maintainers to please add me to the relevant user group for being able to get the JIRA assigned to myself going forward. (I have create ticket and comment

Re: [DISCUSS] Call for PMC Members and Contributors

2023-06-20 Thread Bipin Prasad
I would urge PMC Chair to give us some guidance on this. Sent from Yahoo Mail for iPhone On Tuesday, June 20, 2023, 8:32 AM, 6harat wrote: Hey, Just wanted to follow-up on the thread initiated by Richard, if there were any updates from the PMC circle that can be shared with the wider

Re: [DISCUSS] Call for PMC Members and Contributors

2023-06-20 Thread 6harat
Hey, Just wanted to follow-up on the thread initiated by Richard, if there were any updates from the PMC circle that can be shared with the wider audience. Also, curious about if we have any previous roadmap available for the further evolution of the project. (I checked on JIRA but wasn't able to

[GitHub] [storm] 6harat commented on pull request #3547: [STORM-3925] Allow user resources (in WorkerTopologyContext) to be set by Worker Hooks

2023-06-20 Thread via GitHub
6harat commented on PR #3547: URL: https://github.com/apache/storm/pull/3547#issuecomment-1599002790 Pending tasks: 1. Add feature tests 2. Update documentation / comments -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [storm] bipinprasad commented on pull request #3546: [STORM-XXXX] Support for declaring WorkerHook in Flux topology definitions

2023-06-19 Thread via GitHub
bipinprasad commented on PR #3546: URL: https://github.com/apache/storm/pull/3546#issuecomment-1597358946 Activated the checks. There may be some PMD exceptions - that can be checked locally as "mvn pmd:check" -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [storm] bipinprasad merged pull request #3545: [STORM-3923] Increase MAVEN_OPTS max memory setting to 2gb

2023-06-19 Thread via GitHub
bipinprasad merged PR #3545: URL: https://github.com/apache/storm/pull/3545 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: Storm Cassandra

2023-06-19 Thread Richard Zowalla
The PR by Bipin seems to fix it? ;) Am Freitag, dem 16.06.2023 um 20:19 +0200 schrieb Richard Zowalla: > Ok from looking into gitact-script.sh, it looks like we are skipping > storm-cassandra, storm-hive, storm-hdfs, storm-hbase, storm-sql, > storm- > hdfs-blobstore if we are building with Java

Re: [DISCUSS] Call for PMC Members and Contributors

2023-06-19 Thread Richard Zowalla
Don't want to come across as an annoying user, but have we made any progress in this area yet? Everyone who has expressed interest (usually) doesn't have read access to private@, so no one knows what the current status is regarding the continuation of the project by expanding the committer / pmc

[GitHub] [storm] 6harat commented on pull request #3546: [STORM-XXXX] Support for declaring WorkerHook in Flux topology definitions

2023-06-18 Thread via GitHub
6harat commented on PR #3546: URL: https://github.com/apache/storm/pull/3546#issuecomment-1596224842 I was going through other parts of the code and found this TODO. Seems like it was intended to be connected to WorkerHooks when those were first introduced. ``` private Map

[PR Review] Support for declaring WorkerHook in Flux topology definitions

2023-06-18 Thread 6harat
Hey, I submitted a PR for review yesterday and am looking for some help from existing contributors regarding the below: 1. new JIRA ticket for the feature (and if someone can also approve my JIRA account creation request if they are one of the approvers) 2. trigger the github workflow:

[GitHub] [storm] 6harat commented on pull request #3546: [STORM-XXXX] Support for declaring WorkerHook in Flux topology definitions

2023-06-17 Thread via GitHub
6harat commented on PR #3546: URL: https://github.com/apache/storm/pull/3546#issuecomment-1595677853 Please note the following observations / callouts: 1. have not added workerHoosk to executioncontext. AFAIS constructed spouts and bolts are being stored here so as to allow creation

[GitHub] [storm] 6harat opened a new pull request, #3546: [STORM-XXXX] Support for declaring WorkerHook in Flux topology definitions

2023-06-17 Thread via GitHub
6harat opened a new pull request, #3546: URL: https://github.com/apache/storm/pull/3546 ## What is the purpose of the change A while back the support for declaring WorkerHook in TopologyBuilder was added. Related JIRA: https://issues.apache.org/jira/browse/STORM-126. This change

Re: Storm Cassandra

2023-06-16 Thread Richard Zowalla
Ok from looking into gitact-script.sh, it looks like we are skipping storm-cassandra, storm-hive, storm-hdfs, storm-hbase, storm-sql, storm- hdfs-blobstore if we are building with Java 11 and run them with Java 8. That explains why Java 11 runs do not fail ;-) Nevertheless, we have runs, which

[GitHub] [storm] bipinprasad opened a new pull request, #3545: [STORM-3923] Increase MAVEN_OPTS max memory setting to 2gb

2023-06-16 Thread via GitHub
bipinprasad opened a new pull request, #3545: URL: https://github.com/apache/storm/pull/3545 ## What is the purpose of the change *Storm Cassandra (external/storm-cassandra) test fail because OOM* ## How was the code tested *Git Actions build and test run as part of this

Re: Storm Cassandra

2023-06-16 Thread Bipin Prasad
Created Jira [STORM-3923] Cassandra module fails tests probably OOM - ASF JIRA  . I think mvn test is using 3gb (inherited from main pom.xml), but will add directly in its pom under Cassandra. Also MAVEN_OPTS in git actions is set to 1.5gb. I wonder if that is limiting the memory. | | | |

Re: Storm Cassandra

2023-06-16 Thread Richard Zowalla
Hi, looks like it only falls occasionally in Java 8 with an OOM issue / JVM crash. Might be possible to just increase the JVM memory for that module. I remember from the Travis migration, that we had some heap adjustments in the scripts. Wdyt? Gruß Richard Am 16. Juni 2023 17:57:21 MESZ

Re: Storm Cassandra

2023-06-16 Thread Alexandre Vermeerbergen
+1 to exclude Storm Cassandra (my 2 cents) Thanks, Alex Le ven. 16 juin 2023 à 17:58, Bipin Prasad a écrit : > > All, >I am working on creating a new release for Storm 2.5.0. Unfortunately, > storm-cassandra has been failing tests since we moved to Git Actions. Can > someone take a look at

Storm Cassandra

2023-06-16 Thread Bipin Prasad
All, I am working on creating a new release for Storm 2.5.0. Unfortunately, storm-cassandra has been failing tests since we moved to Git Actions. Can someone take a look at it and possibly get it to work - with a newer version of Cassandra if possible and/or upgraded tests? If no one is

TAC Applications for Community Over Code North America and Asia now open

2023-06-16 Thread Gavin McDonald
Hi All, (This email goes out to all our user and dev project mailing lists, so you may receive this email more than once.) The Travel Assistance Committee has opened up applications to help get people to the following events: *Community Over Code Asia 2023 - * *August 18th to August 20th in

[GitHub] [storm] bipinprasad merged pull request #3544: [STORM-3922] Acker scheduling changes.

2023-06-13 Thread via GitHub
bipinprasad merged PR #3544: URL: https://github.com/apache/storm/pull/3544 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] bipinprasad opened a new pull request, #3544: [STORM-3922] Acker scheduling changes.

2023-06-12 Thread via GitHub
bipinprasad opened a new pull request, #3544: URL: https://github.com/apache/storm/pull/3544 ## What is the purpose of the change *Acker executors are sorted in a specific order that backtracking needs to account for.* ## How was the change tested *This code has been

Re: [DISCUSS] Call for PMC Members and Contributors

2023-06-12 Thread Alexandre Vermeerbergen
+1 Le lun. 12 juin 2023 à 11:57, Richard Zowalla a écrit : > > Some weeks passed, any updates? > > Gruß > Richard > > Am Donnerstag, dem 11.05.2023 um 19:16 -0400 schrieb P. Taylor Goetz: > > Thanks for the reminder Richard, > > > > In response, I’ve bumped the PMC-private discussion around

Re: [DISCUSS] Call for PMC Members and Contributors

2023-06-12 Thread Richard Zowalla
Some weeks passed, any updates? Gruß Richard Am Donnerstag, dem 11.05.2023 um 19:16 -0400 schrieb P. Taylor Goetz: > Thanks for the reminder Richard, > > In response, I’ve bumped the PMC-private discussion around adding new > members interested in contributing. > > - Taylor > > > On May 11,

[GitHub] [storm] agresch commented on a diff in pull request #3540: [STORM-3916] Add Round Robin Scheduling strategy with optional node limit

2023-06-07 Thread via GitHub
agresch commented on code in PR #3540: URL: https://github.com/apache/storm/pull/3540#discussion_r1221809229 ## storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/BaseResourceAwareStrategy.java: ## @@ -527,13 +538,13 @@ protected

[GitHub] [storm] bipinprasad commented on a diff in pull request #3540: [STORM-3916] Add Round Robin Scheduling strategy with optional node limit

2023-06-06 Thread via GitHub
bipinprasad commented on code in PR #3540: URL: https://github.com/apache/storm/pull/3540#discussion_r1220355853 ## storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/BaseResourceAwareStrategy.java: ## @@ -527,13 +538,13 @@ protected

[GitHub] [storm] bipinprasad commented on a diff in pull request #3540: [STORM-3916] Add Round Robin Scheduling strategy with optional node limit

2023-06-06 Thread via GitHub
bipinprasad commented on code in PR #3540: URL: https://github.com/apache/storm/pull/3540#discussion_r1220322597 ## storm-client/src/jvm/org/apache/storm/Config.java: ## @@ -859,9 +859,11 @@ public class Config extends HashMap { @IsString public static final String

[GitHub] [storm] bipinprasad commented on a diff in pull request #3540: [STORM-3916] Add Round Robin Scheduling strategy with optional node limit

2023-06-06 Thread via GitHub
bipinprasad commented on code in PR #3540: URL: https://github.com/apache/storm/pull/3540#discussion_r1220291317 ## storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/BaseResourceAwareStrategy.java: ## @@ -164,6 +168,7 @@ public

[GitHub] [storm] bipinprasad commented on a diff in pull request #3540: [STORM-3916] Add Round Robin Scheduling strategy with optional node limit

2023-06-06 Thread via GitHub
bipinprasad commented on code in PR #3540: URL: https://github.com/apache/storm/pull/3540#discussion_r1220277226 ## storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/BaseResourceAwareStrategy.java: ## @@ -123,6 +123,10 @@ public

[GitHub] [storm] bipinprasad commented on a diff in pull request #3540: [STORM-3916] Add Round Robin Scheduling strategy with optional node limit

2023-06-06 Thread via GitHub
bipinprasad commented on code in PR #3540: URL: https://github.com/apache/storm/pull/3540#discussion_r1219931053 ## storm-client/src/jvm/org/apache/storm/Config.java: ## @@ -859,9 +859,11 @@ public class Config extends HashMap { @IsString public static final String

[GitHub] [storm] bipinprasad commented on a diff in pull request #3540: [STORM-3916] Add Round Robin Scheduling strategy with optional node limit

2023-06-06 Thread via GitHub
bipinprasad commented on code in PR #3540: URL: https://github.com/apache/storm/pull/3540#discussion_r1219927539 ## storm-client/src/jvm/org/apache/storm/Config.java: ## @@ -859,9 +859,11 @@ public class Config extends HashMap { @IsString public static final String

[GitHub] [storm] agresch commented on a diff in pull request #3540: [STORM-3916] Add Round Robin Scheduling strategy with optional node limit

2023-06-05 Thread via GitHub
agresch commented on code in PR #3540: URL: https://github.com/apache/storm/pull/3540#discussion_r1218307249 ## storm-client/src/jvm/org/apache/storm/Config.java: ## @@ -859,9 +859,11 @@ public class Config extends HashMap { @IsString public static final String

Call for Presentations, Community Over Code Asia 2023

2023-06-05 Thread Rich Bowen
You are receiving this message because you are subscribed to one more more developer mailing lists at the Apache Software Foundation. The call for presentations is now open at "https://apachecon.com/acasia2023/cfp.html;, and will be closed by Sunday, Jun 18th, 2023 11:59 PM GMT. The event will

[GitHub] [storm] bipinprasad merged pull request #3538: [STORM-3921] Bump spring-core from 5.3.26 to 5.3.27 in /examples/storm-jms-examples

2023-06-02 Thread via GitHub
bipinprasad merged PR #3538: URL: https://github.com/apache/storm/pull/3538 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] bipinprasad commented on a diff in pull request #3536: [STORM-3911] fetch net.minidev:json-smart:2.3 artifact from maven repo before conjars.org repo is referenced

2023-06-01 Thread via GitHub
bipinprasad commented on code in PR #3536: URL: https://github.com/apache/storm/pull/3536#discussion_r1213687480 ## storm-core/pom.xml: ## @@ -137,6 +137,7 @@ org.mockito mockito-core +${mockito.version} Review Comment:

[GitHub] [storm] agresch commented on a diff in pull request #3536: [STORM-3911] fetch net.minidev:json-smart:2.3 artifact from maven repo before conjars.org repo is referenced

2023-06-01 Thread via GitHub
agresch commented on code in PR #3536: URL: https://github.com/apache/storm/pull/3536#discussion_r1213655650 ## storm-core/pom.xml: ## @@ -137,6 +137,7 @@ org.mockito mockito-core +${mockito.version} Review Comment: why was

Re: Does storm 2.4.0 supports IPv6 address

2023-06-01 Thread Bipin Prasad
Hello Aishwarya,    Are you using a helm chart for this deploy and/or k8s deployment/service files that you can share? I don't believe Storm has been extensively tested in IPv6. Thanks--Bipin On Thursday, June 1, 2023 at 12:20:45 PM PDT, Aishwarya Soni wrote: Hi, I have a storm 2.4.0

Does storm 2.4.0 supports IPv6 address

2023-06-01 Thread Aishwarya Soni
Hi, I have a storm 2.4.0 cluster with all the storm components such as Nimbus, UI, supervisors, and drpc. Recently we moved to a new GCP Kubernetes cluster where both traffic was coming from both IPv4 and IPv6 addresses. Previously, we were on a hashicorp nomad cluster where only IPv4 was

Does storm 2.4.0 supports IPv6 address

2023-06-01 Thread Aishwarya Soni
Hi, I have a storm 2.4.0 cluster with all the storm components such as Nimbus, UI, supervisors, and drpc. Recently we moved to a new GCP Kubernetes cluster where both traffic was coming from both IPv4 and IPv6 addresses. Previously, we were on a hashicorp nomad cluster where only IPv4 was

[GitHub] [storm] bipinprasad merged pull request #3535: [STORM-3910] Enhanced logging for rocksdb metrics store.

2023-06-01 Thread via GitHub
bipinprasad merged PR #3535: URL: https://github.com/apache/storm/pull/3535 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm-site] ScottBrenner commented on pull request #35: Update meetups.yml

2023-05-26 Thread via GitHub
ScottBrenner commented on PR #35: URL: https://github.com/apache/storm-site/pull/35#issuecomment-1565183279 @bipinprasad please review -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [storm] vesense commented on pull request #3543: STORM-3920 Update security document

2023-05-22 Thread via GitHub
vesense commented on PR #3543: URL: https://github.com/apache/storm/pull/3543#issuecomment-1557058816 Thanks @snikhil5 Merged into master. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [storm] vesense merged pull request #3543: STORM-3920 Update security document

2023-05-22 Thread via GitHub
vesense merged PR #3543: URL: https://github.com/apache/storm/pull/3543 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] snikhil5 opened a new pull request, #3543: STORM-3920 Update security document

2023-05-18 Thread via GitHub
snikhil5 opened a new pull request, #3543: URL: https://github.com/apache/storm/pull/3543 ## What is the purpose of the change Update the document for setting up secure cluster ## How was the change tested Tested on a secure cluster -- This is an automated message from

Re: Any comment on my proposal to update snakeyaml to 2.0 ?

2023-05-12 Thread Richard Zowalla
Hi, https://github.com/apache/storm/commit/4fb1feb9466ec84ec7d2b9ac00c10e1bc9d60333 Should be already fixed but awaits a release. Gruß Richard Am 12. Mai 2023 14:57:23 MESZ schrieb Alexandre Vermeerbergen : >Hello, > >I just created https://issues.apache.org/jira/browse/STORM-3918 >Any

Any comment on my proposal to update snakeyaml to 2.0 ?

2023-05-12 Thread Alexandre Vermeerbergen
Hello, I just created https://issues.apache.org/jira/browse/STORM-3918 Any objection for this dependency update ? I guess I can take care of this - I just need a reminder about how to properly create a git branch to deliver an update to .pom and make this branch linked to the JIRA Thanks, Alex

[GitHub] [storm] vesense commented on pull request #3542: [STORM-3917] Remove explicit worker heap size in ThroughputVsLatency

2023-05-12 Thread via GitHub
vesense commented on PR #3542: URL: https://github.com/apache/storm/pull/3542#issuecomment-1545705349 Thanks @jmartell7 Merged into master. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [storm] vesense merged pull request #3542: [STORM-3917] Remove explicit worker heap size in ThroughputVsLatency

2023-05-12 Thread via GitHub
vesense merged PR #3542: URL: https://github.com/apache/storm/pull/3542 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [DISCUSS] Call for PMC Members and Contributors

2023-05-12 Thread Richard Zowalla
Thanks, Taylor! Am Donnerstag, dem 11.05.2023 um 19:16 -0400 schrieb P. Taylor Goetz: > Thanks for the reminder Richard, > > In response, I’ve bumped the PMC-private discussion around adding new > members interested in contributing. > > - Taylor > > > On May 11, 2023, at 8:39 AM, Richard

Re: [DISCUSS] Call for PMC Members and Contributors

2023-05-11 Thread P. Taylor Goetz
Thanks for the reminder Richard, In response, I’ve bumped the PMC-private discussion around adding new members interested in contributing. - Taylor > On May 11, 2023, at 8:39 AM, Richard Zowalla wrote: > > Hello, > > I know we are all busy with our day jobs, but is there any news? > > We

Re: [DISCUSS] Call for PMC Members and Contributors

2023-05-11 Thread Richard Zowalla
Hello, I know we are all busy with our day jobs, but is there any news? We had some activity after a move to the attic was proposed. In the process, a few people volunteered to help with the project. A few weeks later, IMHO we are back to where we were before the proposal. Mails are going

Call for Presentations, Community Over Code 2023

2023-05-09 Thread Rich Bowen
(Note: You are receiving this because you are subscribed to the dev@ list for one or more Apache Software Foundation projects.) The Call for Presentations (CFP) for Community Over Code (formerly Apachecon) 2023 is open at https://communityovercode.org/call-for-presentations/, and will close Thu,

[GitHub] [storm] jmartell7 opened a new pull request, #3542: Remove explicit worker heap size in ThroughputVsLatency

2023-05-08 Thread via GitHub
jmartell7 opened a new pull request, #3542: URL: https://github.com/apache/storm/pull/3542 Allow users to change worker heap size in ThroughputVsLatency example topo. ## What is the purpose of the change ThroughputVsLatency adds an explicit 2GB worker heapsize overriding the

[GitHub] [storm] agresch merged pull request #3541: Fix infinite loop when no files are left to remove

2023-05-04 Thread via GitHub
agresch merged PR #3541: URL: https://github.com/apache/storm/pull/3541 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] jmartell7 opened a new pull request, #3541: Fix infinite loop when no files are left to remove

2023-05-03 Thread via GitHub
jmartell7 opened a new pull request, #3541: URL: https://github.com/apache/storm/pull/3541 ## What is the purpose of the change An infinite loop can occur when the worker directory takes too much space, but no files are found to remove. ## How was the change tested I have not

[GitHub] [storm] bipinprasad opened a new pull request, #3540: [STORM-3916] Add Round Robin Scheduling stretgy with optional node li…

2023-05-03 Thread via GitHub
bipinprasad opened a new pull request, #3540: URL: https://github.com/apache/storm/pull/3540 …mit. ## What is the purpose of the change *Round Robin strategy will distribute components evenly across nodes. Default behavior will be to spread across all nodes. But can be limited

Re: [DISCUSS] Call for PMC Members and Contributors

2023-05-02 Thread Richard Zowalla
Any updates? :-) Think Storm would benefit from a new release soon... Am Mittwoch, dem 22.02.2023 um 18:02 -0500 schrieb P. Taylor Goetz: > Quick update: > > The ASF Board voted to accept both our report and the resolution to > change PMC Chair. Board feedback on our report was positive. > >

[GitHub] [storm] bipinprasad merged pull request #3539: [STORM-3913] Update rocksdb to version 8.1.1 that has Mac OSX JNI lib

2023-05-01 Thread via GitHub
bipinprasad merged PR #3539: URL: https://github.com/apache/storm/pull/3539 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] bipinprasad commented on pull request #3539: [STORM-3913] Update rocksdb to version 8.1.1 that has Mac OSX JNI lib

2023-05-01 Thread via GitHub
bipinprasad commented on PR #3539: URL: https://github.com/apache/storm/pull/3539#issuecomment-1529862186 storm-cassandra test failure will be fixed in a separate Jira https://issues.apache.org/jira/browse/STORM-3915 -- This is an automated message from the Apache Git Service. To respond

[GitHub] [storm] bipinprasad commented on a diff in pull request #3539: [STORM-3913] Update rocksdb to version 8.1.1 that has Mac OSX JNI lib

2023-04-28 Thread via GitHub
bipinprasad commented on code in PR #3539: URL: https://github.com/apache/storm/pull/3539#discussion_r1180617441 ## storm-server/pom.xml: ## @@ -60,6 +60,7 @@ org.rocksdb rocksdbjni +${rocksdb-version} Review Comment: I missed

[GitHub] [storm] bipinprasad commented on a diff in pull request #3539: [STORM-3913] Update rocksdb to version 8.1.1 that has Mac OSX JNI lib

2023-04-28 Thread via GitHub
bipinprasad commented on code in PR #3539: URL: https://github.com/apache/storm/pull/3539#discussion_r1180600883 ## storm-server/pom.xml: ## @@ -60,6 +60,7 @@ org.rocksdb rocksdbjni +${rocksdb-version} Review Comment: The

[GitHub] [storm] agresch commented on a diff in pull request #3539: [STORM-3913] Update rocksdb to version 8.1.1 that has Mac OSX JNI lib

2023-04-28 Thread via GitHub
agresch commented on code in PR #3539: URL: https://github.com/apache/storm/pull/3539#discussion_r1180543200 ## storm-server/pom.xml: ## @@ -60,6 +60,7 @@ org.rocksdb rocksdbjni +${rocksdb-version} Review Comment: even though

[GitHub] [storm] bipinprasad commented on a diff in pull request #3539: [STORM-3913] Update rocksdb to version 8.1.1 that has Mac OSX JNI lib

2023-04-28 Thread via GitHub
bipinprasad commented on code in PR #3539: URL: https://github.com/apache/storm/pull/3539#discussion_r1180506358 ## storm-server/pom.xml: ## @@ -60,6 +60,7 @@ org.rocksdb rocksdbjni +${rocksdb-version} Review Comment: Not

[GitHub] [storm] agresch commented on a diff in pull request #3539: [STORM-3913] Update rocksdb to version 8.1.1 that has Mac OSX JNI lib

2023-04-28 Thread via GitHub
agresch commented on code in PR #3539: URL: https://github.com/apache/storm/pull/3539#discussion_r1180464412 ## storm-server/pom.xml: ## @@ -60,6 +60,7 @@ org.rocksdb rocksdbjni +${rocksdb-version} Review Comment: just curious

[GitHub] [storm] bipinprasad opened a new pull request, #3539: [STORM-3913] Update rocksdb to version 8.1.1 that has Mac OSX JNI lib

2023-04-27 Thread via GitHub
bipinprasad opened a new pull request, #3539: URL: https://github.com/apache/storm/pull/3539 ## What is the purpose of the change *Version 6.27.3 does not have Mac OSX JNI Library. Some tests fail to load because of this missing dependency on Mac.* ## How was the change tested

[GitHub] [storm] vesense merged pull request #3537: [STORM-3912] Reference new clojar for carbonite version 1.6.0

2023-04-27 Thread via GitHub
vesense merged PR #3537: URL: https://github.com/apache/storm/pull/3537 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [storm] bipinprasad commented on a diff in pull request #3537: [STORM-3912] Reference new clojar for carbonite version 1.6.0

2023-04-17 Thread via GitHub
bipinprasad commented on code in PR #3537: URL: https://github.com/apache/storm/pull/3537#discussion_r1169127101 ## DEPENDENCY-LICENSES: ## @@ -14,6 +14,7 @@ List of third-party dependencies grouped by their license type. Apache License +* carbonite

<    3   4   5   6   7   8   9   10   11   12   >