Karl Baum

Results 34 comments of Karl Baum

I definitely agree that it's a rails bug but was thinking it would be helpful if bullet reported all n+1 occurrences regardless of whether it's a rails bug or not....

If I explicitly include user_spaces, it fixes or if I taker the order clause away from the has many through and instead order from within memory.— Sent from Mailbox On...

Thx!— Sent from Mailbox On Sun, Jul 27, 2014 at 11:56 PM, Richard Huang [email protected] wrote: > ## @kbaum I'll try > > Reply to this email directly or view...

So it looks like because i am doing the backups myself and also the heroku backup add on, there's a mixture of backups that it returns: ``` 2012/05/15 20:02.13 DATABASE_URL...

> I think there's a missing optimization there. Agreed. I think the reporting code is agnostic of the coverage adapter. With HashRedisStore the data for each file is stored under...

> The coverage adapters can (as far as I can see) stay agnostic, but would provide a method for the webUI's file details which returns coverage data for just the...

I am using rails 4.2 and I noticed that just commenting out the collapse_where method solves the issue for me. I also noticed that only a few squeel tests fail...

Just an update on this one. The problem seems to be in the [collapse_where ](https://github.com/activerecord-hackery/squeel/blob/master/lib/squeel/adapters/active_record/relation_extensions.rb#L269)method. If i simply comment this monkey patch out, then the test passes.

Update again. The problem is now reproduced within a spec on a fork of squeel. https://github.com/viewthespace/squeel/commit/5571d4a96eb9d52f9427172acff6685b1692873f As of now i am bit clueless of what the collapse_where method does and...

@azhi - that looks to be the same issue. I noticed that the issue is resolved if i simply comment out the collapse_where method. Is this monkey patch even needed...