JAMES-4156 refactoring Blobstore deleted message vault for appending messages into single bucket#5502
Closed
Arsnael wants to merge 435 commits intolinagora:masterfrom
Closed
JAMES-4156 refactoring Blobstore deleted message vault for appending messages into single bucket#5502Arsnael wants to merge 435 commits intolinagora:masterfrom
Arsnael wants to merge 435 commits intolinagora:masterfrom
Conversation
…test rather than rewrite them.
…nager to manipulate tests Without exercising the mapper API directly, which makes tests brittle and hard to manage.
- Reorganise the lookup DAO in order to include the ThreadId as part of the primary key and relegate the messageId as a clustering key. Acceptable for our usage pattern, this enables listing content of a thread. Migration strategy: none. Truncate the old v2 table and start with an empty v3 table which is acceptable. - Rely on this method for thread resolution instead of the search index - Pay an attention that messages in the tread are accessible to the end user (right resolution)
…iewWithAttachmentMetadataWhenSupportedBodyPropertiesAtAttachmentReadLevel (linagora#2873) Forcing the test on the CI, reports of issues down the line regarding PG integration tests
This is because th fast view projection might not be populated fast enough for triggering the attachment view.
This reverts commit e0a3ee7.
The previous fix was doing an await on the response without wrapping the request too
This is because the fast view projection might not be populated fast enough for triggering the attachment view.
locally at least this fixes the issue documented in apache#2842 (comment)
- Mutualize code for choosing how code shall be validated between IMAP and SMTP - Turn OidcJwtTokenVerifier as a class - Have SASL configuration as a member in order to limit count of arguments
…ingle bucket Keep old code as appendV1 for testing retro-compatibility
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replacing apache#2899