Re: [PR] [STORM-4015][DOCS] Update the minimum Java version required to Java 11 (storm)

engrravijain commented on PR #3613: URL: https://github.com/apache/storm/pull/3613#issuecomment-1877183419 Welcome @rzo1! :) Yes, I have a jira account with username **engrravijain** -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] [STORM-4015][DOCS] Update the minimum Java version required to Java 11 (storm)

rzo1 commented on PR #3613: URL: https://github.com/apache/storm/pull/3613#issuecomment-1877179116 Thanks @engrravijain - will re-generate the website + close the Jira afterwards. If you have a Jira account, plz let me know, so I can assign the issue accordingly ;-) -- This is an

Re: [PR] [STORM-4015][DOCS] Update the minimum Java version required to Java 11 (storm)

jnioche merged PR #3613: URL: https://github.com/apache/storm/pull/3613 -- 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: [PR] [STORM-4015][DOCS] Update the minimum Java version required to Java 11 (storm)

engrravijain commented on PR #3613: URL: https://github.com/apache/storm/pull/3613#issuecomment-1877111485 @rzo1 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

[PR] [STORM-4015][DOCS] Update the minimum Java version required to Java 11 (storm)

engrravijain opened a new pull request, #3613: URL: https://github.com/apache/storm/pull/3613 ## What is the purpose of the change Update the documentation to reflect the minimum version of Java required. *(Explain why we should have this change)* Updated documentation

Re: [PR] [STORM-4016] KafkaSpout: moving to poll(Duration) due to deprecation of poll(long) (storm)

rzo1 commented on PR #3612: URL: https://github.com/apache/storm/pull/3612#issuecomment-1866000242 @reiabreu Great. Don't forget to assign the jira to you + set the fix version and close it :) -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] [STORM-4016] KafkaSpout: moving to poll(Duration) due to deprecation of poll(long) (storm)

reiabreu merged PR #3612: URL: https://github.com/apache/storm/pull/3612 -- 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: [PR] [STORM-4016] KafkaSpout: moving to poll(Duration) due to deprecation of poll(long) (storm)

reiabreu commented on PR #3612: URL: https://github.com/apache/storm/pull/3612#issuecomment-1865996752 @rzo1 Cheers for that, the linking process was incomplete, I have solved that now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] [STORM-4016] KafkaSpout: moving to poll(Duration) due to deprecation of poll(long) (storm)

rzo1 commented on PR #3612: URL: https://github.com/apache/storm/pull/3612#issuecomment-1865965588 @reiabreu You just need to link your ASF ID with GitHub via https://gitbox.apache.org/boxer/ (than it will automatically provide your GitHub account with the required permissions) -- This

Re: [PR] [STORM-4016] KafkaSpout: moving to poll(Duration) due to deprecation of poll(long) (storm)

reiabreu commented on PR #3612: URL: https://github.com/apache/storm/pull/3612#issuecomment-1865962703 @rzo1 I'm afraid I don't have write permissions -- 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

Re: [PR] [STORM-4016] KafkaSpout: moving to poll(Duration) due to deprecation of poll(long) (storm)

rzo1 commented on PR #3612: URL: https://github.com/apache/storm/pull/3612#issuecomment-1865695510 @reiabreu feel free to merge it yourself ;-) -- 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

Re: [PR] [STORM-4016] KafkaSpout: moving to poll(Duration) due to deprecation of poll(long) (storm)

reiabreu closed pull request #3611: [STORM-4016] KafkaSpout: moving to poll(Duration) due to deprecation of poll(long) URL: https://github.com/apache/storm/pull/3611 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] STORM-4009 - Regenerate using thrift 0.18.1 (storm)

rzo1 merged PR #3608: URL: https://github.com/apache/storm/pull/3608 -- 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

Re: [PR] STORM-4011 - Migrate Storm itself to use Commons Collections 4 (storm)

rzo1 merged PR #3610: URL: https://github.com/apache/storm/pull/3610 -- 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

Re: [PR] STORM-4010 - Reduce ASM Mess in dependency tree (storm)

rzo1 merged PR #3609: URL: https://github.com/apache/storm/pull/3609 -- 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

