[GitHub] [storm] bipinprasad opened a new pull request, #3510: [STORM-3884] Update calcite to 1.16.0

2023-01-12 Thread GitBox
bipinprasad opened a new pull request, #3510: URL: https://github.com/apache/storm/pull/3510 ## What is the purpose of the change *Dependabot recommendation is to bump up calcite to 1.32.0, however this is an intermediate step that compiles* ## How was the change tested

[GitHub] [storm] bipinprasad opened a new pull request, #3509: [STORM-3884] Bump calcite-core from 1.14.0 to 1.32.0

2023-01-10 Thread GitBox
bipinprasad opened a new pull request, #3509: URL: https://github.com/apache/storm/pull/3509 ## What is the purpose of the change *Recommended by Dependabot - however that PR fails to build* ## How was the change tested *mvn test* -- This is an automated message from t

[GitHub] [storm] xiayingfeng closed pull request #3435: Improve MAVEN build Performance

2023-01-09 Thread GitBox
xiayingfeng closed pull request #3435: Improve MAVEN build Performance URL: https://github.com/apache/storm/pull/3435 -- 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 unsubsc

[GitHub] [storm] vesense merged pull request #3508: [hotfix] fix typo

2023-01-09 Thread GitBox
vesense merged PR #3508: URL: https://github.com/apache/storm/pull/3508 -- 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: dev-unsubscr...@storm.apache.org

[GitHub] [storm] wuyouwuyoulian opened a new pull request, #3508: [hotfix] fix typo

2023-01-08 Thread GitBox
wuyouwuyoulian opened a new pull request, #3508: URL: https://github.com/apache/storm/pull/3508 ## What is the purpose of the change fix typo ## How was the change tested fix typo -- This is an automated message from the Apache Git Service. To respond to the messag

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

2023-01-08 Thread GitBox
skysiders commented on PR #3506: URL: https://github.com/apache/storm/pull/3506#issuecomment-1375017907 Hi @bipinprasad , could you please have a look at this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [storm] vesense merged pull request #3505: [STORM-3887] Add AWS Kafka command processor

2023-01-08 Thread GitBox
vesense merged PR #3505: URL: https://github.com/apache/storm/pull/3505 -- 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: dev-unsubscr...@storm.apache.org

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

2023-01-06 Thread GitBox
ScottBrenner commented on PR #35: URL: https://github.com/apache/storm-site/pull/35#issuecomment-1374223661 @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 specific

[GitHub] [storm] dependabot[bot] closed pull request #3504: [STORM-3889] Bump snakeyaml from 1.26 to 1.32

2023-01-05 Thread GitBox
dependabot[bot] closed pull request #3504: [STORM-3889] Bump snakeyaml from 1.26 to 1.32 URL: https://github.com/apache/storm/pull/3504 -- 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 c

[GitHub] [storm] dependabot[bot] commented on pull request #3504: [STORM-3889] Bump snakeyaml from 1.26 to 1.32

2023-01-05 Thread GitBox
dependabot[bot] commented on PR #3504: URL: https://github.com/apache/storm/pull/3504#issuecomment-1372429246 Looks like org.yaml:snakeyaml is no longer updatable, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [storm] bipinprasad commented on pull request #3504: [STORM-3889] Bump snakeyaml from 1.26 to 1.32

2023-01-05 Thread GitBox
bipinprasad commented on PR #3504: URL: https://github.com/apache/storm/pull/3504#issuecomment-1372428506 @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 #3507: [STORM-3889] Bump snakeyaml from 1.26 to 1.32

2023-01-05 Thread GitBox
bipinprasad merged PR #3507: URL: https://github.com/apache/storm/pull/3507 -- 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: dev-unsubscr...@storm.apache

[GitHub] [storm] bipinprasad commented on a diff in pull request #3497: fix(sec): upgrade org.fusesource.mqtt-client:mqtt-client to 1.15

2023-01-03 Thread GitBox
bipinprasad commented on code in PR #3497: URL: https://github.com/apache/storm/pull/3497#discussion_r1061040175 ## external/storm-mqtt/pom.xml: ## @@ -73,7 +73,7 @@ org.fusesource.mqtt-client mqtt-client -1.10 +1.15 R

[GitHub] [storm] bipinprasad commented on a diff in pull request #3497: fix(sec): upgrade org.fusesource.mqtt-client:mqtt-client to 1.15

2023-01-03 Thread GitBox
bipinprasad commented on code in PR #3497: URL: https://github.com/apache/storm/pull/3497#discussion_r1061039904 ## external/storm-mqtt/pom.xml: ## @@ -13,8 +14,7 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific lan

[GitHub] [storm] bipinprasad commented on a diff in pull request #3497: fix(sec): upgrade org.fusesource.mqtt-client:mqtt-client to 1.15

2023-01-03 Thread GitBox
bipinprasad commented on code in PR #3497: URL: https://github.com/apache/storm/pull/3497#discussion_r1061039453 ## examples/storm-mqtt-examples/pom.xml: ## @@ -13,9 +14,7 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the spe

[GitHub] [storm] bipinprasad opened a new pull request, #3507: [STORM-3889] Bump snakeyaml from 1.26 to 1.32

2023-01-03 Thread GitBox
bipinprasad opened a new pull request, #3507: URL: https://github.com/apache/storm/pull/3507 ## What is the purpose of the change *Dependabot recommendation* ## How was the change tested *Run unit and integration test* -- This is an automated message from the Apache Gi

[GitHub] [storm] skysiders opened a new pull request, #3506: STORM-3888 HdfsBlobStoreFile set wrong permission for file

2022-12-24 Thread GitBox
skysiders opened a new pull request, #3506: URL: https://github.com/apache/storm/pull/3506 ```java public OutputStream getOutputStream() throws IOException { FsPermission fileperms = new FsPermission(BLOBSTORE_FILE_PERMISSION); try { out = fileSystem

[GitHub] [storm] bipinprasad opened a new pull request, #3505: [STORM-3887] Add AWS Kafka command processor

2022-12-23 Thread GitBox
bipinprasad opened a new pull request, #3505: URL: https://github.com/apache/storm/pull/3505 ## What is the purpose of the change *Command processor to interact with AWS Kafka cluster* ## How was the change tested *Ran each command* -- This is an automated message from

[GitHub] [storm] dependabot[bot] closed pull request #3490: Bump snakeyaml from 1.26 to 1.31

2022-12-14 Thread GitBox
dependabot[bot] closed pull request #3490: Bump snakeyaml from 1.26 to 1.31 URL: https://github.com/apache/storm/pull/3490 -- 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 un

[GitHub] [storm] dependabot[bot] commented on pull request #3490: Bump snakeyaml from 1.26 to 1.31

2022-12-14 Thread GitBox
dependabot[bot] commented on PR #3490: URL: https://github.com/apache/storm/pull/3490#issuecomment-1352145129 Superseded by #3504. -- 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] dependabot[bot] opened a new pull request, #3504: Bump snakeyaml from 1.26 to 1.32

2022-12-14 Thread GitBox
dependabot[bot] opened a new pull request, #3504: URL: https://github.com/apache/storm/pull/3504 [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.yaml:snakeyaml&package-manager=maven&previous-version=1.26&new-ve

[GitHub] [storm] bipinprasad merged pull request #3503: Adding IgnoreUnrecognizedVMOptions to make worker start with jdk-11

2022-12-12 Thread GitBox
bipinprasad merged PR #3503: URL: https://github.com/apache/storm/pull/3503 -- 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: dev-unsubscr...@storm.apache

[GitHub] [storm] atulsm commented on pull request #3503: Adding IgnoreUnrecognizedVMOptions to make worker start with jdk-11

2022-12-11 Thread GitBox
atulsm commented on PR #3503: URL: https://github.com/apache/storm/pull/3503#issuecomment-1345868301 @bipinprasad , can you please merge to master if all looks good ? -- 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] atulsm commented on pull request #3503: Adding IgnoreUnrecognizedVMOptions to make worker start with jdk-11

2022-12-07 Thread GitBox
atulsm commented on PR #3503: URL: https://github.com/apache/storm/pull/3503#issuecomment-1341872275 @bipinprasad , can this be merged to 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 t

[GitHub] [storm] atulsm opened a new pull request, #3503: Adding IgnoreUnrecognizedVMOptions to make worker start with jdk-11

