absinthe-graphql / absinthe_ecto

DEPRECATED: Use dataloader
MIT License
130 stars 36 forks source link

Uniq the ids in the batch fun before querying #3

Closed hanrelan closed 7 years ago

avitex commented 7 years ago

An alternative way is to simply use Enum.dedup/1, which I was using upstream

hanrelan commented 7 years ago

dedup only removes consecutive duplicated elements which isn't the case here

avitex commented 7 years ago

Good point, I didn't read the spec close enough :

benwilson512 commented 7 years ago

Thanks!