[LUCENE-9499] Clean up package name conflicts between modules (split packages)

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 9.0
• Fix Version/s: 9.0
• Lucene Fields: New
• Assignee: Tomoko Uchida
• Reporter: Tomoko Uchida
• Created: 02/Sep/20 14:39
• Updated: 28/Aug/22 16:07
• Resolved: 12/Nov/20 23:29

Description

We have lots of package name conflicts (shared package names) between modules in the source tree. It is not only annoying for devs/users but also indeed bad practice since Java 9 (according to my understanding), and we already have some problems with Javadocs due to these splitted packages as some of us would know. Also split packages make migrating to the Java 9 module system impossible.

This is the placeholder to fix all package name conflicts in Lucene.

See the dev list thread for more background.
https://lists.apache.org/thread.html/r6496963e89a5e0615e53206429b6843cc5d3e923a2045cc7b7a1eb03%40%3Cdev.lucene.apache.org%3E

Modules that need to be fixed / cleaned up:

• analyzers-common (LUCENE-9317)
• analyzers-icu (LUCENE-9558)
• backward-codecs (LUCENE-9318)
• sandbox (LUCENE-9319)
• misc (LUCENE-9600)
• (test-framework: this can be excluded for the moment)

Also lucene-core will be heavily affected (some classes have to be moved into core, or some classes' and methods' in core visibility have to be relaxed).

Probably most work would be done in a parallel manner, but conflicts can happen. If someone want to help out, please open an issue before working and share your thoughts with me and others.

I set "Fix version" to 9.0 - means once we make a commit on here, this will be a blocker for release 9.0.0. (I don't think the changes should be delivered across two major releases; all changes have to be out at once in a major release.) If there are any objections or concerns, please leave comments. For now I have no idea about the total volume of changes or technical obstacles that have to be handled.

Issue Links

• is a parent of: LUCENE-9317 Resolve package name conflicts for StandardAnalyzer to allow Java module system support
• is a parent of: LUCENE-9318 Fix Codec API to not rely on package-private classes as part of changes to support java module system
• is a parent of: LUCENE-9319 Clean up Sandbox project by retiring/delete functionality or move it to Lucene core
• is a parent of: LUCENE-9558 Clean up package name conflicts for analyzers-icu module
• is a parent of: LUCENE-9600 Clean up package name conflicts for misc module
• links to: GitHub Pull Request #2072