OGM-600 Establishing ForwardingGridDialect base class to facilitate the ...

...addition of further dialect facets and chaining them
On Wed 2014-09-17  2:09, Gunnar Morling wrote:
> > +import org.hibernate.ogm.dialect.queryable.spi.QueryableGridDialect;
> > +import org.hibernate.ogm.dialect.spi.GridDialect;
> > +
> > +/**
> > + * Useful functionality for handling grid dialects and their facets.
> > + * <p>
> > + * <b>Note:</b> Most dialect clients should obtain the facet they are interested in directly from the service registry
> > + * instead of using this class.
> > + * <p>
> > + * Dialect clients <b>must never</b> obtain facets by down-casting the current {@link GridDialect}, as dialect wrappers
> > + * such as {@link GridDialectLogger} need to implement all the possible dialect facets, also if the current dialect does
> > + * actually not support a specific facet such as {@link QueryableGridDialect}.
> > + *
> > + * @author Gunnar Morling
> > + */
> > +class GridDialects {
> 
> I like the JDK style for naming helpers after the pluralized form of the
> type they deal with: Collections, Objects, Executors etc.

But our other helper classes use *Helper or *Util, no?
When I saw it I expected to get some kind of GridDialect factory or
container.