[ckan-changes] [ckan/ckan] 4b309c: [#3243] Fix problems with database usage in backgr...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: 4b309c5192e2349a5ad85300dbb74c3c9debba10
      https://github.com/ckan/ckan/commit/4b309c5192e2349a5ad85300dbb74c3c9debba10
  Author: Florian Brucker <florian.brucker at mb.karlsruhe.de>
  Date:   2016-09-23 (Fri, 23 Sep 2016)

  Changed paths:
    M ckan/config/environment.py
    M ckan/lib/jobs.py
    M ckan/tests/lib/test_jobs.py

  Log Message:
  -----------
  [#3243] Fix problems with database usage in background workers.

RQ workers execute jobs in separate processes. However, the CKAN
environment in these processes wasn't initialized correctly, leading to
problems with SQLAlchemy, whose engines cannot be shared between
processes.


  Commit: 559a816328636c12c490311ccdca7da8525ed4ed
      https://github.com/ckan/ckan/commit/559a816328636c12c490311ccdca7da8525ed4ed
  Author: Florian Brucker <florian.brucker at mb.karlsruhe.de>
  Date:   2016-09-23 (Fri, 23 Sep 2016)

  Changed paths:
    M doc/contributing/architecture.rst
    M doc/maintaining/background-tasks.rst

  Log Message:
  -----------
  [#3243] Document database access in background jobs.


  Commit: 658ac4e0186fa784ca1e8529e496c449b3e8712e
      https://github.com/ckan/ckan/commit/658ac4e0186fa784ca1e8529e496c449b3e8712e
  Author: Florian Brucker <florian.brucker at mb.karlsruhe.de>
  Date:   2016-09-23 (Fri, 23 Sep 2016)

  Changed paths:
    M ckan/plugins/interfaces.py
    M ckan/tests/test_coding_standards.py

  Log Message:
  -----------
  [#3243] Document when `IConfigurable.configure` is called.


  Commit: df570b9d74bbe9ffd44bbb4846a19929b0aad6a7
      https://github.com/ckan/ckan/commit/df570b9d74bbe9ffd44bbb4846a19929b0aad6a7
  Author: Florian Brucker <florian.brucker at mb.karlsruhe.de>
  Date:   2016-09-23 (Fri, 23 Sep 2016)

  Changed paths:
    M ckan/tests/config/test_environment.py

  Log Message:
  -----------
  [#3243] Fix broken test case `test_update_config_env_vars`.

The test case changes some configuration options, including the
PostgreSQL URL, to invalid values but failed to correctly restore the
original values afterwards. This wasn't a problem previously because
`ckan.config.update_config` didn't re-initialize the SQLAlchemy engine
if it had already been initialized. The latter behavior has been changed
while fixing #3243, and hence the invalid config values now triggered an
error.


  Commit: a5408647c18bcc3a6317a8c83cfb37c72068e90c
      https://github.com/ckan/ckan/commit/a5408647c18bcc3a6317a8c83cfb37c72068e90c
  Author: Ian Ward <ian at excess.org>
  Date:   2016-09-23 (Fri, 23 Sep 2016)

  Changed paths:
    M ckan/config/environment.py
    M ckan/lib/jobs.py
    M ckan/plugins/interfaces.py
    M ckan/tests/config/test_environment.py
    M ckan/tests/lib/test_jobs.py
    M ckan/tests/test_coding_standards.py
    M doc/contributing/architecture.rst
    M doc/maintaining/background-tasks.rst

  Log Message:
  -----------
  Merge pull request #3247 from torfsen/3243-multiple-db-background-jobs

[#3243] Fix problems with database usage in background workers


Compare: https://github.com/ckan/ckan/compare/e0fdb8dcd279...a5408647c18b