[PR] STORM-4011 - Migrate Storm itself to use Commons Collections 4 (storm)

rzo1 opened a new pull request, #3610: URL: https://github.com/apache/storm/pull/3610 ## What is the purpose of the change - Migrate to commons collections 4 inside storm. ## How was the change tested - CI -- This is an automated message from the Apache Git Service.

[PR] STORM-4009 - Regenerate using thrift 0.18.1 (storm)

rzo1 opened a new pull request, #3608: URL: https://github.com/apache/storm/pull/3608 ## What is the purpose of the change - Regenerate old thrift code with new thrift compiler for the sake of newer language features. ## How was the change tested - CI -- This is an

Re: [PR] STORM-4008 - Implement timed GH actions to publish SNAPSHOTS (storm)

rzo1 merged PR #3607: URL: https://github.com/apache/storm/pull/3607 -- 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

[PR] STORM-4008 - Implement timed GH actions to publish SNAPSHOTS (storm)

rzo1 opened a new pull request, #3607: URL: https://github.com/apache/storm/pull/3607 ## What is the purpose of the change With STORM-4006 we are now publishing the tar.gz / zip files every day to https://nightlies.apache.org/storm/ In addition, we should publish SNAPSHOT

Re: [PR] STORM-4006 - Implement timed GH actions to publish to nightlies.a.o (storm)

rzo1 merged PR #3606: URL: https://github.com/apache/storm/pull/3606 -- 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

Re: [PR] [STORM-3565] add support for adding dimensions to metric data points (storm)

rzo1 merged PR #3194: URL: https://github.com/apache/storm/pull/3194 -- 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

[PR] STORM-4006 - Implement timed GH actions to publish to nightlies.a.o (storm)

rzo1 opened a new pull request, #3606: URL: https://github.com/apache/storm/pull/3606 ## What is the purpose of the change Publish a nightlies build once a day to nightlies.a.o ## How was the change tested Actually, I couldn't test it because the credentials are only

Re: [PR] STORM-3186: Customizable configuration for metric reporting interval (storm)

rzo1 merged PR #3132: URL: https://github.com/apache/storm/pull/3132 -- 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

Re: [PR] STORM-4004 - Upgrade Kafka Clients to 3.6.0 (storm)

rzo1 merged PR #3604: URL: https://github.com/apache/storm/pull/3604 -- 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

Re: [PR] [STORM-3639] Replace asserts in daemon code. (storm)

jnioche merged PR #3274: URL: https://github.com/apache/storm/pull/3274 -- 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: [PR] [STORM-3152]Storm has supported ipv6 but Troubleshooting.md didn't update (storm)

rzo1 commented on PR #2765: URL: https://github.com/apache/storm/pull/2765#issuecomment-1838450021 We are currently cleaning up old issues with stale discussions. The last comment here was made 5 years ago and Storm as evolved. If this is still an issue or relevant for your work, feel free

Re: [PR] [STORM-3152]Storm has supported ipv6 but Troubleshooting.md didn't update (storm)

rzo1 closed pull request #2765: [STORM-3152]Storm has supported ipv6 but Troubleshooting.md didn't update URL: https://github.com/apache/storm/pull/2765 -- 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

Re: [PR] STORM-3173: flush metrics to ScheduledReporter on shutdown (storm)

rzo1 closed pull request #2789: STORM-3173: flush metrics to ScheduledReporter on shutdown URL: https://github.com/apache/storm/pull/2789 -- 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

Re: [PR] STORM-3173: flush metrics to ScheduledReporter on shutdown (storm)

rzo1 commented on PR #2789: URL: https://github.com/apache/storm/pull/2789#issuecomment-1838446194 I am going to close this PR as #2764 was also closed after a list discussion. In addition, we are currently cleaning up old issues with stale discussions. The last comment here was made 5

Re: [PR] Bug fix for BaseResourceAwareStrategy to avoid NullPointerException (storm)

rzo1 commented on PR #2774: URL: https://github.com/apache/storm/pull/2774#issuecomment-1838441720 We are currently cleaning up old issues with stale discussions. The last comment here was made 5 years ago and Storm as evolved. If this is still an issue or relevant for your work, feel free

Re: [PR] Bug fix for BaseResourceAwareStrategy to avoid NullPointerException (storm)

