Re: [PR] JAMES-2586 Implement PostgresDomainList [james-project]

2023-11-22 Thread via GitHub
vttranlina commented on PR #1807: URL: https://github.com/apache/james-project/pull/1807#issuecomment-1823928483 > Not sure the last commit is very clean though... letting people judge. It looks quite tricks, but we can polish it later > Maybe create a class like

Re: [PR] JAMES-2586 Implement PostgresDomainList [james-project]

2023-11-22 Thread via GitHub
quantranhong1999 commented on PR #1807: URL: https://github.com/apache/james-project/pull/1807#issuecomment-1823820346 > Not sure the last commit is very clean though... letting people judge. Maybe create a class like `SessionWithInitializedTablesFactory implements Provider` to both

[PR] [WIP] JAMES-2586 Implement PostgresMailboxMessageDAO [james-project]

2023-11-22 Thread via GitHub
vttranlina opened a new pull request, #1812: URL: https://github.com/apache/james-project/pull/1812 jira https://issues.apache.org/jira/browse/JAMES-2586 resolve https://github.com/linagora/james-project/issues/4899 -- This is an automated message from the Apache Git Service. To

Re: [PR] [WIP] JAMES-2586 Implement PostgresMailboxMessageDAO [james-project]

2023-11-22 Thread via GitHub
vttranlina commented on PR #1812: URL: https://github.com/apache/james-project/pull/1812#issuecomment-1823880297 My concerns when implementing: - Should we need a `PostgresMailboxMessage implement MailboxMessage`, or try to `SimpleMailboxMessage` - Not sure about insert message

(james-site) branch asf-staging updated: Site checkin for project Apache James: Jenkins Tools

2023-11-22 Thread git-site-role
This is an automated email from the ASF dual-hosted git repository. git-site-role pushed a commit to branch asf-staging in repository https://gitbox.apache.org/repos/asf/james-site.git The following commit(s) were added to refs/heads/asf-staging by this push: new b7680bc5d Site checkin for

Re: [PR] JAMES-2586 postgres users dao and repository [james-project]

2023-11-22 Thread via GitHub
Arsnael merged PR #1803: URL: https://github.com/apache/james-project/pull/1803 -- 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:

(james-project) branch postgresql updated: JAMES-2586 postgres users dao and repository (#1803)

2023-11-22 Thread rcordier
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a commit to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git The following commit(s) were added to refs/heads/postgresql by this push: new 52ebcbc2c4 JAMES-2586 postgres

Re: [PR] JAMES-2586 Implement PostgresDomainList [james-project]

2023-11-22 Thread via GitHub
Arsnael commented on PR #1807: URL: https://github.com/apache/james-project/pull/1807#issuecomment-1823812555 Not sure the last commit is very clean though... letting people judge. Thanks to @quantranhong1999 for helping me on this one :) -- This is an automated message from the

Re: [PR] JAMES-2586 postgres users dao and repository [james-project]

2023-11-22 Thread via GitHub
Arsnael commented on PR #1803: URL: https://github.com/apache/james-project/pull/1803#issuecomment-1823766141 https://github.com/Arsnael/james-project/blob/domain-postgres/server/apps/postgres-app/src/main/resources/META-INF/persistence.xml#L35 Can we remove this line as well?

Re: [PR] [ENHANCEMENT] Improve MDN display for JMAP mail RFC-8621 [james-project]

2023-11-22 Thread via GitHub
Arsnael commented on PR #1809: URL: https://github.com/apache/james-project/pull/1809#issuecomment-1823745250 https://ci-builds.apache.org/job/james/job/ApacheJames/job/PR-1809/3/testReport/ Looks related to me -- This is an automated message from the Apache Git Service. To

Re: [PR] JAMES-3963 Sort From To Cc on display name first [james-project]

2023-11-22 Thread via GitHub
Arsnael commented on PR #1811: URL: https://github.com/apache/james-project/pull/1811#issuecomment-1823746367 ``` 13:26:04,446 [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] JAMES-2586 Adapt Postgres Uid/ModSeq provider to Mapper [james-project]

2023-11-22 Thread via GitHub
vttranlina commented on PR #1808: URL: https://github.com/apache/james-project/pull/1808#issuecomment-1822298290 > How does it depend on it? > > Cascading deletes? ORM on MAIL table,

Re: [PR] JAMES-2586 Implement PostgresDomainList [james-project]

2023-11-22 Thread via GitHub
Arsnael commented on code in PR #1807: URL: https://github.com/apache/james-project/pull/1807#discussion_r1401665489 ## server/apps/postgres-app/src/test/java/org/apache/james/PostgresJamesServerWithAuthenticatedDatabaseSqlValidationTest.java: ## @@ -35,7 +37,7 @@ class

Re: [PR] JAMES-2586 postgres users dao and repository [james-project]

2023-11-22 Thread via GitHub
hungphan227 commented on PR #1803: URL: https://github.com/apache/james-project/pull/1803#issuecomment-1822337175 https://ci-builds.apache.org/blue/organizations/jenkins/james%2FApacheJames/detail/PR-1803/7/tests The test that check the concurrency issue of postgres connection FAILED

Re: [PR] JAMES-2586 Implement PostgresUidProvider, PostgresModSeqProvider [james-project]

2023-11-22 Thread via GitHub
vttranlina commented on code in PR #1804: URL: https://github.com/apache/james-project/pull/1804#discussion_r1401656315 ## backends-common/postgres/src/main/java/org/apache/james/backends/postgres/utils/PostgresExecutor.java: ## @@ -39,13 +41,30 @@ public class

[PR] [ENHANCEMENT] Improve MDN display for JMAP mail RFC-8621 [james-project]

2023-11-22 Thread via GitHub
chibenwa opened a new pull request, #1809: URL: https://github.com/apache/james-project/pull/1809 CF https://github.com/linagora/tmail-flutter/issues/2331#issuecomment-1822230696 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [ENHANCEMENT] Improve MDN display for JMAP mail RFC-8621 [james-project]

2023-11-22 Thread via GitHub
Arsnael commented on PR #1809: URL: https://github.com/apache/james-project/pull/1809#issuecomment-1822431710 ``` 09:45:50,701 [ERROR] Failed to execute goal io.github.evis:scalafix-maven-plugin_2.13:0.1.6_0.9.34:scalafix (scala-check-style) on project james-server-jmap-rfc-8621:

Re: [PR] JAMES-2586 Implement PostgresDomainList [james-project]

2023-11-22 Thread via GitHub
Arsnael commented on code in PR #1807: URL: https://github.com/apache/james-project/pull/1807#discussion_r1401712387 ## server/apps/postgres-app/src/test/java/org/apache/james/PostgresJamesServerWithAuthenticatedDatabaseSqlValidationTest.java: ## @@ -35,7 +37,7 @@ class

Re: [PR] JAMES-2586 postgres users dao and repository [james-project]

2023-11-22 Thread via GitHub
chibenwa commented on PR #1803: URL: https://github.com/apache/james-project/pull/1803#issuecomment-1822358010 `java.util.concurrent.ExecutionException: java.lang.RuntimeException: io.r2dbc.postgresql.ExceptionFactory$PostgresqlBadGrammarException: [42P05] prepared statement "S_0" already

Re: [PR] JAMES-2586 Implement MailboxACL support for PostgresMailboxMapper [james-project]

2023-11-22 Thread via GitHub
vttranlina commented on code in PR #1806: URL: https://github.com/apache/james-project/pull/1806#discussion_r1401723082 ## backends-common/postgres/src/test/java/org/apache/james/backends/postgres/PostgresExtension.java: ## @@ -180,12 +181,33 @@ public ConnectionFactory

Re: [PR] JAMES-2586 Implement MailboxACL support for PostgresMailboxMapper [james-project]

2023-11-22 Thread via GitHub
quantranhong1999 commented on code in PR #1806: URL: https://github.com/apache/james-project/pull/1806#discussion_r1401739565 ## backends-common/postgres/src/test/java/org/apache/james/backends/postgres/PostgresExtension.java: ## @@ -180,12 +181,33 @@ public ConnectionFactory

Re: [PR] JAMES-2586 postgres users dao and repository [james-project]

2023-11-22 Thread via GitHub
hungphan227 commented on PR #1803: URL: https://github.com/apache/james-project/pull/1803#issuecomment-1822376386 > `java.util.concurrent.ExecutionException: java.lang.RuntimeException: io.r2dbc.postgresql.ExceptionFactory$PostgresqlBadGrammarException: [42P05] prepared statement "S_0"

[PR] JAMES-3963 Full text search: AND for combining phrase terms [james-project]

2023-11-22 Thread via GitHub
chibenwa opened a new pull request, #1810: URL: https://github.com/apache/james-project/pull/1810 OR default behaviour includes many not - so - related terms. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[PR] JAMES-3963 Sort From To Cc on display name first [james-project]

2023-11-22 Thread via GitHub
chibenwa opened a new pull request, #1811: URL: https://github.com/apache/james-project/pull/1811 TODO - [ ] backport on tmail... -- 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

(james-project) 01/02: JAMES-2586 Implement MailboxACL support for PostgresMailboxMapper

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git commit 69ebb847cbfe0e556e0023f3bddba36cc392f996 Author: Quan Tran AuthorDate: Tue Nov 21 15:28:02 2023 +0700

(james-project) branch postgresql updated (666c7e1b22 -> b32d220f37)

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a change to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git from 666c7e1b22 JAMES-2586 Implement PostgresExecutor Factory and Mailbox Aggregate Module new

Re: [PR] JAMES-2586 Implement MailboxACL support for PostgresMailboxMapper [james-project]

2023-11-22 Thread via GitHub
chibenwa merged PR #1806: URL: https://github.com/apache/james-project/pull/1806 -- 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:

(james-project) 02/02: JAMES-2586 Create hstore extension if needed upon James startup

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git commit b32d220f372e9abce53fd540e5cc61648a7fb8b2 Author: Quan Tran AuthorDate: Tue Nov 21 16:35:57 2023 +0700

(james-project) branch postgresql updated (02fbe4e47c -> 666c7e1b22)

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a change to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git from 02fbe4e47c JAMES-2586 Clean-up the provision.sh file of postgres-app new 275a8492a2 JAMES-2586

(james-project) 03/04: JAMES-2586 Implement PostgresModSeqProvider

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git commit d18939b3be5e8bc9e3939ebace721f0599f3898a Author: Tung Tran AuthorDate: Wed Nov 22 10:13:45 2023 +0700

(james-project) 02/04: JAMES-2586 Implement PostgresUidProvider

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git commit e84d5f1e1a42eefd6ac3bbbc7975e54dd7169f05 Author: Tung Tran AuthorDate: Wed Nov 22 10:12:28 2023 +0700

(james-project) 01/04: JAMES-2586 Parameterize MailboxSession for getUidProvider/getModSeqProvider methods in MailboxSessionMapperFactory

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git commit 275a8492a2fcab84b87af3390d56d996cb83eb7f Author: Tung Tran AuthorDate: Tue Nov 21 15:10:44 2023 +0700

(james-project) 04/04: JAMES-2586 Implement PostgresExecutor Factory and Mailbox Aggregate Module

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git commit 666c7e1b22f2da9be5148e7e3d409a6791a38e7f Author: Tung Tran AuthorDate: Wed Nov 22 10:16:27 2023 +0700

Re: [PR] JAMES-2586 Implement PostgresUidProvider, PostgresModSeqProvider [james-project]

2023-11-22 Thread via GitHub
chibenwa merged PR #1804: URL: https://github.com/apache/james-project/pull/1804 -- 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] JAMES-2586 Implement PostgresUidProvider, PostgresModSeqProvider [james-project]

2023-11-22 Thread via GitHub
Arsnael commented on code in PR #1804: URL: https://github.com/apache/james-project/pull/1804#discussion_r1401788346 ## backends-common/postgres/src/main/java/org/apache/james/backends/postgres/utils/PostgresExecutor.java: ## @@ -39,13 +41,30 @@ public class PostgresExecutor

Re: [PR] JAMES-2586 Implement MailboxACL support for PostgresMailboxMapper [james-project]

2023-11-22 Thread via GitHub
chibenwa commented on PR #1806: URL: https://github.com/apache/james-project/pull/1806#issuecomment-1822486341 Rebase needed -- 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.

Re: [PR] [Build] Some small enhancements to (try to) reduce James server boot time [james-project]

2023-11-22 Thread via GitHub
chibenwa merged PR #1800: URL: https://github.com/apache/james-project/pull/1800 -- 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:

(james-project) 03/06: [BUILD] Implement caching for ExtendedClassLoader

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git commit 69e90ce0e47a773cfb9374cca4bb03a0110560a5 Author: Benoit TELLIER AuthorDate: Fri Nov 17 14:29:14 2023 +0100

(james-project) branch master updated (3dd1b53612 -> caa28d5580)

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git from 3dd1b53612 [UPGRADE] amqp-client 5.18.0 -> 5.20.0 new c5d9cbf22f [BUILD] Disable JMX for

(james-project) 01/06: [BUILD] Disable JMX for MailDelivrerToHost object pool

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git commit c5d9cbf22f946eb26815b89f6880384caba85b24 Author: Benoit TELLIER AuthorDate: Fri Nov 17 13:46:45 2023 +0100

(james-project) 06/06: [BUILD] Per test index is actually faster

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git commit caa28d5580ffc309ad9789b7b77531a9676809dd Author: Benoit TELLIER AuthorDate: Mon Nov 20 22:19:38 2023 +0100

(james-project) 04/06: [BUILD] Avoid polling without delays for MemoryMailQueue

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git commit 3f24078c67eda1124bbede01dee2a2b8e2df60f2 Author: Benoit TELLIER AuthorDate: Fri Nov 17 17:26:09 2023 +0100

(james-project) 05/06: [BUILD] Lower password hasking requirements

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git commit 7e63c3e6e7f3f852c531ea7e1041d9a9bf951b52 Author: Benoit TELLIER AuthorDate: Fri Nov 17 17:29:56 2023 +0100

(james-project) 02/06: [BUILD] Disable Metrics logger

2023-11-22 Thread btellier
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git commit 9339116155f356e203ac25a9d60959597f3d5d96 Author: Benoit TELLIER AuthorDate: Fri Nov 17 13:47:26 2023 +0100