Crim opened a new pull request #3314:
URL: https://github.com/apache/storm/pull/3314
## What is the purpose of the change
To support future improvements to the storm-redis package, lets upgrade its underlying
library (Jedis) it uses to communicate with Redis.
Upgrading will clear the way for [STORM-3665](https://issues.apache.org/jira/browse/STORM-3665)
as Stream support in the Jedis library was not added until version 3.x
## How was the change tested
As no previous test coverage existed for the bolts provided by `storm-redis` I've started
to add integration tests using TestContainers. Let me know if this is an acceptable way to
provide test coverage and I will build out tests for the remaining bolts.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
|