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

Update solr to 9.4.1 #354

Closed
wants to merge 2 commits into from
Closed

Conversation

kokosing
Copy link
Contributor

Update solr to 9.4.1

@@ -200,7 +200,7 @@
<servlet.api.version>2.5</servlet.api.version>
<slf4j-api.version>2.0.13</slf4j-api.version>
<slf4j.version>2.0.13</slf4j.version>
<solr.version>8.11.2</solr.version>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Build fails with Solr version updated to 9.4.1. Solr 8.11.3 is the last supported version in JDK8. Higher versions require JDK11.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Build fails with Solr version updated to 9.4.1. Solr 8.11.3 is the last supported version in JDK8. Higher versions require JDK11.

I noticed that and I couldn't figure out why. In IDE it was working just fine, but not in CLI.

mneethiraj pushed a commit that referenced this pull request Jul 25, 2024
Signed-off-by: Madhan Neethiraj <madhan@apache.org>
mneethiraj pushed a commit that referenced this pull request Jul 25, 2024
Signed-off-by: Madhan Neethiraj <madhan@apache.org>
(cherry picked from commit e93368d)
@mneethiraj
Copy link
Contributor

@kokosing - Solr version update to 8.11.3 is merged in master and ranger-2.5 branches, via RANGER-4870. I suggest tracking the config cleanup in a separate PR. Thank you!

@mneethiraj mneethiraj closed this Jul 25, 2024
@kokosing kokosing deleted the origin/master/007_solr branch July 26, 2024 09:57
@kokosing
Copy link
Contributor Author

Thank you for merging and feedback!

@kokosing
Copy link
Contributor Author

See #358

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

Successfully merging this pull request may close these issues.

2 participants