Re: [PR] Do not index body [james-project]

2024-02-25 Thread via GitHub
chibenwa merged PR #2018: URL: https://github.com/apache/james-project/pull/2018 -- 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:

Re: [PR] Do not index body [james-project]

2024-02-25 Thread via GitHub
chibenwa commented on PR #2018: URL: https://github.com/apache/james-project/pull/2018#issuecomment-1963076095 https://issues.apache.org/jira/browse/JAMES-4010 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] Do not index body [james-project]

2024-02-23 Thread via GitHub
Arsnael commented on PR #2018: URL: https://github.com/apache/james-project/pull/2018#issuecomment-1961382503 ``` 12:45:54,704 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.3.0:check (check-style) on project apache-james-mailbox-opensearch: You have 1

Re: [PR] Do not index body [james-project]

2024-02-23 Thread via GitHub
chibenwa commented on code in PR #2018: URL: https://github.com/apache/james-project/pull/2018#discussion_r1500462621 ## mailbox/opensearch/src/test/java/org/apache/james/mailbox/opensearch/OpenSearchNoIndexBodyIntegrationTest.java: ## @@ -0,0 +1,197 @@

Re: [PR] Do not index body [james-project]

2024-02-23 Thread via GitHub
chibenwa commented on code in PR #2018: URL: https://github.com/apache/james-project/pull/2018#discussion_r1500456865 ## src/site/xdoc/server/config-opensearch.xml: ## @@ -122,6 +122,8 @@ When set to true, James will attempt to reindex from the indexed message when

Re: [PR] Do not index body [james-project]

2024-02-23 Thread via GitHub
chibenwa commented on code in PR #2018: URL: https://github.com/apache/james-project/pull/2018#discussion_r1500455955 ## mailbox/opensearch/src/test/java/org/apache/james/mailbox/opensearch/OpenSearchNoIndexBodyIntegrationTest.java: ## @@ -0,0 +1,197 @@

Re: [PR] Do not index body [james-project]

2024-02-23 Thread via GitHub
chibenwa commented on code in PR #2018: URL: https://github.com/apache/james-project/pull/2018#discussion_r1500453764 ## mailbox/opensearch/src/main/java/org/apache/james/mailbox/opensearch/events/OpenSearchListeningMessageSearchIndex.java: ## @@ -296,7 +299,14 @@ private Mono

Re: [PR] Do not index body [james-project]

2024-02-23 Thread via GitHub
Arsnael commented on code in PR #2018: URL: https://github.com/apache/james-project/pull/2018#discussion_r1500446233 ## mailbox/opensearch/src/test/java/org/apache/james/mailbox/opensearch/OpenSearchNoIndexBodyIntegrationTest.java: ## @@ -0,0 +1,197 @@

Re: [PR] Do not index body [james-project]

2024-02-19 Thread via GitHub
chibenwa commented on PR #2018: URL: https://github.com/apache/james-project/pull/2018#issuecomment-1952218252 > But the non indexing of the body should not be enabled by default Of course. I'm asking for an emergency safety trigger. Not a default value. Like

Re: [PR] Do not index body [james-project]

2024-02-19 Thread via GitHub
Arsnael commented on PR #2018: URL: https://github.com/apache/james-project/pull/2018#issuecomment-1952148847 I get you can search on other header fields. I can admit maybe in some cases it could be wanted. But the non indexing of the body should not be enabled by default -- This is an

Re: [PR] Do not index body [james-project]

2024-02-19 Thread via GitHub
chibenwa commented on PR #2018: URL: https://github.com/apache/james-project/pull/2018#issuecomment-1952138840 Unavailable on body but you get all the other search properties. This is SUPER handy as a "panic fix" or for people wanting to compromise on the cost of search. (you

Re: [PR] Do not index body [james-project]

2024-02-18 Thread via GitHub
Arsnael commented on code in PR #2018: URL: https://github.com/apache/james-project/pull/2018#discussion_r1494069538 ## mailbox/store/src/main/java/org/apache/james/mailbox/store/search/ListeningMessageSearchIndex.java: ## @@ -132,7 +132,7 @@ protected Mono

[PR] Do not index body [james-project]

2024-02-18 Thread via GitHub
chibenwa opened a new pull request, #2018: URL: https://github.com/apache/james-project/pull/2018 This relax pressure on the AES blob store and APPEND Also this is helpfull upon mass data ingestion. TODO make this configurable -- This is an automated message from the Apache