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

Various db upgrades #1130

Merged
merged 1 commit into from Aug 6, 2013
Merged

Various db upgrades #1130

merged 1 commit into from Aug 6, 2013

Conversation

kindly
Copy link
Contributor

@kindly kindly commented Jul 25, 2013

Three database upgrades bundles together in single pull request for convenience.

  • Metadata modifed

Make metadata modified a field in package table to make it possible for @wardi to speed up package show

  • creator user id

cache datasets creators user id

  • url type

Add url type for later making relative resource possible

@kindly
Copy link
Contributor Author

kindly commented Jul 25, 2013

@wardi would be great if you could help review this one.

@wardi
Copy link
Contributor

wardi commented Jul 26, 2013

This looks like what I need.

Is there some documentation for the intended use of url_type? I wonder if
it's similar to a resource extra we're already using.

@ghost ghost assigned joetsoi and amercader Jul 30, 2013
@kindly
Copy link
Contributor Author

kindly commented Aug 1, 2013

@wardi

There is no documentation yet and is a placeholder for future needs as we have two uses for it.

  • saying the resource is held in the local filestore.
  • saying the resource is one held in the datastore.

This means that we can put in the url field not just full url paths but relative urls or anything relevant to the particular url type. (the datastore one for example will point to a way to get a csv copy of the data)

@amercader amercader merged commit 0d5eb39 into master Aug 6, 2013
@domoritz domoritz deleted the 1130-various-db-upgrades branch August 7, 2013 11:47
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.

None yet

4 participants