2022-12-07 Thread GitBox
atulsm opened a new pull request, #3503: URL: https://github.com/apache/storm/pull/3503 Issue: May of the default worker JVM parameters is no longer supported in java versions > 11. This will result in the worker process getting crashed. eg: https://github.com/31z4/storm-docker/issues/

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

2022-11-06 Thread GitBox
ScottBrenner closed pull request #35: Update meetups.yml URL: https://github.com/apache/storm-site/pull/35 -- 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-mai

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

2022-11-06 Thread GitBox
ScottBrenner commented on PR #35: URL: https://github.com/apache/storm-site/pull/35#issuecomment-1305097243 @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 specific

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

2022-10-26 Thread GitBox
shahidhs-ibm commented on PR #3492: URL: https://github.com/apache/storm/pull/3492#issuecomment-1293025974 Any update? -- 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 un

[GitHub] [storm] bipinprasad commented on pull request #3496: fix(sec): upgrade org.apache.thrift:libthrift to 0.14.0

2022-10-21 Thread GitBox
bipinprasad commented on PR #3496: URL: https://github.com/apache/storm/pull/3496#issuecomment-1287558076 This will need some code change: change class org.apache.storm.thrift.transport.TFramedTransport --> org.apache.storm.thrift.transport.layered.TFramedTransport And some othe

[GitHub] [storm] bipinprasad commented on a diff in pull request #3496: fix(sec): upgrade org.apache.thrift:libthrift to 0.14.0

2022-10-21 Thread GitBox
bipinprasad commented on code in PR #3496: URL: https://github.com/apache/storm/pull/3496#discussion_r1002255399 ## external/storm-hive/pom.xml: ## @@ -202,7 +201,7 @@ org.apache.thrift libthrift - 0.9.3 + 0.14.0 Review Comment: Can this use

[GitHub] [storm] bipinprasad commented on pull request #3498: fix(sec): upgrade org.elasticsearch:elasticsearch to 6.8.17

2022-10-21 Thread GitBox
bipinprasad commented on PR #3498: URL: https://github.com/apache/storm/pull/3498#issuecomment-1287523290 Probably needs some code change. https://app.travis-ci.com/github/apache/storm/jobs/585564257 line 6680 Error in external/storm-elasticsearch/src/test/java/org/apache/storm/elastic

[GitHub] [storm] bipinprasad commented on pull request #3501: [STORM-3885] fix(sec): upgrade com.fasterxml.jackson.core:jackson-databind to 2.12.6.1

2022-10-21 Thread GitBox
bipinprasad commented on PR #3501: URL: https://github.com/apache/storm/pull/3501#issuecomment-1287522460 Probably needs some code change. https://app.travis-ci.com/github/apache/storm/jobs/585564257 line 6680 Error in external/storm-elasticsearch/src/test/java/org/apache/storm/elastic

[GitHub] [storm] bipinprasad commented on pull request #3500: fix(sec): upgrade com.google.guava:guava to 30.0-jre

2022-10-21 Thread GitBox
bipinprasad commented on PR #3500: URL: https://github.com/apache/storm/pull/3500#issuecomment-1287511964 DEPENDENCY-LICENSES needs to be changed as well. License check failed https://app.travis-ci.com/github/apache/storm/jobs/585571698 line 3190 -- This is an automated message fro

[GitHub] [storm] bipinprasad commented on pull request #3494: fix(sec): upgrade commons-codec:commons-codec to 1.13

2022-10-21 Thread GitBox
bipinprasad commented on PR #3494: URL: https://github.com/apache/storm/pull/3494#issuecomment-1287508449 For license failures, you can check the output. There will be current file vs expected file. Diff will show what needs to be changed in DEPENDENCY-LICENSES file. -- This is an automa

[GitHub] [storm] bipinprasad commented on pull request #3494: fix(sec): upgrade commons-codec:commons-codec to 1.13

2022-10-21 Thread GitBox
bipinprasad commented on PR #3494: URL: https://github.com/apache/storm/pull/3494#issuecomment-1287507110 Seems like a good fix. Any reason why this PR should be closed? -- 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 #3501: [STORM-3885] fix(sec): upgrade com.fasterxml.jackson.core:jackson-databind to 2.12.6.1

