Re: [infinispan-dev] Introducing builder subpackages

Please don't break the configuration API again :-( On 26 June 2013 06:29, Navin Surtani <nsurtani(a)redhat.com&gt; wrote: > While working through ISPN-2463, and the sub-tasks I was wondering about the > organisation of the ConfigurationBuilder classes. > > Currently, they are located in org.infinispan.configuration.cache.etc or > org.infinispan.configuration.global.etc. The actual Configuration classes > are within the same package already as well. To me, this sounds a little bit > cluttered and perhaps not very intuitive and I was wondering if it might be > a better idea to have something like: > > org.infinispan.configuration.cache.builders.ConfigurationBuilder (and > others) > org.infinispan.configuration.global.builders.GlobalConfigurationBuilder (etc > etc) > > Another suggestion could be: > > org.infinispan.configuration.builders.cache.etc > org.infinispan.configuration.builders.glocal.etc > > The only problem with that would be breaking backward compatibility, but > from ISPN 6.x onwards I think that there are a fair few classes and packages > being moved around anyway. It's just an idea that might make the API seem a > little bit cleaner as to where it's located. > > Thoughts?