[ https://issues.apache.org/jira/browse/LUCENE-8581?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16726112#comment-16726112 ]
Adrien Grand commented on LUCENE-8581: -------------------------------------- bq. Why is Rectangle2D referring to LatLonShape.BYTES ? Instead, Shouldn't it define a private BYTES as Integer.BYTES? After all, it is this class that encodes/decodes them into a byte array using intToSortableBytes. +1 bq. Why is LatLonShape.BYTES referring to LatLonPoint.BYTES? I think it should define it as Integer.BYTES directly. I also think it could/should be private (or at least package level). I can see the argument going both ways on that one, reusing LatLonPoint.BYTES makes some sense since we are borrowing the way that it discretizes latitudes/longitudes into ints, so reusing the constant participates to conveying the information that we are borrowing encoding logic from LatLonPoint? > Change LatLonShape encoding to use 4 BYTES Per Dimension > -------------------------------------------------------- > > Key: LUCENE-8581 > URL: https://issues.apache.org/jira/browse/LUCENE-8581 > Project: Lucene - Core > Issue Type: New Feature > Reporter: Nicholas Knize > Assignee: Ignacio Vera > Priority: Major > Fix For: master (8.0), 7.7 > > Attachments: LUCENE-8581.patch, LUCENE-8581.patch, LUCENE-8581.patch, > LUCENE-8581.patch, LUCENE-8581.patch, LUCENE-8581.patch, LUCENE-8581.patch, > LUCENE-8581.patch > > > {{LatLonShape}} tessellated triangles currently use a relatively naive > encoding with the first four dimensions as the bounding box of the triangle > and the last three dimensions as the vertices of the triangle. To encode the > {{x,y}} vertices in the last three dimensions requires {{bytesPerDim}} to be > set to 8, with 4 bytes for the x & y axis, respectively. We can reduce > {{bytesPerDim}} to 4 by encoding the index(es) of the vertices shared by the > bounding box along with the orientation of the triangle. This also opens the > door for supporting {{CONTAINS}} queries. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org