N+1 for counts about hanami-pagination HOT 5 OPEN

@bhanuone @jozzi05 those fix does not remove all counts because other methods like

      def total_pages
        pager.total_pages
      end

next_page, etc.
also touches ROM's pager.total indirectly
https://github.com/rom-rb/rom-sql/blob/master/lib/rom/sql/plugin/pagination.rb#L71

that's quite a big issue and seems like can not be fixed by simple "caching" on hanami-pagination side

from hanami-pagination.

@iJackUA I created PR 2 months ago #14 ;)

from hanami-pagination.

@jozzi05 does it really solve issue?
for example when you do

        @total = pager.total
        @total_pages = pager.total_pages

rom-sql pager has such a code

          def total
            dataset.unlimited.count
          end

          def total_pages
            (total / per_page.to_f).ceil
          end

I wonder is this call

          def total
            dataset.unlimited.count
          end

being cached by rom-sql pager or underlying sequel somehow?
or it still tiggers 2 queris for select count(*)...?

from hanami-pagination.

In #14 description:

** Only two count queries are left. The first one when total is touched and the second one when total_pages is touched.

from hanami-pagination.

At least one count query has to be fired(total), and since the total_pages method has some logic inside rom-sql I left it also as a delegate method.

from hanami-pagination.