Fix crash due to Where/In query which has many values

evadne / etso

Ecto 3 adapter allowing use of Ecto schemas held in ETS tables

https://hexdocs.pm/etso

Apache License 2.0

377 stars 36 forks source link

Fix crash due to Where/In query which has many values #10

Closed pobo380 closed 3 years ago

pobo380 commented 3 years ago

Hi! We use etso in production and it works very well. Thanks for the great library. I found a crash that occurs under OTP 23.0, so I made a PR for it.

Summary

This will fix the crash caused by Where/In query which has many values. (cf. https://github.com/evadne/etso/pull/6)
To avoid ERL-592 bug, fix orlese query to be flat.
- This workaround is the one suggested in the comments of ERL-732.

evadne commented 3 years ago

@pobo380 Thank you very much for this change!!

I might rework the code a bit but will give credits to you.