Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split JtsSpatialContext etc. to separate module and 'jar'. #143

Open
dsmiley opened this issue Sep 6, 2016 · 3 comments
Open

Split JtsSpatialContext etc. to separate module and 'jar'. #143

dsmiley opened this issue Sep 6, 2016 · 3 comments

Comments

@dsmiley
Copy link
Contributor

dsmiley commented Sep 6, 2016

I was bemoaning the inconvenience in Solr that the JTS jar file has to be placed in WEB-INF/lib instead of being placeable in more convenient places, like the 'lib' of the node or at a core level. Essentially, this is because spatial4j.jar is inside WEB-INF/lib which is loaded by a lower-level more internal classloader, and so it can only refer to things at its level or lower still.

If all the JTS using classes in Spatial4j were spun off into a separate jar, perhaps named spatial4j-jts-ctx.jar then that jar could go into a convenient lib dir as well as JTS, and it should work.

Another side benefit is that anyone fearing that Spatial4j is somehow tainted in that it refers to LGPL code (even though it's "optional") would be assuaged.

On the down side, especially for anyone without any classpath conundrums, this is yet another 'jar' file. I think it's worth it though.

@jdeolive
Copy link
Contributor

jdeolive commented Sep 6, 2016

+1 I like the idea of breaking out separate jars/modules for things that bring along additional dependencies not actually required by the core. I think that the support for jackson (#138) could also fall into this category as well (even though it's written with jackson as an optional dependency). It also sets the pattern for additional spatial libraries (like geo3d ?) to be added as well without bloating the core library. My $0.02

@dsmiley
Copy link
Contributor Author

dsmiley commented Nov 24, 2017

After #138 then I think I'll take a stab at this.

@dsmiley
Copy link
Contributor Author

dsmiley commented Dec 6, 2017

Ehh; I think I'll let it be after all since Solr will now be able to include JTS and as-such the pain point I felt here should vanish. That said if anyone wants to take this up then be my guest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants