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

ChainAdapter... #97

Closed
guyradford opened this Issue Mar 17, 2017 · 7 comments

Comments

Projects
None yet
3 participants
@guyradford

guyradford commented Mar 17, 2017

Hi,

We are seeing an interesting problem, that we are unable to repeat but have now seen twice during staging.

We are running memcached, with fileSystem cache to persist the data. We are using $cache->save($item) to store the cache item with an expireAfter value of 2 (seconds).

This appears to work for most of the times except for 2 occasions, where the same item was returned by the cache for over 12 hours... We looked at the file on disck and in memcached and I checked that the data and the timestamp is for the previous evening.

We have been using fileSystem Cache for many months without issue, this only started to occur when we added the chain Adapter with memcached.

I have tried to repeat it and written scripts to test the cache and also been through the cachedAdspter and memcachedAdapter code looking for clues.

Any help or suggestions gratefully received.

Thank you
Guy

@Nyholm

This comment has been minimized.

Show comment
Hide comment
@Nyholm

Nyholm Mar 28, 2017

Member

Thank you for reporting this. How does your chain look? Memcached first, right?

What versions are you using? In the latest versions of the adapters we have made some updates regarding this. See php-cache/cache#114

Member

Nyholm commented Mar 28, 2017

Thank you for reporting this. How does your chain look? Memcached first, right?

What versions are you using? In the latest versions of the adapters we have made some updates regarding this. See php-cache/cache#114

@guyradford

This comment has been minimized.

Show comment
Hide comment
@guyradford

guyradford Mar 30, 2017

Hi

Yes the memcached adapter is first.

I thought we where on the latest but looking at the chain adapter it appears to be 0.4.0 not 0.5.0.
I will go back and check all the adapters are up to date and report back if I get any further issue.

Thank you

guyradford commented Mar 30, 2017

Hi

Yes the memcached adapter is first.

I thought we where on the latest but looking at the chain adapter it appears to be 0.4.0 not 0.5.0.
I will go back and check all the adapters are up to date and report back if I get any further issue.

Thank you

@prisis

This comment has been minimized.

Show comment
Hide comment
@prisis

prisis Jun 22, 2017

Member

@guyradford any new for this issue?

Member

prisis commented Jun 22, 2017

@guyradford any new for this issue?

@guyradford

This comment has been minimized.

Show comment
Hide comment
@guyradford

guyradford Jun 24, 2017

We are still in the progress of moving to the latest version. Our test team, didn't want to upgrade immediately as we where to close to a launch. But it should be used in out next release. Ill keep this updated. Happy for you to close if you want.

Thank you

guyradford commented Jun 24, 2017

We are still in the progress of moving to the latest version. Our test team, didn't want to upgrade immediately as we where to close to a launch. But it should be used in out next release. Ill keep this updated. Happy for you to close if you want.

Thank you

@prisis

This comment has been minimized.

Show comment
Hide comment
@prisis

prisis Nov 8, 2017

Member

Hey @guyradford and did you find the problem?

Member

prisis commented Nov 8, 2017

Hey @guyradford and did you find the problem?

@guyradford

This comment has been minimized.

Show comment
Hide comment
@guyradford

guyradford Nov 8, 2017

Hi, @prisis
We haven't seen the issue since updating to the latest version. I guess this thread can now be closed.

Thank you

guyradford commented Nov 8, 2017

Hi, @prisis
We haven't seen the issue since updating to the latest version. I guess this thread can now be closed.

Thank you

@prisis

This comment has been minimized.

Show comment
Hide comment
@prisis

prisis Nov 8, 2017

Member

Okay :), thanks for the info!

Member

prisis commented Nov 8, 2017

Okay :), thanks for the info!

@prisis prisis closed this Nov 8, 2017

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