storm-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [storm] Crim opened a new pull request #3314: [STORM-3680] Upgrade Jedis Library
Date Sun, 26 Jul 2020 02:07:30 GMT

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



Mime
View raw message