rzo1 closed pull request #2774: Bug fix for BaseResourceAwareStrategy to avoid NullPointerException URL: https://github.com/apache/storm/pull/2774 -- 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

Re: [PR] [STORM-3291]Worker can't run as the user who submitted the topology (storm)

rzo1 commented on PR #2915: URL: https://github.com/apache/storm/pull/2915#issuecomment-1838440642 We are currently cleaning up old issues with stale discussions. The last comment here was made 5 years ago and Storm as evolved. If this is still an issue or relevant for your work, feel free

Re: [PR] [STORM-3291]Worker can't run as the user who submitted the topology (storm)

rzo1 closed pull request #2915: [STORM-3291]Worker can't run as the user who submitted the topology URL: https://github.com/apache/storm/pull/2915 -- 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

Re: [PR] fix(JDBC): Syntax error on INSERT (storm)

rzo1 commented on PR #2883: URL: https://github.com/apache/storm/pull/2883#issuecomment-1838438266 We are currently cleaning up old issues with stale discussions. The last comment here was made 5 years ago and Storm as evolved. If this is still an issue or relevant for your work, feel free

Re: [PR] fix(JDBC): Syntax error on INSERT (storm)

rzo1 closed pull request #2883: fix(JDBC): Syntax error on INSERT URL: https://github.com/apache/storm/pull/2883 -- 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

Re: [PR] [STORM-2773]If a drpcserver node in cluster is down,drpc cluster won't work if we don't modify the drpc.server configuration and restart the cluster (storm)

rzo1 commented on PR #2365: URL: https://github.com/apache/storm/pull/2365#issuecomment-1838435409 We are currently cleaning up old issues with stale discussions. The last comment here was made 5 years ago and Storm as evolved. If this is still an issue or relevant for your work, feel free

Re: [PR] [STORM-2773]If a drpcserver node in cluster is down,drpc cluster won't work if we don't modify the drpc.server configuration and restart the cluster (storm)

rzo1 closed pull request #2365: [STORM-2773]If a drpcserver node in cluster is down,drpc cluster won't work if we don't modify the drpc.server configuration and restart the cluster URL: https://github.com/apache/storm/pull/2365 -- This is an automated message from the Apache Git Service. To

Re: [PR] Add the getName() method in order to obtain the applied label to the Trident stream. (storm)

rzo1 commented on PR #2792: URL: https://github.com/apache/storm/pull/2792#issuecomment-1838428972 Fixed with https://github.com/apache/storm/commit/c158dee8c12aa6547c8037596c797da2f48c348d -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Add the getName() method in order to obtain the applied label to the Trident stream. (storm)

rzo1 closed pull request #2792: Add the getName() method in order to obtain the applied label to the Trident stream. URL: https://github.com/apache/storm/pull/2792 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [STORM-3132] Avoid NPE in the Values Constructor (storm)

rzo1 merged PR #2744: URL: https://github.com/apache/storm/pull/2744 -- 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

Re: [PR] STORM-3759 Additonal Trident Kafka Spout Metrics (storm)

rzo1 commented on PR #3385: URL: https://github.com/apache/storm/pull/3385#issuecomment-1838382063 @avermeer Would that be interesting for your topologies? -- 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

Re: [PR] [STORM-3680] Upgrade Jedis Library (storm)

rzo1 merged PR #3314: URL: https://github.com/apache/storm/pull/3314 -- 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

Re: [PR] [STORM-3680] Upgrade Jedis Library (storm)

Crim commented on PR #3314: URL: https://github.com/apache/storm/pull/3314#issuecomment-1838120113 oh snap, is this finally get a 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

Re: [PR] STORM-4004 - Upgrade Kafka Clients to 3.6.0 (storm)

rzo1 commented on PR #3604: URL: https://github.com/apache/storm/pull/3604#issuecomment-1838007051 It was for other potential other interested readers ;-) -- 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

Re: [PR] STORM-4004 - Upgrade Kafka Clients to 3.6.0 (storm)

avermeer commented on PR #3604: URL: https://github.com/apache/storm/pull/3604#issuecomment-1838006084 > > I'm a bit confused by kafka_2.13 references vs goal to update to 3.6.0, can you clarify ? > > Just to write it here to avoid having to look into resolved comments: The _2.x

Re: [PR] STORM-4004 - Upgrade Kafka Clients to 3.6.0 (storm)

rzo1 commented on PR #3604: URL: https://github.com/apache/storm/pull/3604#issuecomment-1838001659 > I'm a bit confused by kafka_2.13 references vs goal to update to 3.6.0, can you clarify ? Just to write it here to avoid having to look into resolved comments: The _2.x denotes the

Re: [PR] STORM-4005 - ElasticSearch 7.17.13 (storm)

rzo1 merged PR #3605: URL: https://github.com/apache/storm/pull/3605 -- 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

[PR] STORM-4005 - ElasticSearch 7.17.13 (storm)

rzo1 opened a new pull request, #3605: URL: https://github.com/apache/storm/pull/3605 ## What is the purpose of the change - Upgrade legacy ES version - License shouldn't be a big problem because we do not distribute it in our binaries. - License check is expected to fail and

Re: [PR] STORM-4004 - Upgrade Kafka Clients to 3.6.0 (storm)

rzo1 commented on code in PR #3604: URL: https://github.com/apache/storm/pull/3604#discussion_r1409361181 ## external/storm-kafka-client/pom.xml: ## @@ -130,7 +130,7 @@ org.apache.kafka -kafka_2.11 +kafka_2.13 Review

Re: [PR] STORM-4004 - Upgrade Kafka Clients to 3.6.0 (storm)

avermeer commented on code in PR #3604: URL: https://github.com/apache/storm/pull/3604#discussion_r1409314648 ## external/storm-kafka-client/pom.xml: ## @@ -130,7 +130,7 @@ org.apache.kafka -kafka_2.11 +kafka_2.13 Review

Re: [PR] STORM-4004 - Upgrade Kafka Clients to 3.6.0 (storm)

rzo1 commented on code in PR #3604: URL: https://github.com/apache/storm/pull/3604#discussion_r1409312271 ## external/storm-kafka-client/pom.xml: ## @@ -110,7 +110,7 @@ org.apache.kafka -kafka_2.11 +kafka_2.13 Review

Re: [PR] STORM-4004 - Upgrade Kafka Clients to 3.6.0 (storm)

avermeer commented on code in PR #3604: URL: https://github.com/apache/storm/pull/3604#discussion_r1409308281 ## external/storm-kafka-client/pom.xml: ## @@ -110,7 +110,7 @@ org.apache.kafka -kafka_2.11 +kafka_2.13 Review

Re: [PR] STORM-4004 - Upgrade Kafka Clients to 3.6.0 (storm)

avermeer commented on code in PR #3604: URL: https://github.com/apache/storm/pull/3604#discussion_r1409308866 ## external/storm-kafka-client/pom.xml: ## @@ -130,7 +130,7 @@ org.apache.kafka -kafka_2.11 +kafka_2.13 Review

Re: [PR] STORM-4004 - Upgrade Kafka Clients to 3.6.0 (storm)

avermeer commented on code in PR #3604: URL: https://github.com/apache/storm/pull/3604#discussion_r1409305813 ## DEPENDENCY-LICENSES: ## @@ -127,7 +127,7 @@ List of third-party dependencies grouped by their license type. * Apache HttpCore

Re: [PR] [STORM-3884] Bump calcite-core from 1.14.0 to 1.32.0 (storm)

rzo1 commented on PR #3509: URL: https://github.com/apache/storm/pull/3509#issuecomment-1831607030 There are also other migration issues due to removed / deprecated stuff in newer calcite versions. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] fix flaky tests testTopologyIsolation and testDefaultResourceAwareStrategySharedMemory (storm)

rzo1 commented on PR #3603: URL: https://github.com/apache/storm/pull/3603#issuecomment-1825240342 These tests didn't fail in the last 100+ runs. Can we adjust the test cases instead of changing the set impl? -- This is an automated message from the Apache Git Service. To respond to the

[PR] fix flaky tests testTopologyIsolation and testDefaultResourceAwareStrategySharedMemory (storm)

ThugJudy opened a new pull request, #3603: URL: https://github.com/apache/storm/pull/3603 ## What is the purpose of the change The following test cases