2022-10-21 Thread GitBox
bipinprasad commented on PR #3501: URL: https://github.com/apache/storm/pull/3501#issuecomment-1287457886 https://mvnrepository.com/artifact/com.fasterxml.jackson.core/jackson-databind shows that 2.12.6.1 has two different vulnerabilities: Direct vulnerabilities: [CVE-2022-42004](htt

[GitHub] [storm] bipinprasad commented on pull request #3501: fix(sec): upgrade com.fasterxml.jackson.core:jackson-databind to 2.12.6.1

2022-10-21 Thread GitBox
bipinprasad commented on PR #3501: URL: https://github.com/apache/storm/pull/3501#issuecomment-1287441366 Looks like this needs some code change since the class com/fasterxml/jackson/core/util/JacksonFeature is not found. -- This is an automated message from the Apache Git Service. To res

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

2022-10-21 Thread GitBox
bipinprasad commented on PR #3491: URL: https://github.com/apache/storm/pull/3491#issuecomment-1287439674 Created Jira https://issues.apache.org/jira/browse/STORM-3884 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [storm] bipinprasad merged pull request #3502: [STORM-3883] Update Kryo to version 4.0.2

2022-10-21 Thread GitBox
bipinprasad merged PR #3502: URL: https://github.com/apache/storm/pull/3502 -- 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: dev-unsubscr...@storm.apache

[GitHub] [storm] 645775992 closed pull request #3494: fix(sec): upgrade commons-codec:commons-codec to 1.13

2022-10-20 Thread GitBox
645775992 closed pull request #3494: fix(sec): upgrade commons-codec:commons-codec to 1.13 URL: https://github.com/apache/storm/pull/3494 -- 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] closed pull request #31: Bump nokogiri from 1.13.3 to 1.13.6 in /content

2022-10-20 Thread GitBox
dependabot[bot] closed pull request #31: Bump nokogiri from 1.13.3 to 1.13.6 in /content URL: https://github.com/apache/storm-site/pull/31 -- 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 specifi

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

2022-10-20 Thread GitBox
dependabot[bot] closed pull request #32: Bump nokogiri from 1.13.3 to 1.13.6 URL: https://github.com/apache/storm-site/pull/32 -- 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. T

[GitHub] [storm-site] dependabot[bot] commented on pull request #31: Bump nokogiri from 1.13.3 to 1.13.6 in /content

2022-10-20 Thread GitBox
dependabot[bot] commented on PR #31: URL: https://github.com/apache/storm-site/pull/31#issuecomment-1286369395 Superseded by #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 specific comm

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

2022-10-20 Thread GitBox
dependabot[bot] commented on PR #32: URL: https://github.com/apache/storm-site/pull/32#issuecomment-1286369388 Superseded by #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 comm

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

2022-10-20 Thread GitBox
dependabot[bot] opened a new pull request, #37: URL: https://github.com/apache/storm-site/pull/37 Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.13.3 to 1.13.9. Release notes Sourced from https://github.com/sparklemotion/nokogiri/releases";>nokogiri's releases.

[GitHub] [storm-site] dependabot[bot] opened a new pull request, #36: Bump nokogiri from 1.13.3 to 1.13.9

2022-10-20 Thread GitBox
dependabot[bot] opened a new pull request, #36: URL: https://github.com/apache/storm-site/pull/36 Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.13.3 to 1.13.9. Release notes Sourced from https://github.com/sparklemotion/nokogiri/releases";>nokogiri's releases.

[GitHub] [storm] bipinprasad opened a new pull request, #3502: [STOM-3883] Update Kryo to version 4.0.2

2022-10-19 Thread GitBox
bipinprasad opened a new pull request, #3502: URL: https://github.com/apache/storm/pull/3502 - Update Maven Dependencies. - Update LICENSE files. - Exclude Kryo 3.0.3 dependency picked by the following dependency: +- org.apache.storm:storm-clojure:jar:2.5.0-SNAPSHOT

[GitHub] [storm-site] ScottBrenner opened a new pull request, #35: Update meetups.yml

