[ 
https://issues.apache.org/jira/browse/LUCENE-9558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17207327#comment-17207327
 ] 

Tomoko Uchida commented on LUCENE-9558:
---------------------------------------

I renamed the packages, and added MIGRATE and CHANGES entries on 
[https://github.com/apache/lucene-solr/pull/1946]. I will merge it into the 
master tomorrow if there is no objection/comment.

> Clean up package name conflicts for analyzers-icu module
> --------------------------------------------------------
>
>                 Key: LUCENE-9558
>                 URL: https://issues.apache.org/jira/browse/LUCENE-9558
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: modules/analysis
>            Reporter: Tomoko Uchida
>            Assignee: Tomoko Uchida
>            Priority: Minor
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> analyzers-icu module shares the package names {{o.a.l.collation}} and 
> {{o.a.l.collation.tokenattributes}} with analyzers-common; they need to be 
> renamed.
> There could be two solutions:
> 1. rename "o.a.l.collation" to "o.a.l.a.icu.collation"
>  2. move classes under "o.a.l.collation" to "o.a.l.a.icu" and classes under 
> "o.a.l.collation.tokenattributes" to "o.a.l.a.icu.tokenattributes", and 
> delete "o.a.l.collation" from analyzers-icu
> I would prefer option 2. 1. may complicate the package hierarchy and there 
> already exist {{o.a.l.a.icu.tokenattributes}}. (All classes under 
> "o.a.l.collation" have prefix "ICUCollation", so I think we don't need to 
> keep "collation" in the package name, do we?)



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to