Github user dsmiley commented on a diff in the pull request:
https://github.com/apache/lucene-solr/pull/323#discussion_r173526922
--- Diff: solr/core/src/test/org/apache/solr/search/TestSolr4Spatial2.java
---
@@ -120,21 +120,27 @@ public void testRptWithGeometryGeo3dField() throws
Exception {
@Test
public void testLatLonRetrieval() throws Exception {
- assertU(adoc("id", "0",
- "llp_1_dv_st", "-75,41",
- "llp_1_dv", "-80.0,20.0",
- "llp_1_dv_dvasst", "40.299599,-74.082728"));
+ assertU(adoc("id", "0", "llp_1_dv_st", "-75,41")); // stored
--- End diff --
Please also test -90 and +90, -180 and +180. This will help ensure there
aren't edge cases (literally) in the DV decoding logic.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
