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

Remove obsolete options 'UseConcMarkSweepGC' #997

Open
Ashita opened this issue Jan 29, 2021 · 3 comments
Open

Remove obsolete options 'UseConcMarkSweepGC' #997

Ashita opened this issue Jan 29, 2021 · 3 comments

Comments

@Ashita
Copy link

Ashita commented Jan 29, 2021

The option UseConcMarkSweepGC as been deprecated since version 9 and ignored until version 14 of the openJDK ( https://openjdk.java.net/jeps/363 ) and it has now been deleted in v15.

It now generate an error at execution and prevent the application to start :

Unrecognized VM option 'UseConcMarkSweepGC'
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

I think it should be removed.

@vakuum
Copy link

vakuum commented Feb 22, 2021

Maybe we can remove all GC_TUNE settings from solr.in.sh and solr.in.cmd?

These settings have already caused some trouble in the past:

For local development these settings are not necessary and the README.md already states that sunspot_solr should not be used in production.

@vakuum
Copy link

vakuum commented Feb 22, 2021

The GC_TUNE settings are also disabled in the current upstream version of Solr:

Comment from https://issues.apache.org/jira/browse/SOLR-13394: "This stuff is magic voodoo without commentary". 🤩

@mlh758
Copy link
Collaborator

mlh758 commented Feb 22, 2021

In #993 there is some work being done to have the library work better with Solr 8. I think the plan is to remove the bundled solr deployment from this gem in what would end up being a major version release. For now I would really recommend you use a docker image or something for developing against instead of the bundled library. Solr 5 probably isn't what you want anyway.

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

No branches or pull requests

3 participants