[ckan-changes] [ckan/ckan] a25974: [#1879] Set DataStore to legacy mode when using Po...

GitHub noreply at github.com
Tue Sep 16 11:53:23 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: a259742f147a4a0809ffc0bd1a687e43f8e53785
      https://github.com/ckan/ckan/commit/a259742f147a4a0809ffc0bd1a687e43f8e53785
  Author: amercader <amercadero at gmail.com>
  Date:   2014-08-12 (Tue, 12 Aug 2014)

  Changed paths:
    M ckanext/datastore/plugin.py
    M ckanext/datastore/tests/test_unit.py

  Log Message:
  -----------
  [#1879] Set DataStore to legacy mode when using PostgreSQL < 9.x

Up until now the user had to actively set legacy mode when running PG
8.x, and if not you could end up with the full datastore running on an
old postgres version. Features like the SQL search are not properly
supported on these versions, so we need to explicitly disable them when
running on PG 8.x, by setting `legacy_mode` to True.

The current behaviour is that legacy mode will be activated if
`ckan.datastore.read_url` is not present in the configuration and/or
postgres < 9.x.


  Commit: bce494b474d23de59a5d00b9566ec392ec006c04
      https://github.com/ckan/ckan/commit/bce494b474d23de59a5d00b9566ec392ec006c04
  Author: amercader <amercadero at gmail.com>
  Date:   2014-08-12 (Tue, 12 Aug 2014)

  Changed paths:
    M ckanext/datastore/tests/test_unit.py

  Log Message:
  -----------
  [#1879] Fix tests

The travis setup explicitly removes the `ckan.datastore.read_url`
setting when running the tests, so we need to use a dummy one.

Also some refactoring


  Commit: 431636c1a116e74f3bb3ea49a339116153aa025f
      https://github.com/ckan/ckan/commit/431636c1a116e74f3bb3ea49a339116153aa025f
  Author: joetsoi <joe.yeung.tsoi at gmail.com>
  Date:   2014-09-16 (Tue, 16 Sep 2014)

  Changed paths:
    M ckanext/datastore/plugin.py
    M ckanext/datastore/tests/test_unit.py

  Log Message:
  -----------
  Merge branch '1879-datastore-legacy-mode'


Compare: https://github.com/ckan/ckan/compare/bb16ca6f701c...431636c1a116


More information about the ckan-changes mailing list