jeantil opened a new pull request, #2805:
URL: https://github.com/apache/james-project/pull/2805

   Removes 
   - james-server-postgres-common-guice -> james-server-guice-distributed 
(which brings cassandra, opensearch and rabbitmq to a module that's suppose to 
provide pg bindings)
   - james-server-guice-common -> james-server-data-jmap (guice-common is 
pulled everywhere and should only contain bindings which are required by all 
apps, protocol specific bindings shouldn't be there)
    
   Arguably mailbox, task and queue shouldn't be there either but I don“t have 
the courage right now.
   
   This change moves 2 bindings to the james postgres app itself, I find it 
really weird to have a binding module for cassandra types in the postgres 
dependencie. I'm not sure if the problem is in having the DLP related types in 
a cassandra module or if it is having DLP in postgres. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to