N+1 queries with meilisearch

[coder2000]

Don't know if this is pagy or meilisearch specific but looking to limit N+1 queries when executing searches. Where would I add the required includes?

[ddnexus]

Why your intention to limit N+1 queries would have anything to do with where to add the required includes?

[coder2000]

When loading related models to reduce N+1 you usually use Model.includes(:relation) do you not? How would I do that with pagy and meilisearch?

[coder2000]

We sort of discussed this in the past when I opened an issue about it and there was a workaround we came up with but obviously it isn't very intuitive.

[ddnexus]

Oh... that "includes". Sorry, I was still sleepy. I was thinking about the includes of pagy modules into your app. 🤦‍♂️

I've found your Gitter messages. I have no direct experience with meilisearch, so I don't know whether there is a better way to do it.

Since avoiding extra queries means having the data included in the collection, the most straightforward and performant way that comes to my mind is including the data in the index. It might not be optimal for other things that I may ignore though.

[ddnexus]

@brunoocasali Maybe you have some official way to tackle this?

[brunoocasali]

Hi @coder2000 I think we discussed that in the Meilisearch repo meilisearch/meilisearch-rails#133 (comment)

I'm not sure where I misread the discussion, but I thought you understood where to put the includes. Also, you must add the ActiveRecord_Relation.include Pagy::Meilisearch in the model.

class Restaurant < ApplicationRecord
  has_many :reviews, dependent: :destroy

  include MeiliSearch::Rails
  extend Pagy::Meilisearch
  ActiveRecord_Relation.include Pagy::Meilisearch # <--- this line

  meilisearch ...
end

Then you can

restaurants = Restaurant.includes(:reviews).pagy_search(params[:query])
@pagy, @restaurants = pagy_meilisearch(restaurants, items: 25)

Are you still having trouble with the N+1?

[coder2000]

No. This was a reply from someone else.

[brunoocasali]

Nice, I think you can accept my answer @ddnexus and close this discussion :)