[GitHub] [storm] rzo1 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
rzo1 commented on code in PR #3537: URL: https://github.com/apache/storm/pull/3537#discussion_r1169089745 ## DEPENDENCY-LICENSES: ## @@ -14,6 +14,7 @@ List of third-party dependencies grouped by their license type. Apache License +* carbonite

[GitHub] [storm] rzo1 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
rzo1 commented on code in PR #3537: URL: https://github.com/apache/storm/pull/3537#discussion_r1169089745 ## DEPENDENCY-LICENSES: ## @@ -14,6 +14,7 @@ List of third-party dependencies grouped by their license type. Apache License +* carbonite

[GitHub] [storm] dependabot[bot] opened a new pull request, #3538: Bump spring-core from 5.3.26 to 5.3.27 in /examples/storm-jms-examples

2023-04-17 Thread via GitHub
dependabot[bot] opened a new pull request, #3538: URL: https://github.com/apache/storm/pull/3538 Bumps [spring-core](https://github.com/spring-projects/spring-framework) from 5.3.26 to 5.3.27. Release notes Sourced from

[GitHub] [storm-site] dependabot[bot] closed pull request #36: Bump nokogiri from 1.13.3 to 1.13.9

2023-04-12 Thread via GitHub
dependabot[bot] closed pull request #36: Bump nokogiri from 1.13.3 to 1.13.9 URL: https://github.com/apache/storm-site/pull/36 -- 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.

[GitHub] [storm-site] dependabot[bot] commented on pull request #36: Bump nokogiri from 1.13.3 to 1.13.9

2023-04-12 Thread via GitHub
dependabot[bot] commented on PR #36: URL: https://github.com/apache/storm-site/pull/36#issuecomment-1504707642 Superseded by #39. -- 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-site] dependabot[bot] opened a new pull request, #39: Bump nokogiri from 1.13.3 to 1.14.3

2023-04-12 Thread via GitHub
dependabot[bot] opened a new pull request, #39: URL: https://github.com/apache/storm-site/pull/39 Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.13.3 to 1.14.3. Release notes Sourced from https://github.com/sparklemotion/nokogiri/releases;>nokogiri's releases.

[GitHub] [storm-site] dependabot[bot] closed pull request #37: Bump nokogiri from 1.13.3 to 1.13.9 in /content

2023-04-12 Thread via GitHub
dependabot[bot] closed pull request #37: Bump nokogiri from 1.13.3 to 1.13.9 in /content URL: https://github.com/apache/storm-site/pull/37 -- 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-site] dependabot[bot] commented on pull request #37: Bump nokogiri from 1.13.3 to 1.13.9 in /content

2023-04-12 Thread via GitHub
dependabot[bot] commented on PR #37: URL: https://github.com/apache/storm-site/pull/37#issuecomment-1504705472 Superseded by #38. -- 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-site] dependabot[bot] opened a new pull request, #38: Bump nokogiri from 1.13.3 to 1.14.3 in /content

2023-04-12 Thread via GitHub
dependabot[bot] opened a new pull request, #38: URL: https://github.com/apache/storm-site/pull/38 Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.13.3 to 1.14.3. Release notes Sourced from https://github.com/sparklemotion/nokogiri/releases;>nokogiri's releases.

[GitHub] [storm] vesense merged pull request #3534: [STORM-3909] Use python3 in metrics test

2023-04-10 Thread via GitHub
vesense merged PR #3534: URL: https://github.com/apache/storm/pull/3534 -- 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, #3537: [STORM-3912] Reference new clojar for carbonite version 1.6.0

2023-04-10 Thread via GitHub
bipinprasad opened a new pull request, #3537: URL: https://github.com/apache/storm/pull/3537 ## What is the purpose of the change *Carbonite:1.5.0 jar hasn't been changed since 2012 and includes some very old dependencies and java version. Create a new carbonite jar with version

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

2023-04-07 Thread via GitHub
bipinprasad opened a new pull request, #3536: URL: https://github.com/apache/storm/pull/3536 ## What is the purpose of the change *Transitive dependency from hadoop-auth pulls in net.minidev:json-smart from http://conjars.org. conjars.org site is no longer active. And timeouts in

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

2023-04-07 Thread via GitHub
bipinprasad opened a new pull request, #3535: URL: https://github.com/apache/storm/pull/3535 ## What is the purpose of the change *Rocksdb requires a platform specific JNI library. This library is included in the rocksdbjni jar but is not loaded unless the dynamic library is in a

[GitHub] [storm] bipinprasad opened a new pull request, #3534: [STORM-3909] Use python3 in metrics test

2023-04-07 Thread via GitHub
bipinprasad opened a new pull request, #3534: URL: https://github.com/apache/storm/pull/3534 ## What is the purpose of the change *Ensure that python version 3 is used* ## How was the change tested *Run test* -- This is an automated message from the Apache Git

[GitHub] [storm] vesense merged pull request #3533: [STORM-3908] Increase heap memory for MAVEN_OPTS used in github actions

2023-04-07 Thread via GitHub
vesense merged PR #3533: URL: https://github.com/apache/storm/pull/3533 -- 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, #3533: [STORM-3908] Increase heap memory for MAVEN_OPTS used in github actions

2023-04-05 Thread via GitHub
bipinprasad opened a new pull request, #3533: URL: https://github.com/apache/storm/pull/3533 ## What is the purpose of the change *Sometimes github actions get out-of-memory exception. Since the Github Action Runners as bigger VMs that Travis used in the past, the heap can be made

[GitHub] [storm] bipinprasad merged pull request #3532: [STORM-3907] Update mockito to version 4.11.0 and change code to match

2023-04-05 Thread via GitHub
bipinprasad merged PR #3532: URL: https://github.com/apache/storm/pull/3532 -- 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 #3530: [STORM-3905] Replace anonymous class with lambda in storm-core commands

2023-04-05 Thread via GitHub
bipinprasad merged PR #3530: URL: https://github.com/apache/storm/pull/3530 -- 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 #3531: [STORM-3906] Remove org.mockito.internal.util.reflection.FieldSetter and set mockito.version to 3.11.2

2023-03-30 Thread via GitHub
bipinprasad merged PR #3531: URL: https://github.com/apache/storm/pull/3531 -- 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:

A Message from the Board to PMC members

2023-03-29 Thread Rich Bowen
Dear Apache Project Management Committee (PMC) members, The Board wants to take just a moment of your time to communicate a few things that seem to have been forgotten by a number of PMC members, across the Foundation, over the past few years. Please note that this is being sent to all projects

[GitHub] [storm] rzo1 commented on pull request #3492: Enable s390x builds on Travis

2023-03-29 Thread via GitHub
rzo1 commented on PR #3492: URL: https://github.com/apache/storm/pull/3492#issuecomment-1488488044 Think it needs https://github.com/actions/runner/issues/2263 to support x390 for GH actions. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [storm] shahidhs-ibm closed pull request #3492: Enable s390x builds on Travis

2023-03-29 Thread via GitHub
shahidhs-ibm closed pull request #3492: Enable s390x builds on Travis URL: https://github.com/apache/storm/pull/3492 -- 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

[GitHub] [storm] shahidhs-ibm commented on pull request #3492: Enable s390x builds on Travis

2023-03-29 Thread via GitHub
shahidhs-ibm commented on PR #3492: URL: https://github.com/apache/storm/pull/3492#issuecomment-1488485620 Closing this PR as it is no longer valid as Storm has replaced Travis with GitHub Actions. Ref. https://github.com/apache/storm/pull/3520 -- This is an automated message from the

[GitHub] [storm] bipinprasad merged pull request #3528: [STORM-3904] Bump spring-core from 5.3.20 to 5.3.26 in /examples/storm-jms-examples

2023-03-28 Thread via GitHub
bipinprasad merged PR #3528: URL: https://github.com/apache/storm/pull/3528 -- 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 #3529: [STORM-3903] Bump commons-fileupload from 1.3.3 to 1.5

2023-03-28 Thread via GitHub
bipinprasad merged PR #3529: URL: https://github.com/apache/storm/pull/3529 -- 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, #3532: [STORM-3907] Update mockito to version 4.11.0 and change code to martch

