linagora / james-project

Mirror of Apache James Project
Apache License 2.0
72 stars 62 forks source link

[PGSQL] Implement PostgresCustomIdentityDAO #5017

Closed quantranhong1999 closed 9 months ago

quantranhong1999 commented 10 months ago

Why

We need a CustomIdentityDAO backed by Postgres (for Identity/get + set usage).

How

DoD

Pass contract test CustomIdentityDAOContract.

chibenwa commented 10 months ago

Listing all signature of a given user shall be quick.

Otherwise OK to me.

Arsnael commented 10 months ago

Listing all signature of a given user shall be quick.

why? There is no api that fetches only signatures for theuser, we always return the full identity object. Or is it something new? A query fetching just signatures of a user? But I think the front just get directly the all Identity so we are confused here, as we don't do that with Cassandra too.

chibenwa commented 10 months ago

I mean listing all identities for a given user shall be quick.

quantranhong1999 commented 10 months ago

I mean listing all identities for a given user shall be quick.

This should be solved with an additional index on username.

hungphan227 commented 10 months ago

CassandraCustomIdentity use UDT array for reply_to and bbc. Postgresl support UDT but jooq does not support it well. Therefore, I intend to use 2 dimension array (TEXT[][]) for reply_to and bbc

vttranlina commented 10 months ago

CassandraCustomIdentity use UDT array for reply_to and bbc. Postgresl support UDT but jooq does not support it well. Therefore, I intend to use 2 dimension array (TEXT[][]) for reply_to and bbc

or JSON for simple

hungphan227 commented 9 months ago

pr https://github.com/apache/james-project/pull/1960