Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

There was a problem

You must be a member of this account.

Tobias Lütke's latest activity

Tuesday, January 18 2011
6268 [Arel] Offset + Count != good was updated in Ruby on Rails
  • Tobias Lütke
    Tobias Lütke commented at 11:04 PM

    It comes from our application code, we have methods such as
    shop.orders.filter_by_params(params) which looks at things like :page,
    :order_status etc.

    Because it...

Friday, January 14 2011
6268 [Arel] Offset + Count != good was updated in Ruby on Rails
  • Tobias Lütke
    Tobias Lütke commented at 8:03 PM

    yes this is mysql. Frankly i think the right fix is to always clean out any offset if it's a .count query. I found quite a few current_orders.except(:offset).co...

Saturday, January 08 2011
6268 [Arel] Offset + Count != good was created in Ruby on Rails
  • Tobias Lütke
    Tobias Lütke created the ticket at 8:18 PM

    Bug in query planner, produces poor SQL:

    ree-1.8.7-2010.02 > Order.offset(1).count #=> 0 # (200 in table)
    SQL (0.3ms) SELECT COUNT(*) FROM orders LIMIT 18446744...

Monday, April 27 2009
2249 autosaving associations skips validations and persists invalid data in the DB was updated in Ruby on Rails
  • Tobias Lütke
    Tobias Lütke commented at 7:24 PM

    We are running this as part of Shopify since the patch was posted. +1

Thursday, May 15 2008
203 Added fragment_exist? method was updated in Ruby on Rails
  • Tobias Lütke
    Tobias Lütke commented at 7:34 PM

    Looks good, i'm inclined to apply the patch.

    Can you guys add me to the rails project?