Parameter limitation on prefetch_related about mssql-django HOT 7 CLOSED

I have committed a fix in dev branch. Let me know if you encountered any problems.

from mssql-django.

@techknowfile - Thanks for the issue report. We'll take a look and get back to you regarding a potential solution and/or temp workaround. Cheers

from mssql-django.

Thanks, @vwarchu. I think we're going to temporarily hack around this issue by monkey-patching the base.execute() function to identify when we have too many parameters with a query that contains a single IN clause and move those parameters into a temp table instead.

from mssql-django.

@techknowfile I'm able to reproduce the issue, I think Django has some code to split parameters. But it doesn't work with SQL Server. I'll try to provide a solution for that.

from mssql-django.

Thank you, @absci! We had come to a similar approach of creating a temp table, but this is a much cleaner place of putting it

One issue we have rn is that your solution doesn't work when UUIDs are used as the PK. Added comments for suggested changes to your commit

from mssql-django.

I see. If that's the case, it might be better to use different SQL data types for each Django models.

from mssql-django.

This issue has been fixed.

from mssql-django.