[
https://issues.apache.org/jira/browse/JAMES-1074?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12923958#action_12923958
]
Norman Maurer commented on JAMES-1074:
--------------------------------------
Hi Eric,
I don't agree 100% here. So here are my comments / concerns:
* core-api
I think I would let lifecycle and service in the module and just rename the
packages, like:
org.apache.james.core.api
org.apache.james.core.api.lifecycle
repository should get moved to mail-api (even if its a bit strange as it is
used by UsersFileRepository too, but I think we should get rid of the class
anyway in the future)
* core-library
I think the MailImpl, MailHeaders etc belong in core. As they are "core" parts
of James. So just move them to org.apache.james.core.lib
filepair should get moved to mail-library / mail-file, Constants to core-api.
The only class in service is AbstractJSR250InstanceFactory which is only used
for unit tests. So move it to core-api tests (its a kind of mock)
* core-function
Both implementations should stay there.. just rename package
> Align packages hierarchy of the services
> ----------------------------------------
>
> Key: JAMES-1074
> URL: https://issues.apache.org/jira/browse/JAMES-1074
> Project: JAMES Server
> Issue Type: Improvement
> Affects Versions: 3.0-M1
> Reporter: Eric Charles
> Assignee: Eric Charles
> Fix For: 3.0-M1
>
>
> We need to align package service hierarchy to have stable api, uniformity and
> osgi-deployable.
> For example, the proposed hierarchy for a "mail" service is:
> org.apache.james.mail.api (in the mail api project)
> org.apache.james.mail.lib (in the mail library project)
> org.apache.james.mail.jpa (in the mail jpa impl project)
> org.apache.james.mail.jcr (in the mail jcr impl project)
> ....
> The conf files and spring-beans.xml need to be updated according to the
> repackaging.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
