----- Original Message ----- > From: "Yevgeny Zaspitsky" <[email protected]> > To: "Moti Asayag" <[email protected]> > Cc: "Yair Zaslavsky" <[email protected]>, "Allon Mureinik" > <[email protected]>, [email protected] > Sent: Thursday, August 21, 2014 11:26:40 AM > Subject: Re: [ovirt-devel] [ENGINE] thoughts about LinqUtils > > It seems like we can try moving to common-collections4. Yum on my Fedora20 > computer finds apache-commons-collections4 package. Fortunately somebody > packed the jar into for a rpm for us. :-)
What about RHEL 6.5? Can you please run a quick check? > > > ----- Original Message ----- > > From: "Moti Asayag" <[email protected]> > > To: "Yair Zaslavsky" <[email protected]> > > Cc: "Allon Mureinik" <[email protected]>, [email protected], "Yevgeny > > Zaspitsky" <[email protected]> > > Sent: Thursday, August 21, 2014 11:18:53 AM > > Subject: Re: [ovirt-devel] [ENGINE] thoughts about LinqUtils > > > > > > > > ----- Original Message ----- > > > From: "Yair Zaslavsky" <[email protected]> > > > To: "Allon Mureinik" <[email protected]> > > > Cc: "Moti Asayag" <[email protected]>, [email protected], "Yevgeny > > > Zaspitsky" <[email protected]> > > > Sent: Thursday, August 21, 2014 11:04:13 AM > > > Subject: Re: [ovirt-devel] [ENGINE] thoughts about LinqUtils > > > > > > > > > > > > ----- Original Message ----- > > > > From: "Allon Mureinik" <[email protected]> > > > > To: "Moti Asayag" <[email protected]> > > > > Cc: "Yair Zaslavsky" <[email protected]>, [email protected] > > > > Sent: Thursday, August 21, 2014 10:53:01 AM > > > > Subject: Re: [ovirt-devel] [ENGINE] thoughts about LinqUtils > > > > > > > > IMO, we have too many classes that do the same (useless) thing - > > > > org.ovirt.engine.core.utils.linq.LinqUtils and > > > > org.ovirt.engine.core.common.utils.ListUtils, which both have no place > > > > in > > > > oVirt-engine. > > > > Why not scrap both of them, upgrade commons-collections to the modern > > > > 4.0 > > > > and > > > > use org.apache.commons.collections4.CollectionUtils which has all of > > > > their > > > > functionality, fully-tested, and maintained by someone else? > > > > > > Regarding upgrade to 4.0 - IMHO there was some packaging issue. > > > I think Yevgeny Zaspitsky raised the issue at the past. > > > > > > > This is the mentioned thread: > > http://lists.ovirt.org/pipermail/devel/2014-April/007050.html > > > > > But yeah, if there is no issue there, I'm in favor. > > > > > > > > > > > > > > ----- Original Message ----- > > > > > From: "Moti Asayag" <[email protected]> > > > > > To: "Yair Zaslavsky" <[email protected]> > > > > > Cc: [email protected] > > > > > Sent: Thursday, August 21, 2014 9:04:05 AM > > > > > Subject: Re: [ovirt-devel] [ENGINE] thoughts about LinqUtils > > > > > > > > > > > > > > > > > > > > ----- Original Message ----- > > > > > > From: "Yair Zaslavsky" <[email protected]> > > > > > > To: [email protected] > > > > > > Sent: Thursday, August 21, 2014 5:50:44 AM > > > > > > Subject: [ovirt-devel] [ENGINE] thoughts about LinqUtils > > > > > > > > > > > > Hi Engine developers (and others :) ), > > > > > > I would like to share some thoughts & questions with you about the > > > > > > future > > > > > > of > > > > > > LinqUtils > > > > > > > > > > > > 1. Location of LinqUtils > > > > > > > > > > > > Currently LinqUtils resides at utils. > > > > > > However, as LinqUtils contains some non engine related code, I > > > > > > think > > > > > > it > > > > > > should be moved to the new uutils (universal utils) project. > > > > > > uutils was introduced during the work on AAA, in order to provide a > > > > > > set > > > > > > of > > > > > > utilities that are not engine related, and hopefully in the future > > > > > > will > > > > > > be > > > > > > published as a separate artifcat on > > > > > > maven central (Alon, feel free to correct me here :) ) > > > > > > > > > > > > > > > > +1 > > > > > You'd probably have to address the entire > > > > > org.ovirt.engine.core.utils.linq > > > > > package > > > > > and not just that specific class. > > > > > A sort of a clean-up is also required (i.e. remove IDE generated > > > > > comments), > > > > > seems > > > > > like a good chance to do it. > > > > > > > > > > > 2. The name LinqUtils - Linq comes from c#. I think these days are > > > > > > way > > > > > > behind > > > > > > us (3 years or so now?) i would like to suggest renaming the class. > > > > > > CollectionUtils? Any other suggestions are more than welcome > > > > > > > > > > > > > > > > +1 > > > > > > > > > > > > > > > > > > > > > > > Thanks, > > > > > > Yair > > > > > > _______________________________________________ > > > > > > Devel mailing list > > > > > > [email protected] > > > > > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > > > > > > _______________________________________________ > > > > > Devel mailing list > > > > > [email protected] > > > > > http://lists.ovirt.org/mailman/listinfo/devel > > > > > > > > > > > > > > > _______________________________________________ Devel mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/devel
