Thanks! That's merged now.
everything looked very good, I only fixed some whitespace -> tabs.
Cheers,
Sanne
On 12 October 2012 15:56, Nicolas Helleringer
<nicolas.helleringer(a)gmail.com> wrote:
Hi,
Just pushed and created pull request for removal of SpatialQueryBuilder in
tests ans docs.
Added @Longitude & @Latitude in docs.
Niko
2012/10/10 Emmanuel Bernard <emmanuel(a)hibernate.org>
>
> > > onCoordinates(String... names)
>
> Does `onCoordinates()` means all coordinates or simply the default one. If
> all, then
> existing queries will break if you add an new coordinate property to an
> entity.
>
> Also, I'm not sure there is a use case for targeting in the same query,
> the various coordinates defined on a given entity. But I have no real
> idea.
>
> Otherwise, I'm fairly neutral to the proposal.
>
> > > +1 to not consider
> > > "SpatialQueryBuilder.buildSpatialQueryByGrid" a public API but in
that
> > > case I think we should update the tests using it to use the DSL.
> >
> > About this, we should also not mention it in the docs: we're using it
> > in most examples, excluding only the paragraph introducing the Spacial
> > specific DSL.
> > We also refer to it frequently in 9.6, the low level details
> > explanation chapter.
>
> Damn, I thought we discussed that already to remove it from the doc. But
> maybe it was for the unit tests.
> If we go for the private route, we need to move those classes to an
> `impl` package. What's weird is that I thought I did it in the past :/