ucam-department-of-psychiatry / crate

Create and use de-identified research databases. Preprocess, extract text, anonymise/de-identify, link, apply natural language processing, query for research, manage consent for contact.
GNU General Public License v3.0
19 stars 7 forks source link

Fix full text index error on SQL Server #145

Closed martinburchell closed 4 months ago

martinburchell commented 4 months ago

The actual fix is in https://github.com/RudolfCardinal/pythonlib/pull/19. Here we upgrade pythonlib and add tests for full index creation on MySQL and SQL Server.

I've added support for full text index in the SQL Server Dockerfile. The mssql-server-ftspackage seems to be pulling in mssql-server, which is slow and seems unnecessary. It might be that the base docker image bypasses the package management system.

I've also made pytest abort because I forget to pass in --create-db instead of --create-test-db. Confusingly I added --create-db to CamCOPS to mimic the behaviour of pytest-django. With CRATE we have pytest-django already (though we don't currently make use of Django database-level testing) so we have to call the option something else. We should probably make CamCOPS and CRATE consistent with each other.