quantranhong1999 commented on code in PR #1857:
URL: https://github.com/apache/james-project/pull/1857#discussion_r1429602334


##########
src/adr/0071-postgresql-mailbox-tables-structure.md:
##########
@@ -0,0 +1,58 @@
+# 71.  Postgresql Mailbox tables structure
+
+Date: 2023-12-14
+
+## Status
+
+Implemented
+
+## Context
+
+As described in [ADR-70](link), we are willing to provide a Postgres 
implementation for Apache James.
+The current document is willing to detail the inner working of the mailbox of 
the target implementation.
+
+## Decision
+
+![diagram for mailbox tables](img/adr-71-mailbox-tables-diagram.png)
+
+Table list:
+- mailbox
+- mailbox_annotations
+- message
+- message_mailbox
+- subscription
+
+Indexes in table message_mailbox: 
+- message_mailbox_message_id_index (message_id)
+- mailbox_id_mail_uid_index (mailbox_id, message_uid)
+- mailbox_id_is_seen_mail_uid_index (mailbox_id, is_seen, message_uid)
+- mailbox_id_is_recent_mail_uid_index (mailbox_id, is_recent, message_uid)
+- mailbox_id_is_delete_mail_uid_index (mailbox_id, is_deleted, message_uid)
+
+Indexes are used to find records faster
+
+The table structure is mostly normalized which mitigates storage costs and 
achieve consistency easily

Review Comment:
   ```suggestion
   The table structure is mostly normalized which mitigates storage costs and 
achieves consistency easily.
   ```



##########
src/adr/0071-postgresql-mailbox-tables-structure.md:
##########
@@ -0,0 +1,58 @@
+# 71.  Postgresql Mailbox tables structure
+
+Date: 2023-12-14
+
+## Status
+
+Implemented
+
+## Context
+
+As described in [ADR-70](link), we are willing to provide a Postgres 
implementation for Apache James.
+The current document is willing to detail the inner working of the mailbox of 
the target implementation.
+
+## Decision
+
+![diagram for mailbox tables](img/adr-71-mailbox-tables-diagram.png)
+
+Table list:
+- mailbox
+- mailbox_annotations
+- message
+- message_mailbox
+- subscription
+
+Indexes in table message_mailbox: 
+- message_mailbox_message_id_index (message_id)
+- mailbox_id_mail_uid_index (mailbox_id, message_uid)
+- mailbox_id_is_seen_mail_uid_index (mailbox_id, is_seen, message_uid)
+- mailbox_id_is_recent_mail_uid_index (mailbox_id, is_recent, message_uid)
+- mailbox_id_is_delete_mail_uid_index (mailbox_id, is_deleted, message_uid)
+
+Indexes are used to find records faster

Review Comment:
   ```suggestion
   Indexes are used to find records faster.
   ```



-- 
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