Skip to content
This repository has been archived by the owner on Jan 21, 2020. It is now read-only.

Distribute translations invalidation to all servers #660

Closed
fauvel opened this issue Jun 27, 2013 · 6 comments
Closed

Distribute translations invalidation to all servers #660

fauvel opened this issue Jun 27, 2013 · 6 comments
Assignees
Labels
Milestone

Comments

@fauvel
Copy link
Contributor

fauvel commented Jun 27, 2013

No description provided.

@fauvel fauvel closed this as completed Jun 27, 2013
@njam
Copy link
Member

njam commented Jun 27, 2013

Dispatching of the invalidation request to all servers is actually not happening. So this is a bug, agree?
Would be part of: https://github.com/cargomedia/fuboo/issues/44

@njam njam reopened this Jun 27, 2013
@fauvel
Copy link
Contributor Author

fauvel commented Jun 27, 2013

Yes, exactly :)

@fauvel
Copy link
Contributor Author

fauvel commented Jun 27, 2013

We should also remove the CM_Model_Language::onChange() method, which gives a false impression of security whereas it actually doesn't make anything (since the paging source for the translations isn't using the cache at all).

@tomaszdurka probably forgot about removing it as he moved the caching from the paging source to the CM_Model_Language::getTranslation() method.

@njam
Copy link
Member

njam commented Jun 27, 2013

Can you open a new issue for this and discuss with/assign to tomasz?

@ppp0
Copy link
Member

ppp0 commented Jun 27, 2013

As soon as we will run several web servers, we'll have a problem with local vs distributed cache.
Facebook does it like this:

screen shot 2013-06-27 at 4 31 22 pm

@ghost ghost assigned fauvel Aug 8, 2013
@njam
Copy link
Member

njam commented Aug 12, 2013

Closing in favor of https://github.com/cargomedia/fuboo/issues/1888

@njam njam closed this as completed Aug 12, 2013
@cargomedia cargomedia deleted a comment from fauvel May 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants