Github user dsmiley commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/323#discussion_r172258894
  
    --- Diff: lucene/core/src/java/org/apache/lucene/geo/GeoEncodingUtils.java 
---
    @@ -127,6 +130,10 @@ public static double decodeLatitude(int encoded) {
       public static double decodeLatitude(byte[] src, int offset) {
         return decodeLatitude(NumericUtils.sortableBytesToInt(src, offset));
       }
    +  
    +  public static double decodeLatitudeCeil(long encoded) {
    --- End diff --
    
    @nknize  could you please review this part?  It's not clear to me why 
GeoEncodingUtils has dual ceil/floor for encode but not for decode.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to