sinfin / auctify

Rails engine for auctions of items
MIT License
11 stars 4 forks source link

Use ordered_applied_bids in bids_appender to optimize SQL usage #25

Closed mreq closed 3 years ago

mreq commented 3 years ago

S touhle zmenou se usetri na kazdem requestu N SQL dotazu, kde N je roven poctu karticek s predmetama. Posilam radeji pro kontrolu, abych ti tim neco nerozbil. Testy jsou zelene.

foton commented 3 years ago

@mreq Hmm, pěkné. Dík.
Jak koukám má pak smysl mít v registraci applied_bids? Respektive jestli pod ně rovnou nedat i řazení.

mreq commented 3 years ago

Jak koukám má pak smysl mít v registraci applied_bids? Respektive jestli pod ně rovnou nedat i řazení.

V AA se nikde nepouziva. Ale mozna bych nechal ve jmenu to ordered_, at je jasne, ze je pripadne potreba unscope(:order)...