Skip to content

SOLR-17321: Update ecj compiler versions to JDK 17#2804

Merged
malliaridis merged 2 commits intoapache:mainfrom
malliaridis:improvements/ecj-updates
Nov 5, 2024
Merged

SOLR-17321: Update ecj compiler versions to JDK 17#2804
malliaridis merged 2 commits intoapache:mainfrom
malliaridis:improvements/ecj-updates

Conversation

@malliaridis
Copy link
Contributor

@malliaridis malliaridis commented Oct 28, 2024

https://issues.apache.org/jira/browse/SOLR-17321

Description

With the JDK upgrade we can bump the JDT compiler versions as well. This allows us to use language features that were introduced after Java 11 (like switch expressions, text blocks, pattern matching of instanceof and more) without ECJ complaining.

I am linking these PRs to the Jira issue because they are related to the JDK changes.

Solution

This PR changes the JDT compiler configuration to Java 17 (because some of our modules targeting 17+).

Tests

No tests were modified.

Checklist

Please review the following and check all that apply:

  • I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
  • I have created a Jira issue and added the issue ID to my pull request title.
  • I have given Solr maintainers access to contribute to my PR branch. (optional but recommended, not available for branches on forks living under an organisation)
  • I have developed this patch against the main branch.
  • I have run ./gradlew check.
  • I have added tests for my changes.
  • I have added documentation for the Reference Guide

This was referenced Oct 30, 2024
@HoustonPutman
Copy link
Contributor

Just pushed to yours! It would be cool to get it to java 21 for the non-solrj/api modules. But this is a good first step!

@malliaridis malliaridis merged commit 6561135 into apache:main Nov 5, 2024
tboeghk pushed a commit to otto-de/solr that referenced this pull request Feb 18, 2026
* Increase ecj version, fix one issue found

---------

Co-authored-by: Houston Putman <houston@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments