New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flush only our keys #212

Closed
szepeviktor opened this Issue Dec 5, 2017 · 4 comments

Comments

Projects
None yet
2 participants
@szepeviktor

szepeviktor commented Dec 5, 2017

Sometime there are more than one WordPress installation is using the same db:

sprintf( "EVAL \"return redis.call('del', unpack(redis.call('keys', ARGV[1])))\" 0 %s:*", WP_CACHE_KEY_SALT );

Would it make sense?

@danielbachhuber

This comment has been minimized.

Show comment
Hide comment
@danielbachhuber

danielbachhuber Dec 6, 2017

Member

Sometime there are more than one WordPress installation is using the same db:

Why not switch them to separate databases?

Would it make sense?

I'm not convinced it makes sense to change the current behavior, as it's similar to how the memcached-based drop-ins behave.

Member

danielbachhuber commented Dec 6, 2017

Sometime there are more than one WordPress installation is using the same db:

Why not switch them to separate databases?

Would it make sense?

I'm not convinced it makes sense to change the current behavior, as it's similar to how the memcached-based drop-ins behave.

@szepeviktor

This comment has been minimized.

Show comment
Hide comment
@szepeviktor

szepeviktor Dec 6, 2017

Why not switch them to separate databases?

Do we have FLUSHDB in wp-redis' cli command or in wp-cli?

I'm not convinced it makes sense to change the current behavior, as it's similar to how the memcached-based drop-ins behave.

I mean to provide another version of flush, something like: flush-prefixed

szepeviktor commented Dec 6, 2017

Why not switch them to separate databases?

Do we have FLUSHDB in wp-redis' cli command or in wp-cli?

I'm not convinced it makes sense to change the current behavior, as it's similar to how the memcached-based drop-ins behave.

I mean to provide another version of flush, something like: flush-prefixed

@danielbachhuber

This comment has been minimized.

Show comment
Hide comment
@danielbachhuber

danielbachhuber Dec 6, 2017

Member

Do we have FLUSHDB in wp-redis' cli command or in wp-cli?

Hm, not currently.

I mean to provide another version of flush, something like: flush-prefixed

Could make sense.

Member

danielbachhuber commented Dec 6, 2017

Do we have FLUSHDB in wp-redis' cli command or in wp-cli?

Hm, not currently.

I mean to provide another version of flush, something like: flush-prefixed

Could make sense.

@danielbachhuber

This comment has been minimized.

Show comment
Hide comment
@danielbachhuber

danielbachhuber Feb 26, 2018

Member

Probably makes most sense to implement this on your own for now.

Member

danielbachhuber commented Feb 26, 2018

Probably makes most sense to implement this on your own for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment