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

2023-08-01 Thread via GitHub
rzo1 commented on PR #3547: URL: https://github.com/apache/storm/pull/3547#issuecomment-1661608128 Guess we can include this after 2.5.0 vote is over. -- 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

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

2023-08-01 Thread via GitHub
rzo1 commented on PR #3546: URL: https://github.com/apache/storm/pull/3546#issuecomment-1661607392 @6harat triggered 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 URL above to go to the specific com

[GitHub] [storm] bipinprasad commented on pull request #3556: Missing dependency prevents Storm from being built, fixes #3940

2023-08-01 Thread via GitHub
bipinprasad commented on PR #3556: URL: https://github.com/apache/storm/pull/3556#issuecomment-1660768551 storm builds on all pull requests under github actions currently. However, I think it is better to consistent in excluding this dependency. -- This is an automated message from the Ap

Re: [GitHub] [storm] rzo1 commented on pull request #3556: Missing dependency prevents Storm from being built, fixes #3940

2023-08-01 Thread Bipin Prasad
I think it may be safe to exclude this dependency. However, we know that currently storm builds with github actions. On 2023/08/01 16:14:08 "rzo1 (via GitHub)" wrote: > > rzo1 commented on PR #3556: > URL: https://github.com/apache/storm/pull/3556#issuecomment-1660648311 > >> This artifact

[GitHub] [storm] rzo1 commented on pull request #3556: Missing dependency prevents Storm from being built, fixes #3940

2023-08-01 Thread via GitHub
rzo1 commented on PR #3556: URL: https://github.com/apache/storm/pull/3556#issuecomment-1660648311 > This artifact is available on maven central. Is there a need to exclude it? https://mvnrepository.com/artifact/org.pentaho/pentaho-aggdesigner-algorithm/5.1.5-jhyde I think, that the

Re: Storm 2.5.0 Release Candidate Vote

2023-08-01 Thread Richard Zowalla
On a clean, vanilla system without a custom maven mirror in place, I am hitting the same issue. The problem is, that Hive 2.3.9 declares the conjars.org repo: https://github.com/apache/hive/blob/rel/release-2.3.9/pom.xml#L225 This is the reason imho, why it gets pulled into our Maven build. Gruß

Re: Storm 2.5.0 Release Candidate Vote

2023-08-01 Thread Bipin Prasad
The failure in your build > http://conjars.org/repo/org/pentaho/pentaho-aggdesigner-algorithm/5.1.5-jhyde/pentaho-aggdesigner-algorithm-5.1.5-jhyde.pom > connection timeout conjars.org does not exist anymore. If this exists in your ~/.m2/settings.xml, please remove and retry. On 2023/07/28 04

[GitHub] [storm] bipinprasad commented on pull request #3556: Missing dependency prevents Storm from being built, fixes #3940

2023-08-01 Thread via GitHub
bipinprasad commented on PR #3556: URL: https://github.com/apache/storm/pull/3556#issuecomment-1660419649 This artifact is available on maven central. Is there a need to exclude it? https://mvnrepository.com/artifact/org.pentaho/pentaho-aggdesigner-algorithm/5.1.5-jhyde -- This is an auto

Re: Storm 2.5.0 Release Candidate Vote

2023-08-01 Thread Alexandre Vermeerbergen
Hello, Regarding my tests with Apache Storm 2.5.0 RC2 on a pre-production cluster at scale, with 10+ topologies, running on RockyLinux 8 with IBM Semeru 17.0.7 (based on OpenJ9 JVM) on Linux x64, I have detected no functional regression, not any noticeable performance changes. On side node, our t

[GitHub] [storm] jnioche opened a new pull request, #3556: Missing dependency prevents Storm from being built, fixes #3940

2023-08-01 Thread via GitHub
jnioche opened a new pull request, #3556: URL: https://github.com/apache/storm/pull/3556 ## What is the purpose of the change Storm can't be built because the dependency pentaho-aggdesigner-algorithm is not available.  This dependency is excluded in some places of the poms but

Re: Storm 2.5.0 Release Candidate Vote

2023-08-01 Thread Julien Nioche
Hi, Have experienced the same problems as Derek when trying to build. This is due to the dependency *pentaho-aggdesigner-algorithm *not being available. This dependency is excluded in some places of the poms but not systematically, I will open an issue and submit a PR shortly. Otherwise, I manage