You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by GitBox <gi...@apache.org> on 2021/08/06 10:46:29 UTC

[GitHub] [james-project] chibenwa opened a new pull request #581: JAMES-3630 Group quota limit reads

chibenwa opened a new pull request #581:
URL: https://github.com/apache/james-project/pull/581


   This avoids reading the quota limits twice, for instance when receiving emails...
   
   <img width="1482" alt="mailet_transactions_by_percent_queries" src="https://user-images.githubusercontent.com/6928740/128499104-0caf2181-b7b3-4b4e-ac65-6659b63a5b6e.png">
   


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


[GitHub] [james-project] vttranlina commented on pull request #581: JAMES-3630 Group quota limit reads

Posted by GitBox <gi...@apache.org>.
vttranlina commented on pull request #581:
URL: https://github.com/apache/james-project/pull/581#issuecomment-898301079


   I read it


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


[GitHub] [james-project] chibenwa commented on pull request #581: JAMES-3630 Group quota limit reads

Posted by GitBox <gi...@apache.org>.
chibenwa commented on pull request #581:
URL: https://github.com/apache/james-project/pull/581#issuecomment-898192846


   Force pushed to squash fixups...


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


[GitHub] [james-project] chibenwa merged pull request #581: JAMES-3630 Group quota limit reads

Posted by GitBox <gi...@apache.org>.
chibenwa merged pull request #581:
URL: https://github.com/apache/james-project/pull/581


   


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


[GitHub] [james-project] Arsnael commented on a change in pull request #581: JAMES-3630 Group quota limit reads

Posted by GitBox <gi...@apache.org>.
Arsnael commented on a change in pull request #581:
URL: https://github.com/apache/james-project/pull/581#discussion_r685664568



##########
File path: upgrade-instructions.md
##########
@@ -18,6 +18,23 @@ Change list:
  - [Adding the threadId to the ElasticSearch index](#adding-the-threadid-to-the-elasticsearch-index)
  - [Rework message denormalization](#rework-message-denormalization)
  - [Adding threadId column to message metadata tables](#adding-threadid-column-to-message-metadata-tables)
+ - [Restructure maximum quotas definition](#restructure-maximum-quotas-definition)
+ 
+### Restructure maximum quotas definition
+
+Date 06/08/2021
+
+JIRA: https://issues.apache.org/jira/browse/JAMES-3630
+
+Concerned product: Distributed James and
+Concerned product: Distributed James Cassandra James server
+
+We restructured maximum global quota in order to store them in a single Cassandra row. This enables to retrieve this 
+data on a single primary key read, which is important for performance when receiving many emails over SMTP.
+
+Please recreate the maximum quota after the upgrade. 

Review comment:
       By recreate you mean manually right? It's judged ok as it's just one global quota for the entire system, correct?




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


[GitHub] [james-project] chibenwa commented on a change in pull request #581: JAMES-3630 Group quota limit reads

Posted by GitBox <gi...@apache.org>.
chibenwa commented on a change in pull request #581:
URL: https://github.com/apache/james-project/pull/581#discussion_r685671305



##########
File path: upgrade-instructions.md
##########
@@ -18,6 +18,23 @@ Change list:
  - [Adding the threadId to the ElasticSearch index](#adding-the-threadid-to-the-elasticsearch-index)
  - [Rework message denormalization](#rework-message-denormalization)
  - [Adding threadId column to message metadata tables](#adding-threadid-column-to-message-metadata-tables)
+ - [Restructure maximum quotas definition](#restructure-maximum-quotas-definition)
+ 
+### Restructure maximum quotas definition
+
+Date 06/08/2021
+
+JIRA: https://issues.apache.org/jira/browse/JAMES-3630
+
+Concerned product: Distributed James and
+Concerned product: Distributed James Cassandra James server
+
+We restructured maximum global quota in order to store them in a single Cassandra row. This enables to retrieve this 
+data on a single primary key read, which is important for performance when receiving many emails over SMTP.
+
+Please recreate the maximum quota after the upgrade. 

Review comment:
       Correct, correct.




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