[ckan-dev] test performance and sqlalchemy complete

Thu Jan 6 14:28:03 UTC 2011

I've commited a monkeypatch now.

Seb

On 6 January 2011 14:16, Seb Bacon <seb.bacon at gmail.com> wrote:
> 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
>

-- 
skype: seb.bacon
mobile: 07790 939224
land: 0207 183 9618
web: http://baconconsulting.co.uk