2023-03-28 Thread via GitHub
bipinprasad opened a new pull request, #3532: URL: https://github.com/apache/storm/pull/3532 ## What is the purpose of the change *Update mockito so to version 4 with eventual goal of moving to version 5* ## How was the change tested *Local build and test* -- This is

[GitHub] [storm] rzo1 commented on pull request #3509: [STORM-3884] Bump calcite-core from 1.14.0 to 1.32.0

2023-03-27 Thread via GitHub
rzo1 commented on PR #3509: URL: https://github.com/apache/storm/pull/3509#issuecomment-1486255145 Rebase for GH actions? -- 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.

[GitHub] [storm] rzo1 commented on pull request #3506: STORM-3888 HdfsBlobStoreFile set wrong permission for file

2023-03-27 Thread via GitHub
rzo1 commented on PR #3506: URL: https://github.com/apache/storm/pull/3506#issuecomment-1486254715 @skysiders can you rebase? -- 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] rzo1 commented on pull request #3510: [STORM-3884] Update calcite to 1.16.0

2023-03-27 Thread via GitHub
rzo1 commented on PR #3510: URL: https://github.com/apache/storm/pull/3510#issuecomment-1486254414 @bipinprasad can you rebase so we get GH actions? -- 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 #3525: [STORM-3900] Use CVE suppression in storm-cassandra project

2023-03-27 Thread via GitHub
rzo1 commented on PR #3525: URL: https://github.com/apache/storm/pull/3525#issuecomment-1486254072 No changes? -- 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

[GitHub] [storm] bipinprasad opened a new pull request, #3531: [STORM-3906] Remove org.mockito.internal.util.reflection.FieldSetter and set mockito.version to 3.11.2

2023-03-27 Thread via GitHub
bipinprasad opened a new pull request, #3531: URL: https://github.com/apache/storm/pull/3531 ## What is the purpose of the change *FieldSetter class has been deprecated in minor version of Mockito, replace with reflection API* ## How was the change tested *Build and

[GitHub] [storm] bipinprasad opened a new pull request, #3530: [STORM-3905] Replace anonymous class with lambda in storm-core commands

2023-03-27 Thread via GitHub
bipinprasad opened a new pull request, #3530: URL: https://github.com/apache/storm/pull/3530 ## What is the purpose of the change *Lambda is more concise and makes it easier to understand* ## How was the change tested *run tests* -- This is an automated message from

[GitHub] [storm] dependabot[bot] commented on pull request #3511: [STORM-3903] Bump commons-fileupload from 1.3.3 to 1.5

2023-03-27 Thread via GitHub
dependabot[bot] commented on PR #3511: URL: https://github.com/apache/storm/pull/3511#issuecomment-1485377407 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let

[GitHub] [storm] bipinprasad closed pull request #3511: [STORM-3903] Bump commons-fileupload from 1.3.3 to 1.5

2023-03-27 Thread via GitHub
bipinprasad closed pull request #3511: [STORM-3903] Bump commons-fileupload from 1.3.3 to 1.5 URL: https://github.com/apache/storm/pull/3511 -- 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] bipinprasad commented on pull request #3511: [STORM-3903] Bump commons-fileupload from 1.3.3 to 1.5

2023-03-27 Thread via GitHub
bipinprasad commented on PR #3511: URL: https://github.com/apache/storm/pull/3511#issuecomment-1485377310 duplicated by https://github.com/apache/storm/pull/3529. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [storm] bipinprasad opened a new pull request, #3529: [STORM-3903] Bump commons-fileupload from 1.3.3 to 1.5

2023-03-27 Thread via GitHub
bipinprasad opened a new pull request, #3529: URL: https://github.com/apache/storm/pull/3529 ## What is the purpose of the change *Based on dependabot PR that did not trigger a rebuild https://github.com/apache/storm/pull/3511 and add changes to license files* ## How was the

[GitHub] [storm] bipinprasad commented on pull request #3511: [STORM-3903] Bump commons-fileupload from 1.3.3 to 1.5

2023-03-27 Thread via GitHub
bipinprasad commented on PR #3511: URL: https://github.com/apache/storm/pull/3511#issuecomment-1485259829 @dependabot rebase -- 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.

[GitHub] [storm] dependabot[bot] opened a new pull request, #3528: Bump spring-core from 5.3.20 to 5.3.26 in /examples/storm-jms-examples

2023-03-23 Thread via GitHub
dependabot[bot] opened a new pull request, #3528: URL: https://github.com/apache/storm/pull/3528 Bumps [spring-core](https://github.com/spring-projects/spring-framework) from 5.3.20 to 5.3.26. Release notes Sourced from

[GitHub] [storm] bipinprasad merged pull request #3527: [STORM-3902] Print summary of differences between expected and actual licenses

2023-03-23 Thread via GitHub
bipinprasad merged PR #3527: URL: https://github.com/apache/storm/pull/3527 -- 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, #3527: [STORM-3902] Print summary of differences between expected and actual licenses

2023-03-22 Thread via GitHub
bipinprasad opened a new pull request, #3527: URL: https://github.com/apache/storm/pull/3527 in DEPENDENCY-LICENSES ## What is the purpose of the change *Usual update to the build process occurs in small increments, and therefore the license differences are usually small to

[GitHub] [storm] bipinprasad opened a new pull request, #3526: [STORM-3901] Upgrade Kryo to version 5.40 and remove reflectasm-shaded

2023-03-22 Thread via GitHub
bipinprasad opened a new pull request, #3526: URL: https://github.com/apache/storm/pull/3526 ## What is the purpose of the change *Kryo version 4 used reflectasm-shaded jar, which in turn uses reflection API that have been deprecated or removed from JDK11. Upgrde to Kryo verion

[GitHub] [storm] bipinprasad commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-17 Thread via GitHub
bipinprasad commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1474428057 @dependabot ignore this dependency -- 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] dependabot[bot] commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-17 Thread via GitHub
dependabot[bot] commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1474428090 OK, I won't notify you about org.yaml:snakeyaml again, unless you re-open this PR.  -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [storm] dependabot[bot] closed pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-17 Thread via GitHub
dependabot[bot] closed pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0 URL: https://github.com/apache/storm/pull/3516 -- 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 opened a new pull request, #3525: [STORM-3900] Use CVE suppression in storm-cassandra project

2023-03-17 Thread via GitHub
bipinprasad opened a new pull request, #3525: URL: https://github.com/apache/storm/pull/3525 ## What is the purpose of the change *Cassandra project specifically suppresses certain CVE's. In Storm project, snakeyaml dependency has been upgraded to version 2.0. However Cassandra uses

[GitHub] [storm] vesense merged pull request #3524: STORM-3899 - Migrate dev-tools travis scripts to "gitact"

2023-03-16 Thread via GitHub
vesense merged PR #3524: URL: https://github.com/apache/storm/pull/3524 -- 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] vesense commented on pull request #3524: STORM-3899 - Migrate dev-tools travis scripts to "gitact"

2023-03-16 Thread via GitHub
vesense commented on PR #3524: URL: https://github.com/apache/storm/pull/3524#issuecomment-1473066439 I re-triggered the GH actions, and all checks passed. Thanks @rzo1 I will merge this PR later. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [storm] bipinprasad commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-16 Thread via GitHub
bipinprasad commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1472603630 Created a Pull Request in Cassandra: https://github.com/apache/cassandra/pull/2224 -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [storm] bipinprasad commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-13 Thread via GitHub
bipinprasad commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1466451357 This dependabot has been rebased and shows only storm-cassandra pom.xml that needs snakeyaml update. When cassandra updates their snakeyaml dependency, then we can update with proper

[GitHub] [storm] rzo1 commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-13 Thread via GitHub
rzo1 commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1466328732 Think the cassandra guys need to update to snakeyaml 2.0 first :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [storm] bipinprasad commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-13 Thread via GitHub
bipinprasad commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1466316396 > Think we can close this one. Still need a proper version of Cassandra to close this one out. https://issues.apache.org/jira/browse/STORM-3900 -- This is an automated message

[GitHub] [storm] bipinprasad commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-13 Thread via GitHub
bipinprasad commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1466301257 @dependabot rebase -- 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.

[GitHub] [storm] rzo1 commented on pull request #3524: STORM-3899 - Migrate dev-tools travis scripts to "gitact"

2023-03-13 Thread via GitHub
rzo1 commented on PR #3524: URL: https://github.com/apache/storm/pull/3524#issuecomment-1465647409 @vesense @bipinprasad Can you re-trigger GH actions? -- 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

[GitHub] [storm] rzo1 commented on pull request #3524: STORM-3899 - Migrate dev-tools travis scripts to "gitact"

2023-03-13 Thread via GitHub
rzo1 commented on PR #3524: URL: https://github.com/apache/storm/pull/3524#issuecomment-1465647064 Note: Fail looks unrelated as 11 passes. I do not have the karma to re-trigger the CI. Note, that the same workflow passes on my fork pr: https://github.com/rzo1/storm/pull/4 -- This is an

[GitHub] [storm] rzo1 opened a new pull request, #3524: STORM-3899 - Migrate dev-tools travis scripts to "gitact"

2023-03-12 Thread via GitHub
rzo1 opened a new pull request, #3524: URL: https://github.com/apache/storm/pull/3524 ## What is the purpose of the change - Removes left-overs of Travis to GH migration ## How was the change tested - Via GitHub actions -- This is an automated message from the Apache

[GitHub] [storm] rzo1 commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-11 Thread via GitHub
rzo1 commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1464957669 Think we can close this one. -- 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] vesense merged pull request #3523: STORM-3894 - SnakeYAML 2.0

2023-03-10 Thread via GitHub
vesense merged PR #3523: URL: https://github.com/apache/storm/pull/3523 -- 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 #3522: STORM-3898 - Add Integration Test Github Action

2023-03-09 Thread via GitHub
bipinprasad merged PR #3522: URL: https://github.com/apache/storm/pull/3522 -- 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] rzo1 commented on pull request #3522: STORM-3898 - Add Integration Test Github Action

2023-03-09 Thread via GitHub
rzo1 commented on PR #3522: URL: https://github.com/apache/storm/pull/3522#issuecomment-1461803334 > Lets have this merged first. After fully functional github actions, the script paths and names can be tweaked: https://issues.apache.org/jira/browse/STORM-3899 The GitHub actions are fast

[GitHub] [storm] rzo1 commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-08 Thread via GitHub
rzo1 commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1461505858 > @rzo1 Yes. Just created a Jira with the affected components. Feel free to reassign to yourself. Thanks https://github.com/apache/storm/pull/3523 -- This is an automated message

[GitHub] [storm] rzo1 opened a new pull request, #3523: STORM-3894 - SnakeYAML 2.0

2023-03-08 Thread via GitHub
rzo1 opened a new pull request, #3523: URL: https://github.com/apache/storm/pull/3523 ## What is the purpose of the change - Upgrade to SnakeYAML 2.0 as suggested in https://github.com/apache/storm/pull/3516 - Fixes related compile errors due to removed default constructors -

[GitHub] [storm] bipinprasad commented on pull request #3522: STORM-3898 - Add Integration Test Github Action

2023-03-08 Thread via GitHub
bipinprasad commented on PR #3522: URL: https://github.com/apache/storm/pull/3522#issuecomment-1461277129 Lets have this merged first. After fully functions github actions, the script paths and names can be tweaked: https://issues.apache.org/jira/browse/STORM-3899 -- This is an

[GitHub] [storm] rzo1 commented on pull request #3522: STORM-3898 - Add Integration Test Github Action

2023-03-08 Thread via GitHub
rzo1 commented on PR #3522: URL: https://github.com/apache/storm/pull/3522#issuecomment-1461267121 > Thanks for adding and testing integration test. devtools travis directory and files should be renamed. Scripts sould also be renamed and one variable in then ..TRAVIS.. Separate Jira or

[GitHub] [storm] bipinprasad commented on pull request #3522: STORM-3898 - Add Integration Test Github Action

2023-03-08 Thread via GitHub
bipinprasad commented on PR #3522: URL: https://github.com/apache/storm/pull/3522#issuecomment-1460949759 Thanks for adding and testing integration test. devtools travis directory and files should be renamed. Scripts sould also be renamed and one variable in then ..TRAVIS.. Separate Jira

[GitHub] [storm] rzo1 opened a new pull request, #3522: STORM-3898 - Add Integration Test Github Action

2023-03-08 Thread via GitHub
rzo1 opened a new pull request, #3522: URL: https://github.com/apache/storm/pull/3522 ## What is the purpose of the change - This re-adds the integration test in GitHub Actions similar to the one previously executed in Travis CI. ## How was the change tested - Via

[GitHub] [storm] rzo1 closed pull request #3521: STORM-3894 - SnakeYAML 2.0

2023-03-08 Thread via GitHub
rzo1 closed pull request #3521: STORM-3894 - SnakeYAML 2.0 URL: https://github.com/apache/storm/pull/3521 -- 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,

[GitHub] [storm] rzo1 opened a new pull request, #3521: STORM-3894 - SnakeYAML 2.0

2023-03-08 Thread via GitHub
rzo1 opened a new pull request, #3521: URL: https://github.com/apache/storm/pull/3521 ## What is the purpose of the change *(Explain why we should have this change)* ## How was the change tested *(Explain what tests did you do to verify the code change)* -- This is an

[GitHub] [storm] bipinprasad commented on pull request #3518: [STORM-3895] Add JDK17

2023-03-08 Thread via GitHub
bipinprasad commented on PR #3518: URL: https://github.com/apache/storm/pull/3518#issuecomment-1460301386 Travis has been discontinued since Feb 15, 2023. jdk17 will be added in GitHub actions. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [storm] bipinprasad closed pull request #3518: [STORM-3895] Add JDK17

2023-03-08 Thread via GitHub
bipinprasad closed pull request #3518: [STORM-3895] Add JDK17 URL: https://github.com/apache/storm/pull/3518 -- 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,

[GitHub] [storm] bipinprasad merged pull request #3520: STORM-3897 Replace Travis with GitHub Actions

2023-03-08 Thread via GitHub
bipinprasad merged PR #3520: URL: https://github.com/apache/storm/pull/3520 -- 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 closed pull request #3519: Create maven.yml githug action

2023-03-07 Thread via GitHub
bipinprasad closed pull request #3519: Create maven.yml githug action URL: https://github.com/apache/storm/pull/3519 -- 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

[GitHub] [storm] bipinprasad commented on pull request #3519: Create maven.yml githug action

2023-03-07 Thread via GitHub
bipinprasad commented on PR #3519: URL: https://github.com/apache/storm/pull/3519#issuecomment-1458690894 Fixed in https://github.com/apache/storm/pull/3520 -- 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] bipinprasad commented on pull request #3520: STORM-3897 Replace Travis with GitHub Actions

2023-03-07 Thread via GitHub
bipinprasad commented on PR #3520: URL: https://github.com/apache/storm/pull/3520#issuecomment-1458685328 Agreed. No problem security-wise, and not an issue at this point. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [storm] rzo1 commented on pull request #3520: STORM-3897 Replace Travis with GitHub Actions

2023-03-07 Thread via GitHub
rzo1 commented on PR #3520: URL: https://github.com/apache/storm/pull/3520#issuecomment-1458679593 Thanks for the review! > Looks good. Thanks. It reverts a security fix (which was incompatible with jdk8). We will have to drop jdk8 and add jdk17 very soon. But till then, this will

[GitHub] [storm] bipinprasad commented on pull request #3520: STORM-3897 Replace Travis with GitHub Actions

2023-03-07 Thread via GitHub
bipinprasad commented on PR #3520: URL: https://github.com/apache/storm/pull/3520#issuecomment-1458677063 https://issues.apache.org/jira/browse/STORM-3898 for Integration test github action -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [storm] bipinprasad commented on pull request #3520: STORM-3897 Replace Travis with GitHub Actions

2023-03-07 Thread via GitHub
bipinprasad commented on PR #3520: URL: https://github.com/apache/storm/pull/3520#issuecomment-1458665342 Looks good. Thanks. It reverts a security fix (which was incompatible with jdk8). We will have to drop jdk8 and add jdk17 very soon. But till then, this will have to do. -- This is

[GitHub] [storm] rzo1 commented on pull request #3519: Create maven.yml githug action

2023-03-07 Thread via GitHub
rzo1 commented on PR #3519: URL: https://github.com/apache/storm/pull/3519#issuecomment-1457836925 see https://github.com/apache/storm/pull/3520 -- 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] rzo1 commented on pull request #3520: STORM-3897 Replace Travis with GitHub Actions

2023-03-07 Thread via GitHub
rzo1 commented on PR #3520: URL: https://github.com/apache/storm/pull/3520#issuecomment-1457836466 @bipinprasad Feel free to have a look at this PR. It should build wit 8 / 11 for most of the modules. I didn't touch the `Integration-Test` module atm as it requires some rewriting of the

[GitHub] [storm] rzo1 opened a new pull request, #3520: GitHub Action CI for Storm

2023-03-07 Thread via GitHub
rzo1 opened a new pull request, #3520: URL: https://github.com/apache/storm/pull/3520 ## What is the purpose of the change - Adds GitHub actions on the basis of the old travis script (and similar to them) - Downgrades TestNG as it isn't compatible with Java 8 - Fixes license

[GitHub] [storm] bipinprasad opened a new pull request, #3519: Create maven.yml githug action

2023-03-06 Thread via GitHub
bipinprasad opened a new pull request, #3519: URL: https://github.com/apache/storm/pull/3519 Migrating away from discontinued Travis ## Add a github action from sample *Travis builds no longer happen as of Feb 15, 2023* ## How was the change tested *Change has not

[GitHub] [storm] agresch merged pull request #3440: Storm-Pulsar bolt and spout

2023-03-06 Thread via GitHub
agresch merged PR #3440: URL: https://github.com/apache/storm/pull/3440 -- 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] rzo1 commented on pull request #3518: [STORM-3895] Add JDK17

2023-03-03 Thread via GitHub
rzo1 commented on PR #3518: URL: https://github.com/apache/storm/pull/3518#issuecomment-1453865638 @bipinprasad ASF removed support for Travis a few days ago. Think we need to migrate to GH actions or ASF Jenkins. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [storm] bipinprasad opened a new pull request, #3518: [STORM-3895] Add JDK17

2023-03-03 Thread via GitHub
bipinprasad opened a new pull request, #3518: URL: https://github.com/apache/storm/pull/3518 ## What is the purpose of the change *Add JDK17 build* ## How was the change tested *Run Travis* -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [storm] rzo1 commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-03 Thread via GitHub
rzo1 commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1453743062 "rzo1" :-) -- 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

[GitHub] [storm] bipinprasad commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-03 Thread via GitHub
bipinprasad commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1453729290 Hmm, somehow I don't find your name in the Apache Jira dropdown, searching with "rz" or Richard - another name perhaps? Created Jira with the Apache Infrastucture group:

[GitHub] [storm] rzo1 commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-03 Thread via GitHub
rzo1 commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1453704407 @bipinprasad I don't have the Karma to reassign the Jira but I am happy to have a look. Can you assign it to me? -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [storm] bipinprasad commented on pull request #3516: [STORM-3894] Bump snakeyaml from 1.32 to 2.0

2023-03-03 Thread via GitHub
bipinprasad commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1453697214 @rzo1 Yes. Just created a Jira with the affected components. Feel free to reassign to yourself. Thanks -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [storm] rzo1 commented on pull request #3516: Bump snakeyaml from 1.32 to 2.0

2023-03-03 Thread via GitHub
rzo1 commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1453683868 This most likely breaks current flux integration as constructors used are removed in 2.0 We need to manually adjust :-) -- This is an automated message from the Apache Git Service.

[GitHub] [storm] bipinprasad commented on pull request #3516: Bump snakeyaml from 1.32 to 2.0

2023-03-03 Thread via GitHub
bipinprasad commented on PR #3516: URL: https://github.com/apache/storm/pull/3516#issuecomment-1453662261 @dependabot rebase -- 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.

