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

Many to Many Support #8

Closed saifelokour closed 3 years ago

saifelokour commented 4 years ago

Hello,

I've created a minimal test to show how the library does not currently support many_to_many in Ecto schemas using the Northwind Demo. Here's the output:

image

I've also created a PR in hopes that I could get some guidance on how we might go about adding support for this feature.

The issue seems to be in that the build conditions function head is hard coded here: https://github.com/evadne/etso/blob/f25bdea063893328272adb14004a95973ad7f007/lib/etso/ets/match_specification.ex#L57

However it's not clear how I might change this to support many_to_many.

Please let me know if I can provide more information.

Thanks :)

evadne commented 3 years ago

I shall close this as the Pull Request as been opened.