2022-10-17 Thread GitBox
ScottBrenner opened a new pull request, #35: URL: https://github.com/apache/storm-site/pull/35 Seems these groups no longer exist, proposing deleting. -- 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] bitmasks closed pull request #3499: fix(sec): upgrade com.google.guava:guava to 30.0-jre

2022-10-15 Thread GitBox
bitmasks closed pull request #3499: fix(sec): upgrade com.google.guava:guava to 30.0-jre URL: https://github.com/apache/storm/pull/3499 -- 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 c

[GitHub] [storm] bitmasks commented on pull request #3499: fix(sec): upgrade com.google.guava:guava to 30.0-jre

2022-10-15 Thread GitBox
bitmasks commented on PR #3499: URL: https://github.com/apache/storm/pull/3499#issuecomment-1279893608 close -- 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] bitmasks closed pull request #3493: fix(sec): upgrade org.apache.kafka:kafka-clients to 2.7.2

2022-10-14 Thread GitBox
bitmasks closed pull request #3493: fix(sec): upgrade org.apache.kafka:kafka-clients to 2.7.2 URL: https://github.com/apache/storm/pull/3493 -- 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 speci

[GitHub] [storm] bitmasks commented on pull request #3493: fix(sec): upgrade org.apache.kafka:kafka-clients to 2.7.2

2022-10-14 Thread GitBox
bitmasks commented on PR #3493: URL: https://github.com/apache/storm/pull/3493#issuecomment-1279253357 close PR -- 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 unsubscri

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

2022-09-20 Thread GitBox
shahidhs-ibm commented on PR #3492: URL: https://github.com/apache/storm/pull/3492#issuecomment-1251940882 The build failures seems unrelated with the commit. Please re-trigger the build. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [storm] shahidhs-ibm opened a new pull request, #3492: Enable s390x builds on Travis

2022-09-19 Thread GitBox
shahidhs-ibm opened a new pull request, #3492: URL: https://github.com/apache/storm/pull/3492 Signed-off-by: Shahid Shaikh ## What is the purpose of the change Enable s390x builds on Travis. ## How was the change tested Changes are tested by running Travis [build

[GitHub] [storm] dependabot[bot] opened a new pull request, #3491: Bump calcite-core from 1.14.0 to 1.32.0

2022-09-16 Thread GitBox
dependabot[bot] opened a new pull request, #3491: URL: https://github.com/apache/storm/pull/3491 Bumps [calcite-core](https://github.com/apache/calcite) from 1.14.0 to 1.32.0. Commits https://github.com/apache/calcite/commit/413eded693a9087402cc1a6eefeca7a29445d337";>413eded [

[GitHub] [storm] dependabot[bot] opened a new pull request, #3490: Bump snakeyaml from 1.26 to 1.31

2022-09-14 Thread GitBox
dependabot[bot] opened a new pull request, #3490: URL: https://github.com/apache/storm/pull/3490 Bumps [snakeyaml](https://bitbucket.org/snakeyaml/snakeyaml) from 1.26 to 1.31. Commits https://bitbucket.org/snakeyaml/snakeyaml/commits/a3e641b8d6337d9916b5d709278fd668b0eb8507";>

[GitHub] [storm] YuBingZhou opened a new pull request, #3489: Test

2022-08-10 Thread GitBox
YuBingZhou opened a new pull request, #3489: URL: https://github.com/apache/storm/pull/3489 新增storm-kafka- client调试类 ## 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 veri

[GitHub] [storm-site] dependabot[bot] opened a new pull request, #34: Bump tzinfo from 1.2.4 to 1.2.10 in /content

2022-07-22 Thread GitBox
dependabot[bot] opened a new pull request, #34: URL: https://github.com/apache/storm-site/pull/34 Bumps [tzinfo](https://github.com/tzinfo/tzinfo) from 1.2.4 to 1.2.10. Release notes Sourced from https://github.com/tzinfo/tzinfo/releases";>tzinfo's releases. v1.2.10 Fi

[GitHub] [storm-site] dependabot[bot] opened a new pull request, #33: Bump tzinfo from 1.2.4 to 1.2.10

