You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by "quantranhong1999 (via GitHub)" <gi...@apache.org> on 2026/07/14 07:29:23 UTC

[PR] [BUILD] Last JMAP integration tests batch: apply user isolation + per class life cycle [james-project]

quantranhong1999 opened a new pull request, #3090:
URL: https://github.com/apache/james-project/pull/3090

   All JMAP integration tests should be much faster now.


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


Re: [PR] [BUILD] Last JMAP integration tests batch: apply user isolation + per class life cycle [james-project]

Posted by "Arsnael (via GitHub)" <gi...@apache.org>.
Arsnael commented on PR #3090:
URL: https://github.com/apache/james-project/pull/3090#issuecomment-4977659521

   Latest master:
   06:19:01,676 [INFO] Apache James :: Server :: JMAP RFC-8621 :: Distributed Integration Testing SUCCESS [04:16 min]
   06:19:01,676 [INFO] Apache James :: Server :: JMAP RFC-8621 :: Memory Integration Testing SUCCESS [02:39 min]
   06:19:01,677 [INFO] Apache James :: Server :: JMAP RFC-8621 :: Postgres Integration Testing SUCCESS [02:16 min]
   
   @quantranhong1999 hmmm do we miss something here?


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


Re: [PR] [BUILD] Last JMAP integration tests batch: apply user isolation + per class life cycle [james-project]

Posted by "quantranhong1999 (via GitHub)" <gi...@apache.org>.
quantranhong1999 commented on PR #3090:
URL: https://github.com/apache/james-project/pull/3090#issuecomment-4979427464

   > I suggest doing local runs before / after in Tmail AND James repo to be sure we do not ship a regression.
   
   I did run all the JMAP ITs on the James side on my IDE (these changes are not related to TMail and no need TMail adoption)
   
   ## Before
   memory: 3 min 50 sec
   postgres: 12 min
   
   ## After
   memory: 2 min 38 sec
   postgres: 4 min 15 sec
   
   
   
   I did not run distributed tests because it takes time, but the changes should logically be good and beneficial.
   


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


Re: [PR] [BUILD] Last JMAP integration tests batch: apply user isolation + per class life cycle [james-project]

Posted by "chibenwa (via GitHub)" <gi...@apache.org>.
chibenwa commented on PR #3090:
URL: https://github.com/apache/james-project/pull/3090#issuecomment-4979465928

   Awwesome!
   IMO it does even open the room to remove "smoke tests" for at least PG and always run those tests.
   With a 4 minute budget the added value is nice.


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


Re: [PR] [BUILD] Last JMAP integration tests batch: apply user isolation + per class life cycle [james-project]

Posted by "chibenwa (via GitHub)" <gi...@apache.org>.
chibenwa commented on PR #3090:
URL: https://github.com/apache/james-project/pull/3090#issuecomment-4977616922

   > 08:09:38,456 [INFO] Apache James :: Server :: JMAP RFC-8621 :: Distributed Integration Testing SUCCESS [09:27 min]
   > 08:09:38,456 [INFO] Apache James :: Server :: JMAP RFC-8621 :: Memory Integration Testing SUCCESS [04:04 min]
   > 08:09:38,459 [INFO] Apache James :: Server :: JMAP RFC-8621 :: Postgres Integration Testing SUCCESS [04:41 min]
   
   How much was it before, as a reference?


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


Re: [PR] [BUILD] Last JMAP integration tests batch: apply user isolation + per class life cycle [james-project]

Posted by "chibenwa (via GitHub)" <gi...@apache.org>.
chibenwa commented on PR #3090:
URL: https://github.com/apache/james-project/pull/3090#issuecomment-4977673444

   There might be variability dependig on the VM running the tests...
   
   I suggest doing local runs before / after in Tmail AND James repo to be sure we do not ship a regression.


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


Re: [PR] [BUILD] Last JMAP integration tests batch: apply user isolation + per class life cycle [james-project]

Posted by "Arsnael (via GitHub)" <gi...@apache.org>.
Arsnael commented on PR #3090:
URL: https://github.com/apache/james-project/pull/3090#issuecomment-4976236076

   08:09:38,456 [INFO] Apache James :: Server :: JMAP RFC-8621 :: Distributed Integration Testing SUCCESS [09:27 min]
   08:09:38,456 [INFO] Apache James :: Server :: JMAP RFC-8621 :: Memory Integration Testing SUCCESS [04:04 min]
   08:09:38,459 [INFO] Apache James :: Server :: JMAP RFC-8621 :: Postgres Integration Testing SUCCESS [04:41 min]


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


Re: [PR] [BUILD] Last JMAP integration tests batch: apply user isolation + per class life cycle [james-project]

Posted by "chibenwa (via GitHub)" <gi...@apache.org>.
chibenwa merged PR #3090:
URL: https://github.com/apache/james-project/pull/3090


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