We have many instances of OrderBy annotation in our domain objects. And
most of then are defaulting to the ID field, typically just the sequence
value.

I'm not sure why we use these but in effect they cause in-memory
sorting by hibernate on the fetched data.  This of course slows things
down. A quick look shows that most are probably not a big deal given
low cardinality, but a few look interesting.  In general, if not needed
they just add confusion, along with a possibe perf hit.

One in particular to look at is Resource.childResources.