[GitHub] [james-project] Arsnael commented on pull request #1666: DRAFT JAMES-4824 Refactor quota for Cassandra: create POJOs and DAOs

["Arsnael (via GitHub)" <gi...@apache.org>]

Arsnael commented on PR #1666:
URL: https://github.com/apache/james-project/pull/1666#issuecomment-1663616425

   > IMO a good indicator here would be to cut ~halth the count of lines of code.
   
   Agreed, code should be simpler and also the work should be cut in 2 or 3 smaller PRs like:
   - POJOs (made simpler following @chibenwa comments)
   - DAO interfaces and memory implementation
   - Cassandra table defs and implementation
   
   Poentially POJOs and memory implem could be together, but not the Cassandra implem


-- 
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: notifications-unsubscribe@james.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org