[JAMES-3878] Remove icu4j dependency and replace required functionality with other dependencies

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: master
• Fix Version/s: 3.8.0
• Component/s: mailbox
• Assignee: Unassigned
• Reporter: Wojtek
• Created: 20/Jan/23 16:47
• Updated: 25/Jan/23 14:40
• Resolved: 25/Jan/23 14:40

Description

(context discussion: https://gitter.im/apache/james-project?at=63c87300c47d581062986327)

 

Currently James has `icu4j` as dependency but only uses `UnicodeSet.containsAll()` from it. Given that the dependency is 15M and the usage is minimal and could be replaced with functions from other dependencies it would be good to do so.

 

Right now it's used in:

• `org.apache.james.mailbox.model.MailboxAnnotationKey`
• {{org.apache.james.jmap.draft.model.Keyword }}
• (.java){}
• org.apache.james.jmap.mail.Keyword (.scala)

 

It could be replaced with `re2` from google (https://github.com/google/re2j), which is already present in dependencies and it's only ~100K) and performance wise it should be on pair.

Issue Links

• links to: GitHub Pull Request #1406