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

Eliminate split packages #78166

Closed
55 of 58 tasks
ChrisHegarty opened this issue Sep 22, 2021 · 2 comments
Closed
55 of 58 tasks

Eliminate split packages #78166

ChrisHegarty opened this issue Sep 22, 2021 · 2 comments
Assignees
Labels
>bug modularization Java Modules related

Comments

@ChrisHegarty
Copy link
Contributor

ChrisHegarty commented Sep 22, 2021

To allow for the future modularization of Elasticsearch with Java Modules, there are a number preparatory tasks that need be completed. This is one such task: eliminate split packages.

Split packages with Lucene:

#### Split packages within ES itself:

@ChrisHegarty ChrisHegarty added >bug modularization Java Modules related labels Sep 22, 2021
@ChrisHegarty ChrisHegarty self-assigned this Sep 22, 2021
@ChrisHegarty ChrisHegarty reopened this Sep 23, 2021
ChrisHegarty added a commit that referenced this issue Oct 5, 2021
With LUCENE-10118 integrated, we can now remove the package
-private dependency with org.apache.lucene.index.OneMergeHelper, and
intercept the info/log messages coming from merge threads.

This change alters the logging a little, but the fundamental information
captured remains more or less the same. It is worth noting that since
the merges occur asynconously, the actual post-merge statistics are best
captured when the merge thread completes its operation - which is the
case with the change in this PR.

relates (#78166)
ChrisHegarty added a commit that referenced this issue Oct 5, 2021
#78605)

With LUCENE-10132 integrated, we can now replace the package
-private dependency on the setDiagnostics method with the newly added
addDiagnostics. With this change, SFMP can be relocated to its
only-use package org.elasticsearch.index.engine.

relates (#78166)
@SympathyForTheDev
Copy link

SympathyForTheDev commented Jan 3, 2022

Hello @ChrisHegarty , it's look like the changes in
#78831

are breaking backwards compatibility (in a minor version update)

i'm getting error like Caused by: java.lang.NoClassDefFoundError: org/elasticsearch/common/xcontent/json/JsonXContent

after updating Elasticsearch client

please see jobrunr/jobrunr#291

@SympathyForTheDev
Copy link

@ChrisHegarty see #82357

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug modularization Java Modules related
Projects
None yet
Development

No branches or pull requests

2 participants