Re: [PR] Release of Storm 2.6.0 (storm)

rzo1 merged PR #3602: URL: https://github.com/apache/storm/pull/3602 -- 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

[PR] Release of Storm 2.6.0 (storm)

rzo1 opened a new pull request, #3602: URL: https://github.com/apache/storm/pull/3602 ... -- 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: [PR] [STORM-4003] add missing jakarta.xml.bind dependency to storm-kafka-monitor (storm)

avermeer commented on PR #3601: URL: https://github.com/apache/storm/pull/3601#issuecomment-1812763914 okay tests are passing now :) -- 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

Re: [PR] [STORM-4003] add missing jakarta.xml.bind dependency to storm-kafka-monitor (storm)

rzo1 merged PR #3601: URL: https://github.com/apache/storm/pull/3601 -- 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

Re: [PR] [STORM-4003] add missing jakarta.xml.bind dependency to storm-kafka-monitor (storm)

rzo1 commented on PR #3601: URL: https://github.com/apache/storm/pull/3601#issuecomment-1812727234 Most likely flaky test (as it passes on 17,21). Retriggered. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [STORM-4003] add missing jakarta.xml.bind dependency to storm-kafka-monitor (storm)

avermeer commented on PR #3601: URL: https://github.com/apache/storm/pull/3601#issuecomment-1812715590 i see that there's an [integration error](https://github.com/apache/storm/actions/runs/6878858809/job/18709465696?pr=3601 ) , but I don't understand how it could relate to my PR. Any

[PR] STORM-4003: add missing jakarta.xml.bind dependency to storm-kafka-monitor (storm)

avermeer opened a new pull request, #3601: URL: https://github.com/apache/storm/pull/3601 ## What is the purpose of the change When running Storm UI with Java 17 runtime, the lag of Kafka consumers is not visible in the topology view for the topologies which are consuming Kafka

Re: [PR] STORM-4001 - log4j2 2.21.1 (storm)

jnioche merged PR #3600: URL: https://github.com/apache/storm/pull/3600 -- 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:

[PR] STORM-4001 - log4j2 2.21.1 (storm)

rzo1 opened a new pull request, #3600: URL: https://github.com/apache/storm/pull/3600 ## What is the purpose of the change - log4j2 2.21.1 ## How was the change tested - CI -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] STORM-3999 - Add Java 21 to Build Matrix (storm)

rzo1 merged PR #3599: URL: https://github.com/apache/storm/pull/3599 -- 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

Re: [PR] Draft: Java 21 (CI results) (storm)

avermeer commented on PR #3599: URL: https://github.com/apache/storm/pull/3599#issuecomment-1803908138 Huge +1 ! -- 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

[PR] Draft: Java 21 (CI results) (storm)

rzo1 opened a new pull request, #3599: URL: https://github.com/apache/storm/pull/3599 See what happens with CI if we add Java 21 to the matrix -- 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

Re: [PR] STORM-3994 - LocalCluster init fails on Windows because of missing librocksdbjni-win64.dll (storm)

jnioche merged PR #3598: URL: https://github.com/apache/storm/pull/3598 -- 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: [PR] fix(sec): upgrade org.elasticsearch:elasticsearch to 6.8.17 (storm)

rzo1 closed pull request #3498: fix(sec): upgrade org.elasticsearch:elasticsearch to 6.8.17 URL: https://github.com/apache/storm/pull/3498 -- 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

Re: [PR] fix(sec): upgrade org.elasticsearch:elasticsearch to 6.8.17 (storm)

rzo1 commented on PR #3498: URL: https://github.com/apache/storm/pull/3498#issuecomment-1803774980 This needs more work. As the PR is stale for > 1y now, I am going to close it. Feel free to re-open. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] [STORM-3938] Unhandled InterruptedException and missing log in Supervisor's close(… (storm)

rzo1 closed pull request #3554: [STORM-3938] Unhandled InterruptedException and missing log in Supervisor's close(… URL: https://github.com/apache/storm/pull/3554 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [STORM-3938] Unhandled InterruptedException and missing log in Supervisor's close(… (storm)

