pagy-cursor icon indicating copy to clipboard operation
pagy-cursor copied to clipboard

Optimize Query without has more requery

Open Uysim opened this issue 3 years ago • 3 comments

@yunanhelmy had optimize this with PR #19. But from result issue #28 which lead us to revert back to ActiveRecord::Relation. Even so, I still think some developer need to optimize this query. I think we should have two extra modules pagy_cursor/pagy/extras/optimal_cursor and pagy_cursor/pagy/extras/optimal_uuid_cursor which behave the same as #19 and return Ruby Array. From that implementation developer able to pick the implementation that he want.

Uysim avatar Feb 05 '22 09:02 Uysim

@yunanhelmy can you help in this contribution?

Uysim avatar Feb 09 '22 08:02 Uysim

@yunanhelmy can you help in this contribution?

@Uysim I'll help this one.

yunanhelmy avatar Feb 09 '22 10:02 yunanhelmy

@Uysim what about adding vars[:lazy] using default = true. Developer can pass { lazy: false } if they want to return an array.

yunanhelmy avatar Oct 10 '22 10:10 yunanhelmy