On Tue, Aug 28, 2012 at 12:12 AM, Aaron Schulz <aschulz4587(a)gmail.com> wrote:
The idea of some base classes for CRUD and API/Pager
table listings is fine.
It can obviously avoid inconsistency among the DAOs. If these classes are
called ORM*, I guess that's OK too, as longs as they don't scope creep into
a complex system that coupled to everything and hard to change.
This pretty much sums up my position. I'm not wholly opposed to
having these for places in which it makes sense (and if extension
authors like it, then that's a perfect reason to leave the support in
core). My fear, however, is that it begins to creep and we end up
totally dependent on ORM. Totally ORM'd databases are scary
and don't make sense (eg: gerrit's db).
-Chad