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

Dawid Weiss commented on LUCENE-10485:
--------------------------------------

Hi [~tomoko] . If you can eliminate split packages from these tests, then it 
can be a proper module without any hacks... Much like morfologik.tests is 
implemented, for example. You can't have both worlds at the moment - the IDEs 
get confused, build requires many tweaks, etc. 

> Limit o.a.l.a.morph package visibility only to kuromoji and nori
> ----------------------------------------------------------------
>
>                 Key: LUCENE-10485
>                 URL: https://issues.apache.org/jira/browse/LUCENE-10485
>             Project: Lucene - Core
>          Issue Type: Task
>            Reporter: Tomoko Uchida
>            Priority: Minor
>         Attachments: hacky-poc-make-kuromoji-test-named-module.patch
>
>
> LUCENE-10393 added a new package {{org.apache.lucene.analysis.morph}} to 
> analysis-common that is supposed to be used by only analysis-kuromoji and 
> analysis-nori modules. It would be better to reduce its visibility (on 
> module-mode) by "export to" clause in the module descriptor. 
> The major issue here may be unit tests. They also have to be converted to 
> properly named modules otherwise they cannot access the {{o.a.l.a.morph}} 
> package.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to