Re: Defining the expected behavior of DatabaseStatements.execute()

Michael Koziarski
Sun, 21 Jan 2007 22:59:16 -0000

> One more thing - now that 1.2 is out the door, surely now is a good
> time to rationalize ActiveRecord and the various adapters.  To start
> the debate rolling, I''d like to say add_limit_offset! smells
really
> bad.  Surely there''s a cleaner way to manipulate the SQL than
bouncing
> it between ActiveRecord::Base and the adapter?

It definitely would be nice to refactor all the SQL generation out of
Base and into the adapters, where they can safely override whatever
behaviour they need to.   I don''t know that there''s
necessarily going
to be any debate over that :).

By all means, start another thread with what you have in mind :)

--
Cheers

Koz