2022-07-21 Thread GitBox
dependabot[bot] opened a new pull request, #33: URL: https://github.com/apache/storm-site/pull/33 Bumps [tzinfo](https://github.com/tzinfo/tzinfo) from 1.2.4 to 1.2.10. Release notes Sourced from https://github.com/tzinfo/tzinfo/releases";>tzinfo's releases. v1.2.10 Fi

[GitHub] [storm] bipinprasad merged pull request #3488: STORM-3873: Remove junit4 dependency

2022-07-20 Thread GitBox
bipinprasad merged PR #3488: URL: https://github.com/apache/storm/pull/3488 -- 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: dev-unsubscr...@storm.apache

[GitHub] [storm] agresch commented on a diff in pull request #3488: STORM-3873: Remove junit4 dependency

2022-07-19 Thread GitBox
agresch commented on code in PR #3488: URL: https://github.com/apache/storm/pull/3488#discussion_r924587620 ## external/storm-hdfs/src/test/java/org/apache/storm/hdfs/testing/MiniDFSClusterExtensionClassLevel.java: ## @@ -52,6 +55,8 @@ public MiniDFSCluster getDfscluster() {

[GitHub] [storm] snikhil5 commented on a diff in pull request #3488: STORM-3873: Remove junit4 dependency

2022-07-19 Thread GitBox
snikhil5 commented on code in PR #3488: URL: https://github.com/apache/storm/pull/3488#discussion_r924583718 ## external/storm-hdfs/src/test/java/org/apache/storm/hdfs/testing/MiniDFSClusterExtensionClassLevel.java: ## @@ -52,6 +55,8 @@ public MiniDFSCluster getDfscluster() {

[GitHub] [storm] agresch commented on a diff in pull request #3488: STORM-3873: Remove junit4 dependency

2022-07-19 Thread GitBox
agresch commented on code in PR #3488: URL: https://github.com/apache/storm/pull/3488#discussion_r924548205 ## external/storm-hdfs/src/test/java/org/apache/storm/hdfs/testing/MiniDFSClusterExtensionClassLevel.java: ## @@ -52,6 +55,8 @@ public MiniDFSCluster getDfscluster() {

[GitHub] [storm] snikhil5 opened a new pull request, #3488: STORM-3873: Remove junit4 dependency

2022-07-18 Thread GitBox
snikhil5 opened a new pull request, #3488: URL: https://github.com/apache/storm/pull/3488 ## What is the purpose of the change Remove junit4 from storm-hdfs and storm-hdfs blobstore to completely remove junit4 from Storm. ## How was the change tested built locally

[GitHub] [storm] agresch merged pull request #3486: STORM-3875 remove topology gc opts from ThroughputVsLatency to allow …

2022-07-18 Thread GitBox
agresch merged PR #3486: URL: https://github.com/apache/storm/pull/3486 -- 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: dev-unsubscr...@storm.apache.org

[GitHub] [storm] agresch merged pull request #3487: Storm 3872 Excluding some unused packages to enable compiling with maven versions >= 3.8.1

2022-07-14 Thread GitBox
agresch merged PR #3487: URL: https://github.com/apache/storm/pull/3487 -- 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: dev-unsubscr...@storm.apache.org

[GitHub] [storm] snikhil5 opened a new pull request, #3487: Storm 3872 Excluding some unused packages to enable compiling with maven versions >= 3.8.1

2022-07-12 Thread GitBox
snikhil5 opened a new pull request, #3487: URL: https://github.com/apache/storm/pull/3487 ## What is the purpose of the change Excluding some unused packages to enable compiling with maven versions >= 3.8.1 ## How was the change tested built locally. -- This is an au

[GitHub] [storm] agresch opened a new pull request, #3486: STORM-3875 remove topology gc opts from ThroughputVsLatency to allow …

2022-07-12 Thread GitBox
agresch opened a new pull request, #3486: URL: https://github.com/apache/storm/pull/3486 …running on JDK11 ## What is the purpose of the change Allow running the topology on JDK11. ## How was the change tested Ran on JDK8 and JDK11 on internal clusters. -- This

[GitHub] [storm] agresch merged pull request #3485: MINOR: add blobstore documentation for workerRestart

2022-07-05 Thread GitBox
agresch merged PR #3485: URL: https://github.com/apache/storm/pull/3485 -- 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: dev-unsubscr...@storm.apache.org

[GitHub] [storm] agresch opened a new pull request, #3485: MINOR: add blobstore documentation for workerRestart

2022-06-30 Thread GitBox
agresch opened a new pull request, #3485: URL: https://github.com/apache/storm/pull/3485 ## What is the purpose of the change Adding documentation for workerRestart option in the blobstore documentation. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [storm] bipinprasad merged pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-06-16 Thread GitBox
bipinprasad merged PR #3479: URL: https://github.com/apache/storm/pull/3479 -- 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: dev-unsubscr...@storm.apache

[GitHub] [storm] bipinprasad merged pull request #3422: STORM-3805 Changing error to warn for retry update operations

2022-06-09 Thread GitBox
bipinprasad merged PR #3422: URL: https://github.com/apache/storm/pull/3422 -- 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: dev-unsubscr...@storm.apache

[GitHub] [storm] bipinprasad merged pull request #3475: [STORM-3855] Remove python2 support and move to python3.

2022-06-09 Thread GitBox
bipinprasad merged PR #3475: URL: https://github.com/apache/storm/pull/3475 -- 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: dev-unsubscr...@storm.apache

[GitHub] [storm] bipinprasad closed pull request #3414: DO NOT MERGE: change dist from trusty to xenial

2022-05-31 Thread GitBox
bipinprasad closed pull request #3414: DO NOT MERGE: change dist from trusty to xenial URL: https://github.com/apache/storm/pull/3414 -- 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 closed pull request #3458: [STORM-3832] Remove Junit4 dependency

2022-05-31 Thread GitBox
bipinprasad closed pull request #3458: [STORM-3832] Remove Junit4 dependency URL: https://github.com/apache/storm/pull/3458 -- 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 u

[GitHub] [storm] bipinprasad commented on pull request #3458: [STORM-3832] Remove Junit4 dependency

2022-05-31 Thread GitBox
bipinprasad commented on PR #3458: URL: https://github.com/apache/storm/pull/3458#issuecomment-1142991406 Replaced by PR https://github.com/apache/storm/pull/3479 -- 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] dependabot[bot] opened a new pull request, #3484: Bump hadoop-common from 2.10.1 to 3.2.3

2022-05-31 Thread GitBox
dependabot[bot] opened a new pull request, #3484: URL: https://github.com/apache/storm/pull/3484 Bumps hadoop-common from 2.10.1 to 3.2.3. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.apache.hadoop:hadoop

[GitHub] [storm] bipinprasad merged pull request #3460: [STORM-3841] Update hive version from 2.3.4 to 2.3.9

2022-05-31 Thread GitBox
bipinprasad merged PR #3460: URL: https://github.com/apache/storm/pull/3460 -- 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: dev-unsubscr...@storm.apache

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885714008 ## storm-server/src/test/java/org/apache/storm/security/auth/DefaultHttpCredentialsPluginTest.java: ## @@ -18,30 +18,32 @@ import javax.security.auth.Subject; import ja

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885713177 ## storm-server/src/test/java/org/apache/storm/daemon/supervisor/SlotTest.java: ## @@ -177,39 +174,39 @@ public void testLaunchContainerFromEmpty() throws Exception {

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885711253 ## storm-server/src/test/java/org/apache/storm/TickTupleTest.java: ## @@ -93,7 +92,7 @@ public void testTickTupleWorksWithSystemBolt() throws Exception {

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885709882 ## storm-client/test/jvm/org/apache/storm/streams/processors/CoGroupByKeyProcessorTest.java: ## @@ -18,24 +18,26 @@ import java.util.List; import java.util.Set; import

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885710004 ## storm-client/test/jvm/org/apache/storm/trident/TestTridentTopology.java: ## @@ -60,7 +61,7 @@ public void testGenBoltId() { Map cur = topology.get_bolts()

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885709553 ## storm-client/test/jvm/org/apache/storm/security/auth/authorizer/DRPCSimpleACLAuthorizerTest.java: ## @@ -68,101 +68,101 @@ private static ReqContext makeMockContext(St

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885709137 ## storm-client/test/jvm/org/apache/storm/grouping/ShuffleGroupingTest.java: ## @@ -95,24 +95,21 @@ public void testShuffleGroupMultiThreaded() throws InterruptedExcepti

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885708800 ## storm-client/test/jvm/org/apache/storm/grouping/LoadAwareShuffleGroupingTest.java: ## @@ -204,24 +204,21 @@ private void testLoadAwareShuffleGroupingWithEvenLoadMulti

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885708268 ## storm-client/test/jvm/org/apache/storm/TestConfigValidate.java: ## @@ -261,56 +250,57 @@ public void testWorkerChildoptsIsStringOrStringList() throws InvocationTarget

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885708005 ## sql/storm-sql-core/src/test/org/apache/storm/sql/TestStormSql.java: ## @@ -29,13 +29,14 @@ import org.apache.storm.sql.runtime.ISqlStreamsDataSource; import org.apac

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885707398 ## flux/flux-core/src/test/java/org/apache/storm/flux/multilang/MultilangEnvironmentTest.java: ## @@ -38,19 +38,19 @@ public class MultilangEnvironmentTest { public

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885707723 ## integration-test/src/test/java/org/apache/storm/st/tests/window/TumblingWindowTest.java: ## @@ -53,7 +51,7 @@ public void testTumbleCount(int tumbleSize) throws Except

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885706412 ## external/storm-jdbc/src/test/java/org/apache/storm/jdbc/common/UtilTest.java: ## @@ -46,20 +46,7 @@ public void testBasic() { @Test public void testError()

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885706029 ## external/storm-hdfs/src/test/java/org/apache/storm/hdfs/spout/TestHdfsSpout.java: ## @@ -419,45 +419,45 @@ public void testMultipleFileConsumption_Ack() throws Except

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885705645 ## external/storm-hdfs/src/test/java/org/apache/storm/hdfs/spout/TestHdfsSpout.java: ## @@ -419,45 +419,45 @@ public void testMultipleFileConsumption_Ack() throws Except

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885705401 ## external/storm-hdfs/src/test/java/org/apache/storm/hdfs/bolt/TestSequenceFileBolt.java: ## @@ -40,38 +40,30 @@ import org.apache.storm.tuple.Tuple; import org.apache

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-31 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885705109 ## external/storm-hdfs/src/test/java/org/apache/storm/hdfs/bolt/AvroGenericRecordBoltTest.java: ## @@ -72,24 +73,17 @@ public class AvroGenericRecordBoltTest { priva

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-30 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885141605 ## storm-client/test/jvm/org/apache/storm/dependency/DependencyUploaderTest.java: ## @@ -63,51 +63,55 @@ public class DependencyUploaderTest { private DependencyUplo

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-30 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885141228 ## storm-client/test/jvm/org/apache/storm/TestConfigValidate.java: ## @@ -822,20 +805,19 @@ public void TestImpersonationAclUserEntryValidator() throws InvocationTargetE

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-30 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885140762 ## external/storm-jdbc/src/test/java/org/apache/storm/jdbc/common/JdbcClientTest.java: ## @@ -88,24 +86,19 @@ public void testInsertConnectionError() { this.clie

[GitHub] [storm] snikhil5 commented on a diff in pull request #3479: [STORM-3832] Moving unit tests from junit4 to junit5

2022-05-30 Thread GitBox
snikhil5 commented on code in PR #3479: URL: https://github.com/apache/storm/pull/3479#discussion_r885140028 ## external/storm-jdbc/src/test/java/org/apache/storm/jdbc/common/JdbcClientTest.java: ## @@ -88,24 +86,19 @@ public void testInsertConnectionError() { this.clie

[GitHub] [storm] dependabot[bot] opened a new pull request, #3483: Bump elasticsearch from 2.4.4 to 6.8.17 in /examples/storm-elasticsearch-examples

2022-05-26 Thread GitBox
dependabot[bot] opened a new pull request, #3483: URL: https://github.com/apache/storm/pull/3483 Bumps [elasticsearch](https://github.com/elastic/elasticsearch) from 2.4.4 to 6.8.17. Release notes Sourced from https://github.com/elastic/elasticsearch/releases";>elasticsearch's rel

  1   2   3   4   5   6   7   8   9   10   >