bgeisberger commented on a change in pull request #4783:
URL: https://github.com/apache/nifi/pull/4783#discussion_r567076530



##########
File path: nifi-nar-bundles/nifi-redis-bundle/nifi-redis-extensions/pom.xml
##########
@@ -47,7 +47,7 @@
         <dependency>
             <groupId>redis.clients</groupId>
             <artifactId>jedis</artifactId>
-            <version>2.9.0</version>

Review comment:
       Yes, `2.9.3` has the fix in it as well and should work as well. 
   
   I chose `3.3.0` only because it was the direct dependency of the latest 
version of spring-data-redis. As far as I can see in the changelog 
(https://docs.spring.io/spring-data/data-redis/docs/2.3.4.RELEASE/reference/html/#new-features),
 spring-data-redis has no breaking changes to existing APIs. As jedis is only 
used via spring-data-redis, I wanted to make sure that there are no problems 
introduced by incompatibilities with newer jedis versions. 
   




----------------------------------------------------------------
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:
[email protected]


Reply via email to