rzo1 commented on PR #3554: URL: https://github.com/apache/storm/pull/3554#issuecomment-1803772348 IDE reports, that this exception isn't thrown at the place the catch block was added. I am closing this PR now. -- This is an automated message from the Apache Git Service. To respond to

Re: [PR] [STORM-3996] [STORM-3997] [STORM-3998] Dependency Updates (storm)

rzo1 merged PR #3596: URL: https://github.com/apache/storm/pull/3596 -- 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

[PR] STORM-3994 - LocalCluster init fails on Windows because of missing li… (storm)

rzo1 opened a new pull request, #3598: URL: https://github.com/apache/storm/pull/3598 ## What is the purpose of the change - Updates rocksdbjni as 8.1.1 is a broken release regarding Windows (cf. https://github.com/facebook/rocksdb/issues/11420) ## How was the change tested

Re: [PR] [STORM-3986] Get rid of BlacklistScheduler timer [INFO] logs (storm)

rzo1 merged PR #3597: URL: https://github.com/apache/storm/pull/3597 -- 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

Re: [PR] [STORM-3986] Get rid of BlacklistScheduler timer [INFO] logs (storm)

avermeer commented on code in PR #3597: URL: https://github.com/apache/storm/pull/3597#discussion_r1387937669 ## storm-server/src/main/java/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java: ## @@ -194,7 +194,11 @@ private Set refreshBlacklistedSupervisorIds(Cluster

Re: [PR] [STORM-3986] Get rid of BlacklistScheduler timer [INFO] logs (storm)

rzo1 commented on code in PR #3597: URL: https://github.com/apache/storm/pull/3597#discussion_r1387935644 ## storm-server/src/main/java/org/apache/storm/scheduler/blacklist/BlacklistScheduler.java: ## @@ -194,7 +194,11 @@ private Set refreshBlacklistedSupervisorIds(Cluster

Re: [PR] [STORM-3996] [STORM-3997] [STORM-3998] Dependency Updates (storm)

rzo1 commented on code in PR #3596: URL: https://github.com/apache/storm/pull/3596#discussion_r1387934649 ## DEPENDENCY-LICENSES: ## @@ -455,33 +464,33 @@ List of third-party dependencies grouped by their license type. Apache License, Version 2.0, Eclipse Public License

Re: [PR] [STORM-3996] [STORM-3997] [STORM-3998] Dependency Updates (storm)

avermeer commented on code in PR #3596: URL: https://github.com/apache/storm/pull/3596#discussion_r1387931491 ## DEPENDENCY-LICENSES: ## @@ -455,33 +464,33 @@ List of third-party dependencies grouped by their license type. Apache License, Version 2.0, Eclipse Public

[PR] [STORM-3986] Get rid of BlacklistScheduler timer [INFO] logs (storm)

rzo1 opened a new pull request, #3597: URL: https://github.com/apache/storm/pull/3597 ## What is the purpose of the change https://issues.apache.org/jira/projects/STORM/issues/STORM-3986 -- This is an automated message from the Apache Git Service. To respond to the message,

[PR] WIP: Several dependency updates (storm)

rzo1 opened a new pull request, #3596: URL: https://github.com/apache/storm/pull/3596 Note: Currently WIP to ensure CI is ok. Will create tickets later. ## What is the purpose of the change Updates - Guava 32.1.3-jre - Netty 4.1.100 - Jetty 9.4.53.v20231009

Re: [PR] STORM-3995 - commons-text 1.11.0 (storm)

rzo1 merged PR #3595: URL: https://github.com/apache/storm/pull/3595 -- 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

[PR] STORM-3995 - commons-text 1.11.0 (storm)

rzo1 opened a new pull request, #3595: URL: https://github.com/apache/storm/pull/3595 ## What is the purpose of the change - add commons text to dependency management section ## How was the change tested - ci -- This is an automated message from the Apache Git

Re: [PR] [STORM-3949] libthrift 0.18.1 (storm)

avermeer commented on PR #3564: URL: https://github.com/apache/storm/pull/3564#issuecomment-1799118025 I have been running my pre-production topologies with Storm 2.6.0 SNAPSHOT build of the 2023/11/03, with a flow ~1000 tuples/minutes since 2 days,and so far my metrics on latency,

