JAMES-4150 Make Attachment Storage Optional for PostgreSQL Message Store

[omerfarukicen]

This PR introduces a configurable option to enable or disable attachment storage in the PostgreSQL mailbox implementation.

[omerfarukicen]

can you review CI pipeline?
Build Error
or while contacting Gradle Enterprise server at https://ge.apache.org/
javax.net.ssl.SSLHandshakeException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target

[Arsnael]

Sounds like a random CI issue, I restarted the build

[Arsnael]

Overall looks good to me thank you @omerfarukicen ! Can you just be careful with the unnecessary indents in the code though? Adds noise. Thanks :)

[Arsnael]

@omerfarukicen those errors though look related to me: https://ci-builds.apache.org/job/james/job/ApacheJames/job/PR-2885/3/testReport/

Can you have a look please? Thanks :)

[chibenwa]

It looks related to the docker runner apparently

Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.001 s <<< FAILURE! -- in org.apache.james.backends.postgres.PostgresTableManagerTest
org.apache.james.backends.postgres.PostgresTableManagerTest -- Time elapsed: 0.001 s <<< ERROR!
java.lang.NoClassDefFoundError: Could not initialize class org.apache.james.backends.postgres.PostgresTableManagerTest
	at java.base/jdk.internal.misc.Unsafe.ensureClassInitialized0(Native Method)
	at java.base/jdk.internal.misc.Unsafe.ensureClassInitialized(Unsafe.java:1160)
	at java.base/java.lang.reflect.Field.acquireOverrideFieldAccessor(Field.java:1200)
	at java.base/java.lang.reflect.Field.getOverrideFieldAccessor(Field.java:1169)
	at java.base/java.lang.reflect.Field.get(Field.java:444)
	at java.base/java.util.stream.ForEachOps$ForEachOp$OfRef.accept(ForEachOps.java:184)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
	at java.base/java.util.stream.SortedOps$RefSortingSink.end(SortedOps.java:395)
	at java.base/java.util.stream.Sink$ChainedReference.end(Sink.java:261)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:510)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499)
	at java.base/java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:151)
	at java.base/java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:174)
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.base/java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:596)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
Caused by: java.lang.ExceptionInInitializerError: Exception java.lang.NoClassDefFoundError: Could not initialize class org.apache.james.backends.postgres.PostgresExtension [in thread "main"]
	at org.apache.james.backends.postgres.PostgresTableManagerTest.<clinit>(PostgresTableManagerTest.java:41)
	... 16 more

I relaunched a build

[Arsnael]

Same errors, I believe related to the change