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

Purge varnish items with "cdb_tablemetadata" channel on table update #308

Merged
merged 1 commit into from
Dec 10, 2013

Conversation

strk
Copy link

@strk strk commented Dec 3, 2013

Closes #296

Ensures that queries against CDB_TableMetadata for "last_updated"
(for example from Windshaft-CartoDB) are correctly invalidated.

You need to run bundle exec rake cartodb:db:load_functions to upgrade,
as already noted in the UPGRADE file.

Closes CartoDB#296

Ensures that queries against CDB_TableMetadata for "last_updated"
(for example from Windshaft-CartoDB) are correctly invalidated.

You need to run bundle exec rake cartodb:db:load_functions to upgrade,
as already noted in the UPGRADE file.
@luisbosque
Copy link
Contributor

Why is rake cartodb:db:load_functions necessary for a change in the model?

@strk
Copy link
Author

strk commented Dec 4, 2013

It is needed for a change in the database_model.

In this specific case it is needed as it calls
create_function_invalidate_varnish for each user.

--strk;

@luisbosque
Copy link
Contributor

Actually it's only necessary to load the varnish invalidation function.
Added in this commit;
00cc6cc

@strk
Copy link
Author

strk commented Dec 4, 2013

Good, anyway the point is that since "load_functions" is already documented in https://github.com/CartoDB/cartodb/blob/v2.5.6/UPGRADE, there should be no need to note in the NEWS file about the need to run load_varnish_invalidation_function (implied by the former)

@strk
Copy link
Author

strk commented Dec 6, 2013

For the record: that commit ended up in release/2.6.0 -- this pull request has to be closed once it hits master

@rafacas rafacas merged commit 6b0481b into CartoDB:master Dec 10, 2013
@strk strk deleted the master-cdb_tablemetadata-cachepurge branch December 10, 2013 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clear caches when CDB_TableMetadata changes
3 participants