On 3 October 2013 14:10, Adrian Nistor <anis...@redhat.com> wrote: > I know, was just joking. Anyway, I don't see any confusion having two > classes with the same name.
It's going to be hard enough to explain to people why we are providing two different approaches, if we can't even think of a different name to properly highlight the different usage then we have a problem. Try figuring the forum / support question "I'm using QueryBuilder on Infinispan 7.3 and this happens... " The javadoc index will have it listed twice -> annoying. Google for "QueryBuilder Infinispan" -> annoying Or try figuring out the documentation: # Chapter 5: Queries. There are two approaches to run Queries in Infinispan. Either you use the QueryBuilder, which provides simple domain oriented properties and can work both in embedded and remote mode, or you use the more powerfull QueryBuilder. # 5.1 QueryBuilder blah blah # 5.2 QueryBuilder blah blah If they are different, the should really have different names, even just to avoid confusion among ourselves when talking about hem. If you feel they're the same, the interesting alternative is to literally merge them in one single interface, potentially exposing multiple methods. Sanne > > On 10/03/2013 02:29 PM, Emmanuel Bernard wrote: >> It's already productized code. >> >> On Thu 2013-10-03 14:16, Adrian Nistor wrote: >>> I would suggest renaming the old one :)) >>> >>> On 10/02/2013 11:13 PM, Sanne Grinovero wrote: >>>> It seems we have now 2 different interfaces both names "QueryBuilder" >>>> when using Infinispan Query. >>>> One is coming from Hibernate Search, and represents the "classic" way >>>> to build queries for Infinispan Query in embedded mode. >>>> >>>> The other one is new, and represents the simplified approach, also >>>> implemented for remote queries. >>>> >>>> Could we find an alternative name for the new API? >>>> >>>> It's certainly going to be confusing, even more when we'll have to >>>> document the differences, and which one is more suited for one use >>>> cases vs. another. >>>> >>>> Cheers, >>>> Sanne >>>> _______________________________________________ >>>> infinispan-dev mailing list >>>> infinispan-dev@lists.jboss.org >>>> https://lists.jboss.org/mailman/listinfo/infinispan-dev >>> _______________________________________________ >>> infinispan-dev mailing list >>> infinispan-dev@lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/infinispan-dev >> _______________________________________________ >> infinispan-dev mailing list >> infinispan-dev@lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/infinispan-dev > > _______________________________________________ > infinispan-dev mailing list > infinispan-dev@lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev _______________________________________________ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev