On Tue, Aug 28, 2012 at 12:12 AM, Aaron Schulz aschulz4587@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