[GitHub] [storm] dependabot[bot] closed pull request #3514: Bump testng from 6.8.5 to 7.7.0 in /integration-test

2023-03-03 Thread via GitHub
dependabot[bot] closed pull request #3514: Bump testng from 6.8.5 to 7.7.0 in /integration-test URL: https://github.com/apache/storm/pull/3514 -- 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] dependabot[bot] commented on pull request #3514: Bump testng from 6.8.5 to 7.7.0 in /integration-test

2023-03-03 Thread via GitHub
dependabot[bot] commented on PR #3514: URL: https://github.com/apache/storm/pull/3514#issuecomment-1453595642 Looks like org.testng:testng is up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [storm] bipinprasad commented on pull request #3514: Bump testng from 6.8.5 to 7.7.0 in /integration-test

2023-03-03 Thread via GitHub
bipinprasad commented on PR #3514: URL: https://github.com/apache/storm/pull/3514#issuecomment-1453595242 @dependabot rebase -- 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.

[GitHub] [storm] bipinprasad merged pull request #3517: [STORM-3893] Bump testng from 6.8.5 to 7.7.0 in /integration-test

2023-03-03 Thread via GitHub
bipinprasad merged PR #3517: URL: https://github.com/apache/storm/pull/3517 -- 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, #3517: Bump testng from 6.8.5 to 7.7.0 in /integration-test

2023-03-03 Thread via GitHub
bipinprasad opened a new pull request, #3517: URL: https://github.com/apache/storm/pull/3517 Bumps [testng](https://github.com/cbeust/testng) from 6.8.5 to 7.7.0. - [Release notes](https://github.com/cbeust/testng/releases) -

[GitHub] [storm] dependabot[bot] opened a new pull request, #3516: Bump snakeyaml from 1.32 to 2.0

2023-03-03 Thread via GitHub
dependabot[bot] opened a new pull request, #3516: URL: https://github.com/apache/storm/pull/3516 Bumps [snakeyaml](https://bitbucket.org/snakeyaml/snakeyaml) from 1.32 to 2.0. Commits

[GitHub] [storm] bipinprasad merged pull request #3515: [STORM-3892] Bump testng from 6.8.5 to 7.7.0 in /examples/storm-starter

2023-03-03 Thread via GitHub
bipinprasad merged PR #3515: URL: https://github.com/apache/storm/pull/3515 -- 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] vesense commented on pull request #3440: Storm-Pulsar bolt and spout

2023-03-02 Thread via GitHub
vesense commented on PR #3440: URL: https://github.com/apache/storm/pull/3440#issuecomment-1452855236 Overall looks good to me. Thanks @dlg99 for the great job. And do you have any updates for @agresch and @bipinprasad comments? -- This is an automated message from the Apache Git

[GitHub] [storm] vesense commented on pull request #3510: [STORM-3884] Update calcite to 1.16.0

2023-03-02 Thread via GitHub
vesense commented on PR #3510: URL: https://github.com/apache/storm/pull/3510#issuecomment-1452841208 @bipinprasad Could you please have a look on the failed build? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [storm] dependabot[bot] opened a new pull request, #3515: Bump testng from 6.8.5 to 7.7.0 in /examples/storm-starter

2023-03-02 Thread via GitHub
dependabot[bot] opened a new pull request, #3515: URL: https://github.com/apache/storm/pull/3515 Bumps [testng](https://github.com/cbeust/testng) from 6.8.5 to 7.7.0. Release notes Sourced from https://github.com/cbeust/testng/releases;>testng's releases. TestNG v7.7.0

[GitHub] [storm] dependabot[bot] opened a new pull request, #3514: Bump testng from 6.8.5 to 7.7.0 in /integration-test

2023-03-02 Thread via GitHub
dependabot[bot] opened a new pull request, #3514: URL: https://github.com/apache/storm/pull/3514 Bumps [testng](https://github.com/cbeust/testng) from 6.8.5 to 7.7.0. Release notes Sourced from https://github.com/cbeust/testng/releases;>testng's releases. TestNG v7.7.0

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