This is an automated email from the ASF dual-hosted git repository. rcordier pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git.
from 63287bf JAMES-3277 Don't read outbox mailbox for each message in SetMessagesUpdateProcessor new d69c7e2 JAMES-3271 Fixing various checkstyle errors in test classes new 5c87926 JAMES-3277 Rewrite AbstractMessageIdManagerStorageTest::setInMailboxesShouldIgnoreMessagesBelongingToOtherUsers new 36c7e23 JAMES-3277 Add a test about limiting mailboxes reads for StoreMessageIdManager::setInMailboxes new 901517e JAMES-3277 QuotaRootResolver should resolve quotaRoot for a mailbox new 08ffc8e JAMES-3277 Move StoreRightManager::myRights for Mailbox to the API new 0f5e381 JAMES-3277 Rework MessageIdMapper APIs new a7fdccb JAMES-3277 Rework StoreMessageIdManager::setInMailboxes to limit mailbox reads The 7 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: .../cassandra/DockerCassandraSingleton.java | 2 +- .../apache/james/backends/cassandra/Scenario.java | 3 +- .../backends/cassandra/utils/PaggingTest.java | 1 + .../james/backends/es/DockerElasticSearch.java | 2 + .../backends/es/ElasticSearchClusterExtension.java | 9 +- .../backends/es/ElasticSearchHealthCheckTest.java | 3 +- .../backends/rabbitmq/DockerRabbitMQTestRule.java | 2 +- .../james/backends/rabbitmq/RabbitMQExtension.java | 2 +- .../rabbitmq/RabbitMQServerVersionTest.java | 1 + .../org/apache/james/core/MailAddressTest.java | 2 +- .../org/apache/james/dto/FirstDomainObject.java | 8 +- .../java/org/apache/james/dto/FirstNestedType.java | 10 +- .../org/apache/james/dto/SecondDomainObject.java | 8 +- .../org/apache/james/dto/SecondNestedType.java | 8 +- .../org/apache/james/mailbox/RightManager.java | 14 ++ .../james/mailbox/quota/QuotaRootResolver.java | 2 + .../james/mailbox/fixture/MailboxFixture.java | 1 + .../apache/james/mailbox/model/MailboxACLTest.java | 1 + .../mailbox/backup/ZipArchivesLoaderTest.java | 7 +- .../apache/james/mailbox/backup/ZipAssertTest.java | 5 +- .../james/mailbox/backup/ZipEntryTypeContract.java | 2 +- .../cassandra/mail/CassandraMessageIdMapper.java | 17 +- .../cassandra/CassandraMailboxManagerTest.java | 2 +- .../CassandraMessageIdManagerSideEffectTest.java | 27 +++ .../mail/CassandraApplicableFlagDAOTest.java | 1 + .../cassandra/mail/CassandraMessageDAOTest.java | 2 - .../cassandra/mail/CassandraMessageIdDAOTest.java | 1 + .../cassandra/mail/CassandraMessageMapperTest.java | 2 +- .../SolveMessageInconsistenciesServiceTest.java | 1 - ...asticSearchListeningMessageSearchIndexTest.java | 2 +- .../mailbox/elasticsearch/json/MimePartTest.java | 3 +- .../query/DateResolutionFormatterTest.java | 2 - .../events/delivery/InVmEventDeliveryTest.java | 1 - .../mailbox/events/LocalListenerRegistryTest.java | 12 +- .../james/mailbox/events/RabbitMQEventBusTest.java | 1 + .../org/apache/james/event/json/dtos/UserTest.java | 1 - .../inmemory/mail/InMemoryMessageIdMapper.java | 4 +- .../InMemoryCombinationManagerTestSystem.java | 1 - .../inmemory/InMemoryMessageManagerTestSystem.java | 1 - .../james/vault/metadata/UserPerBucketDAOTest.java | 1 - .../james/vault/DeletedMessageZipperTest.java | 2 +- .../blob/BlobStoreDeletedMessageVaultTest.java | 2 +- .../dto/query/QueryElementSerializerTest.java | 1 + .../james/vault/dto/query/QueryTranslatorTest.java | 13 +- .../mailbox/quota/model/QuotaThresholdsTest.java | 1 + .../elasticsearch/UserRoutingKeyFactoryTest.java | 1 - .../mailbox/store/MessageMovesWithMailbox.java | 152 +++++++++++++++++ .../james/mailbox/store/StoreMailboxManager.java | 5 + .../james/mailbox/store/StoreMessageIdManager.java | 186 +++++++++++++-------- .../james/mailbox/store/StoreRightManager.java | 1 + .../james/mailbox/store/mail/MessageIdMapper.java | 8 +- .../store/quota/DefaultUserQuotaRootResolver.java | 5 + .../AbstractMessageIdManagerSideEffectTest.java | 20 +-- .../store/AbstractMessageIdManagerStorageTest.java | 61 ++++--- .../mailbox/store/MessageManagerTestSystem.java | 1 - .../store/mail/model/MessageIdMapperTest.java | 26 +-- .../ListeningMessageSearchIndexContract.java | 2 + .../tools/indexer/CassandraReIndexerImplTest.java | 2 +- .../RecomputeCurrentQuotasServiceContract.java | 5 + .../transport/mailets/ICALToJsonAttributeTest.java | 3 +- .../james/transport/matchers/HasExceptionTest.java | 1 - .../transport/matchers/RecipientIsLocalTest.java | 5 +- .../decode/parser/SelectCommandParserTest.java | 1 - .../AbstractMailboxSelectionRequestTest.java | 2 +- .../UnpooledStatusResponseFactoryTest.java | 1 - .../processor/base/SelectedMailboxImplTest.java | 2 +- .../imap/processor/base/UidMsnConverterTest.java | 3 +- .../blob/api/BucketDumbBlobStoreContract.java | 1 - .../blob/api/MetricableBlobStoreContract.java | 1 - .../blob/api/ReadSaveDumbBlobStoreContract.java | 2 +- .../blob/cassandra/CassandraBlobStoreTest.java | 1 + .../blob/cassandra/cache/CachedBlobStoreTest.java | 2 +- .../cache/CassandraBlobStoreCacheTest.java | 6 +- .../cache/CassandraCacheConfigurationTest.java | 20 +-- .../blob/cassandra/utils/DataChunkerTest.java | 4 +- .../james/blob/memory/MemoryBlobStoreTest.java | 1 - .../org/apache/james/CassandraNodeConfTest.java | 1 + .../org/apache/james/AggregateJunitExtension.java | 1 + .../org/apache/james/GuiceModuleTestExtension.java | 4 +- .../apache/james/MailsShouldBeWellReceived.java | 2 +- .../mailbox/PreDeletionHookLoaderImplTest.java | 2 - .../dlp/api/DLPConfigurationStoreContract.java | 1 + .../apache/james/rrt/lib/CanSendFromContract.java | 2 +- .../apache/james/rrt/lib/DomainRewriterTest.java | 1 - .../apache/james/rrt/lib/MappingSourceTest.java | 1 - .../apache/james/rrt/lib/ReplaceRewriterTest.java | 1 - .../cassandra/CassandraUsersRepositoryTest.java | 2 +- .../api/filtering/FilteringAggregateIdTest.java | 1 - .../rrt/jpa/JPARecipientRewriteTableTest.java | 101 ++++++----- .../java/org/apache/james/rrt/jpa/JPAStepdefs.java | 1 + .../james/sieve/jpa/JpaSieveRepositoryTest.java | 1 - .../james/user/jpa/JpaUsersRepositoryTest.java | 2 +- .../user/ldap/ReadOnlyUsersLDAPRepositoryTest.java | 2 +- .../apache/james/rrt/memory/InMemoryStepdefs.java | 1 + .../user/memory/MemoryUsersRepositoryTest.java | 2 +- .../apache/james/jdkim/mailets/DKIMSignTest.java | 8 +- .../apache/james/jdkim/mailets/DKIMVerifyTest.java | 1 - .../james/mailets/AddDeliveredToHeaderTest.java | 2 +- .../james/mailets/BounceIntegrationTest.java | 6 +- .../mailets/CommonMailetConfigurationTest.java | 2 +- .../apache/james/mailets/DKIMIntegrationTest.java | 2 +- ...ectResolutionRemoteDeliveryIntegrationTest.java | 2 +- .../GatewayRemoteDeliveryIntegrationTest.java | 2 +- .../org/apache/james/mailets/MailetErrorsTest.java | 2 +- .../mailets/NetworkMatcherIntegrationTest.java | 2 +- .../RecipientRewriteTableIntegrationTest.java | 2 +- .../james/mailets/RemoteDeliveryErrorTest.java | 2 +- .../mailets/SenderIsLocalIntegrationTest.java | 2 +- .../org/apache/james/mailets/SieveDelivery.java | 2 +- .../james/mailets/SmtpAuthIntegrationTest.java | 2 +- .../crypto/SMIMEDecryptIntegrationTest.java | 2 +- .../mailets/crypto/SMIMESignIntegrationTest.java | 2 +- .../james/smtp/SmtpAuthorizedAddressesTest.java | 2 +- .../org/apache/james/smtp/SmtpContentTypeTest.java | 2 +- .../org/apache/james/smtp/SmtpNullSenderTest.java | 2 +- .../apache/james/smtp/SmtpRandomStoringTest.java | 2 +- .../james/smtp/extensions/SMTPHeloHooksTest.java | 2 +- .../james/smtp/extensions/StaticInputChecker.java | 3 +- .../james/transport/mailets/AddFooterTest.java | 2 +- .../james/transport/mailets/AliasMappingTest.java | 2 +- .../mailets/AmqpForwardAttachmentTest.java | 2 +- .../transport/mailets/ContactExtractorTest.java | 2 +- .../transport/mailets/DlpIntegrationTest.java | 2 +- .../james/transport/mailets/DomainMappingTest.java | 2 +- .../transport/mailets/GroupMappingRelayTest.java | 2 +- .../james/transport/mailets/GroupMappingTest.java | 2 +- .../mailets/ICSAttachmentWorkflowTest.java | 2 +- .../transport/mailets/IsOverQuotaMatcherTest.java | 2 +- .../mailets/NoClassDefFoundErrorMatcher.java | 1 - .../james/transport/mailets/SpamAssassinTest.java | 2 +- .../transport/mailets/StripAttachmentTest.java | 2 +- .../james/transport/mailets/ToRepositoryTest.java | 2 +- .../mailets/ToSenderDomainRepositoryTest.java | 2 +- .../org/apache/james/mock/smtp/server/Fixture.java | 1 + .../smtp/server/HTTPConfigurationServerTest.java | 2 +- .../mock/smtp/server/model/ConditionTest.java | 4 +- .../org/apache/james/jmap/MessageAppender.java | 4 +- .../integration/GetMessageListMethodTest.java | 4 +- .../methods/integration/SetMessagesMethodTest.java | 2 +- .../methods/integration/SpamAssassinContract.java | 2 +- .../methods/integration/SpamAssassinModule.java | 1 - .../methods/integration/cucumber/MainStepdefs.java | 2 +- .../jmap/memory/MemoryVacationIntegrationTest.java | 4 +- .../memory/MemoryVacationRelayIntegrationTest.java | 4 +- .../RabbitMQSetMailboxesMethodCucumberTest.java | 1 - .../RabbitMQSetMessagesMethodCucumberTest.java | 1 - .../cucumber/awss3/RabbitMQUploadCucumberTest.java | 1 - .../methods/GetVacationResponseMethodTest.java | 2 +- .../draft/methods/MIMEMessageConverterTest.java | 2 - .../jmap/draft/methods/RequestHandlerTest.java | 1 - .../jmap/draft/methods/SetMailboxesMethodTest.java | 2 +- .../methods/SetVacationResponseMethodTest.java | 3 +- .../draft/model/mailbox/MailboxNamespaceTest.java | 1 - .../jmap/draft/utils/FilterToCriteriaTest.java | 72 +++++--- .../jmap/http/JWTAuthenticationStrategyTest.java | 2 +- .../ExtractMDNOriginalJMAPMessageIdTest.java | 1 - .../james/jmap/mailet/VacationMailetTest.java | 1 + .../jmap/mailet/filter/JMAPFilteringTest.java | 1 - ...sandraMappingsSolveInconsistenciesTaskTest.java | 1 + .../webadmin/authentication/JwtFilterTest.java | 4 +- .../webadmin/routes/AddressMappingRoutesTest.java | 2 +- .../james/webadmin/routes/UserRoutesTest.java | 2 +- .../WebadminApiQuerySerializationContractTest.java | 54 +++--- ...erializableReIndexingExecutionFailuresTest.java | 2 +- .../routes/MailboxesRoutesNoTasksTest.java | 3 +- .../james/webadmin/routes/MailboxesRoutesTest.java | 2 +- .../service/DeleteMailsFromMailQueueTaskTest.java | 2 +- ...mMailQueueTaskAdditionalInformationDTOTest.java | 2 +- .../james/queue/api/DelayedMailQueueContract.java | 1 + .../apache/james/queue/api/MailQueueContract.java | 7 +- .../james/queue/api/MailQueueFactoryTest.java | 2 - .../queue/rabbitmq/RabbitMQMailQueueTest.java | 1 - .../rabbitmq/RabbitMqMailQueueFactoryTest.java | 6 - .../CassandraMailQueueViewTestFactory.java | 1 - .../view/cassandra/configuration/DTOTest.java | 2 +- .../james/task/MemoryReferenceWithCounterTask.java | 2 +- ...assandraTaskExecutionDetailsProjectionTest.java | 2 +- .../RabbitMQWorkQueuePersistenceTest.java | 8 +- .../EventSourcingTaskManagerTest.java | 13 +- .../task/eventsourcing/TaskAggregateTest.java | 5 +- .../TerminationSubscriberContract.java | 1 - .../org/apache/james/linshare/CombinedDecoder.java | 3 +- .../james/linshare/LinshareConfigurationTest.java | 8 +- .../org/apache/james/linshare/LinshareFixture.java | 3 +- .../org/apache/james/linshare/LinshareTest.java | 2 +- 185 files changed, 739 insertions(+), 462 deletions(-) create mode 100644 mailbox/store/src/main/java/org/apache/james/mailbox/store/MessageMovesWithMailbox.java --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org