[ 
https://issues.apache.org/jira/browse/GEODE-8786?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17259935#comment-17259935
 ] 

ASF GitHub Bot commented on GEODE-8786:
---------------------------------------

nonbinaryprogrammer commented on a change in pull request #5875:
URL: https://github.com/apache/geode/pull/5875#discussion_r552860024



##########
File path: 
geode-redis/src/main/java/org/apache/geode/redis/internal/data/RedisSetCommandsFunctionExecutor.java
##########
@@ -37,18 +37,22 @@ public RedisSetCommandsFunctionExecutor(
     super(helper);
   }
 
-  private RedisSet getRedisSet(ByteArrayWrapper key) {
-    return helper.getRedisSet(key);
+  private RedisSet getRedisSet(ByteArrayWrapper key, boolean updateStats) {

Review comment:
       I think I made the change that you suggested but I'm not positive that I 
understood this correctly




----------------------------------------------------------------
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:
us...@infra.apache.org


> INFO command hit/miss ratios should match native Redis
> ------------------------------------------------------
>
>                 Key: GEODE-8786
>                 URL: https://issues.apache.org/jira/browse/GEODE-8786
>             Project: Geode
>          Issue Type: Bug
>          Components: redis
>    Affects Versions: 1.14.0
>            Reporter: Raymond Ingles
>            Priority: Major
>              Labels: pull-request-available
>
> Current hit/miss ratios for Geode Redis stats do not match those for native 
> Redis.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to