Redis Cache By Xon 2.18.2

[Wutime]

I've been testing something with multiple cache implementations and noticed there's perhaps an issue with Xon Redis.

Specifically with deleteMultiple()

If a cache namespace is used, the namespace is NOT prefixed to the keys that are requested for deletion with deleteMultiple(). This results in no keys being deleted.

Key Difference

Native Redis: Automatically applies the namespace to deleteMultiple via normalizeKey

XonRedis: Expects namespaced keys for deleteMultiple, requiring manual prefixing or using delete (which applies getNamespacedId)

I'm just curious if this was intentional or not.

It gets a bit hairy because Xon Redis uses a different namespace mechanism depending on whether a namespace is specified or not.

 

[Xon]

I've been testing something with multiple cache implementations and noticed there's perhaps an issue with Xon Redis.

Specifically with deleteMultiple()

If a cache namespace is used, the namespace is NOT prefixed to the keys that are requested for deletion with deleteMultiple(). This results in no keys being deleted.

I thought I fixed this bug, is this with XF2.3?

 

[Wutime]

I thought I fixed this bug, is this with XF2.3?

Yeah

 

[Xon]

I've been testing something with multiple cache implementations and noticed there's perhaps an issue with Xon Redis.

Specifically with deleteMultiple()

You may want to look at how ExpireRedisCacheByPattern::enqueue and PurgeRedisCacheByPattern::enqueue methods which queue a job todo the bulk of the work. These implement pagable adjusting the TTL or deleting items out of the cache.

This is actually used when invalidating css and such to avoid blocking and schedule things to be expire in 5-10 minutes instead of sticking around not being used.

 

[Xon]

Xon updated Redis Cache By Xon with a new update entry:

2.18.2 - Bugfix update

• Fix deleteMultiple() method for XF2.3 (not used by stock XF)

Read the rest of this update entry...

 

[captainmatt29]

I cant see the redis settings on the ACP main page, its not showing the details even though the installation was done and its all set up as expected.

 

[Xon]

You need to configure the config.php entry, not just install the add-on for this add-on to work

 

[captainmatt29]

You need to configure the config.php entry, not just install the add-on for this add-on to work

I did it’s all setup as described in the post unless I’m missing something.

 

[Wutime]

Yeah, check the FAQ for this addon

 

[philmckrackon]

I did it’s all setup as described in the post unless I’m missing something.

Have you checked if Redis is running via SSH? If Redis is not actually running the ACP display will not be visible.

 

[captainmatt29]

Have you checked if Redis is running via SSH? If Redis is not actually running the ACP display will not be visible.

View attachment 325769

View attachment 325768

We use Redis for Wordpress so it’s definitely online and working, it seems it’s functional just doesn’t show the redis data in the ACP.

 

[MentaL]

We use Redis for Wordpress so it’s definitely online and working, it seems it’s functional just doesn’t show the redis data in the ACP.

admin.php?tools/phpinfo

I would double check this way.

 

[Wutime]

There's a noteworthy improvement if you upgrade to version 7.82 of redis from 6. Some savings in memory usage and speedier retrieval.

 

[Xon]

We use Redis for Wordpress so it’s definitely online and working, it seems it’s functional just doesn’t show the redis data in the ACP.

Check AdminCP => Tools => Redis Info or the /admin.php?redis-info/ URL.

If this shows up, it means there is a template modification conflict which is causing the additions to fail to be shown

 

[captainmatt29]

Check AdminCP => Tools => Redis Info or the /admin.php?redis-info/ URL.

If this shows up, it means there is a template modification conflict which is causing the additions to fail to be shown

This was it After much digging around our custom add-ons etc its now working as expected.

 

[philmckrackon]

There's a noteworthy improvement if you upgrade to version 7.82 of redis from 6. Some savings in memory usage and speedier retrieval.

@Wutime - Thanks for pointing this out. I had completely forgotten to upgrade to 8.2

 

[Wutime]

Thanks for pointing this out. I had completely forgotten to upgrade to 8.2

Hah, and I haven't upgraded to 8.2, so I'll do that tonight - thanks

 

[Enes3078]

@Xon some website says phpredis is better than igbinary but some say the opposite. What is the real deal?

 

[Wutime]

phpredis is better than igbinary

They're completely different things; so one can't really be better than the other. They are both extensions though. Stick with both, one for the layer between php and redis (phpredis), and the other one is an alternate serializer for the data stored inside of redis.

 

[Enes3078]

They're completely different things; so one can't really be better than the other. They are both extensions though. Stick with both, one for the layer between php and redis (phpredis), and the other one is an alternate serializer for the data stored inside of redis.

Okay, any config.php recommendations? Should I use php or igbinary as serializer? And I need the best config. Should I use replicas or sentinel etc.