Guava HashFunction is preferred over CommandLongHasher

sazzad16 · sazzad16 · commit e5a7659452ae · 2024-03-10T16:47:11.000+06:00
diff --git a/src/main/java/redis/clients/jedis/csc/GuavaClientSideCache.java b/src/main/java/redis/clients/jedis/csc/GuavaClientSideCache.java
@@ -106,8 +106,8 @@ public GuavaClientSideCache build() {
 
       cb.expireAfterWrite(expireTime, expireTimeUnit);
 
-      return commandHasher != null ? new GuavaClientSideCache(cb.build(), commandHasher, cacheable)
-          : hashFunction != null ? new GuavaClientSideCache(cb.build(), new GuavaCommandHasher(hashFunction), cacheable)
+      return hashFunction != null ? new GuavaClientSideCache(cb.build(), new GuavaCommandHasher(hashFunction), cacheable)
+          : commandHasher != null ? new GuavaClientSideCache(cb.build(), commandHasher, cacheable)
               : new GuavaClientSideCache(cb.build(), cacheable);
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -106,8 +106,8 @@ public GuavaClientSideCache build() {`
`106`	`106`
`107`	`107`	`cb.expireAfterWrite(expireTime, expireTimeUnit);`
`108`	`108`
`109`		`- return commandHasher != null ? new GuavaClientSideCache(cb.build(), commandHasher, cacheable)`
`110`		`- : hashFunction != null ? new GuavaClientSideCache(cb.build(), new GuavaCommandHasher(hashFunction), cacheable)`
	`109`	`+ return hashFunction != null ? new GuavaClientSideCache(cb.build(), new GuavaCommandHasher(hashFunction), cacheable)`
	`110`	`+ : commandHasher != null ? new GuavaClientSideCache(cb.build(), commandHasher, cacheable)`
`111`	`111`	`: new GuavaClientSideCache(cb.build(), cacheable);`
`112`	`112`	`}`
`113`	`113`	`}`