Re: [PR] [STORM-3949] libthrift 0.18.1 (storm)

rzo1 commented on PR #3564: URL: https://github.com/apache/storm/pull/3564#issuecomment-1796531951 @agresch think we will see after a 2.6.0 ist available because I doubt anyone is using the Last SNAPSHOT build in a staging environment ,et. -- This is an automated message from the Apache

Re: [PR] [STORM-3949] libthrift 0.18.1 (storm)

agresch commented on PR #3564: URL: https://github.com/apache/storm/pull/3564#issuecomment-1796384706 When storm upgraded to thrift 0.13 initially, we would see huge CPU spikes on supervisors and nimbus after approximately 24 hours for unknown reasons. The problem went away when we backed

Re: [PR] STORM-3993 - ActiveMQ 5.18.3 (storm)

rzo1 merged PR #3594: URL: https://github.com/apache/storm/pull/3594 -- 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

Re: [PR] Bump org.elasticsearch:elasticsearch from 2.4.4 to 7.17.13 in /examples/storm-elasticsearch-examples (storm)

dependabot[bot] commented on PR #3591: URL: https://github.com/apache/storm/pull/3591#issuecomment-1790560680 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

Re: [PR] Bump org.elasticsearch:elasticsearch from 2.4.4 to 7.17.13 in /examples/storm-elasticsearch-examples (storm)

rzo1 closed pull request #3591: Bump org.elasticsearch:elasticsearch from 2.4.4 to 7.17.13 in /examples/storm-elasticsearch-examples URL: https://github.com/apache/storm/pull/3591 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Bump org.elasticsearch:elasticsearch from 2.4.4 to 7.17.13 in /examples/storm-elasticsearch-examples (storm)

rzo1 commented on PR #3591: URL: https://github.com/apache/storm/pull/3591#issuecomment-1790560615 similar to https://github.com/apache/storm/pull/3591 -- 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

Re: [PR] Bump org.apache.activemq:activemq-client from 5.18.2 to 5.18.3 (storm)

dependabot[bot] commented on PR #3592: URL: https://github.com/apache/storm/pull/3592#issuecomment-1790559329 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

Re: [PR] Bump org.apache.activemq:activemq-client from 5.18.2 to 5.18.3 (storm)

rzo1 closed pull request #3592: Bump org.apache.activemq:activemq-client from 5.18.2 to 5.18.3 URL: https://github.com/apache/storm/pull/3592 -- 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

[PR] STORM-3993 - ActiveMQ 5.18.3 (storm)

rzo1 opened a new pull request, #3594: URL: https://github.com/apache/storm/pull/3594 ## What is the purpose of the change - ## How was the change tested - CI/CD -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [STORM-3992]Print stack trace upon Exception when getting heartbeat timeout (storm)

jnioche merged PR #3593: URL: https://github.com/apache/storm/pull/3593 -- 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:

[PR] [STORM-3992] Fix log line to print stack trace upon Exception when tting heartbeat timeout (storm)

bipinprasad opened a new pull request, #3593: URL: https://github.com/apache/storm/pull/3593 ## 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)*

[PR] Bump org.apache.activemq:activemq-client from 5.18.2 to 5.18.3 (storm)

dependabot[bot] opened a new pull request, #3592: URL: https://github.com/apache/storm/pull/3592 Bumps [org.apache.activemq:activemq-client](https://github.com/apache/activemq) from 5.18.2 to 5.18.3. Commits

[PR] Bump org.elasticsearch:elasticsearch from 2.4.4 to 7.17.13 in /examples/storm-elasticsearch-examples (storm)

dependabot[bot] opened a new pull request, #3591: URL: https://github.com/apache/storm/pull/3591 Bumps [org.elasticsearch:elasticsearch](https://github.com/elastic/elasticsearch) from 2.4.4 to 7.17.13. Release notes Sourced from

Re: [PR] Revert "[STORM-3764] Fix NPE in SchedulingSearcherState.backtrack()" (storm)

jnioche closed pull request #3590: Revert "[STORM-3764] Fix NPE in SchedulingSearcherState.backtrack()" URL: https://github.com/apache/storm/pull/3590 -- 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

<    1   2   3   4   5   6   7   >