afs opened a new pull request, #3703:
URL: https://github.com/apache/jena/pull/3703

   GitHub issue resolved #3593
   
   Pull request Description:
   
   This is an attempt to remove SpatialIndexV1 by deleting the class and seeing 
what the consequences are.
   
   * `SpatialIndexStorage`- deprecated but not marked for removal. Reading the 
java comments, look like this should go.
   * `SpatialIndexIo` - this is marked for removal but should it be retained 
with pass-through to `SpatialIndexIoKryo.load`
   * `testSerdeSpatialIndex2` - it is only the last part of the test to remove?
   
   ----
   
    - [ ] Tests are included.
    - [ ] Documentation change and updates are provided for the [Apache Jena 
website](https://github.com/apache/jena-site/)
    - [ ] Commits have been squashed to remove intermediate development commit 
messages.
    - [ ] Key commit messages start with the issue number (GH-xxxx)
   
   By submitting this pull request, I acknowledge that I am making a 
contribution to the Apache Software Foundation under the terms and conditions 
of the [Contributor's 
Agreement](https://www.apache.org/licenses/contributor-agreements.html).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to