[ckan-dev] test performance and sqlalchemy complete

Thu Jan 6 14:16:08 UTC 2011

On 6 January 2011 11:40, Rufus Pollock <rufus.pollock at okfn.org> wrote:
> On 6 January 2011 10:53, Seb Bacon <seb.bacon at gmail.com> wrote:
>> It's just been pointed out to me that you also need to apply
>> sqlalchemy.patch (in the root of the ckan src) to get everything to
>> pass.
>>
>> I guess the best way to deal with that for now is to monkeypatch it on
>> startup.  Thoughts?
>
> Asking people (including ourselves) to remember to apply a patch is
> asking for trouble so monkey-patch it is. BTW, what exactly is the
> monkey-patch for (e.g. is it sqlite with a specific version or ...)

It's for the sqlite dialect in sqlalchemy 0.6.

The patch would be different for previous versions.  To simplify
matters, we can just say we only support sqlite as a testing backend
for the time being, and only monkeypatch when running tests.

Seb