On 10 October 2012 12:17, Emmanuel Bernard <emman...@hibernate.org> wrote: >> > 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.
I would assume that empty would target the default one, as it's unnamed as well. > > 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. I would be able to find a car park place to rent which is both close to home and office. > > Otherwise, I'm fairly neutral to the proposal. I'm fairly neutral too, mostly wanted to explore this way to avoid the extra method. > >> > +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 :/ don't remember this but agree _______________________________________________ hibernate-dev mailing list hibernate-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/hibernate-dev