cayenne-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrus Adamchik <and...@objectstyle.org>
Subject Re: Ordering chaining PR
Date Thu, 29 Sep 2016 07:14:43 GMT
Even Property.ascs()/descs() and friends are probably redundant for in-memory sorting purposes.

Andrus


> On Sep 29, 2016, at 10:01 AM, Andrus Adamchik <andrus@objectstyle.org> wrote:
> 
> Another consideration - Ordering implements Comparator<Object>. In Java 8 Comparator
has chaining methods. So Orderings class may be redundant.
> 
> Andrus
> 
>> On Sep 29, 2016, at 9:44 AM, Andrus Adamchik <andrus@objectstyle.org> wrote:
>> 
>> https://github.com/apache/cayenne/pull/112
>> 
>> (For those who don't follow commits@ or GitHub)
>> 
>> I am personally +0 on this patch. I commented in more detail on the PR itself.
>> 
>> Thoughts?
>> 
>> Andrus
> 


Mime
View raw message