Skip to content
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

Pecka master #8

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Pecka master #8

wants to merge 4 commits into from

Conversation

Spamercz
Copy link

Legacy PR, this is more application problem than library.

I will leave this open until better application solution is provided.

vitkutny and others added 4 commits February 13, 2017 13:40
When `@unserialize` returns FALSE because of error method `getUnserializedValue` will return NULL and Nette\Caching\Cache::load will regenerate this record. Same for Throwable/Exception thrown from objects implementing \Serializable interface.
there was a multiple job concurrency in RedisStorage::write
 - storage writes tags to journal
 - another process cleans those tags
 - storage writes data

as result there were data saved in cache but tags in journal were missing
 - those records will never invalidate

solved by switching tags/data write order
RedisStorage: getUnserializedValue Exception and error
RedisStorage: prevents clean and write concurrency
@Spamercz Spamercz self-assigned this Oct 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants