Re: Product vs. Profile
Hello, Someone (aka Michael Bailly) suggested "Flavor" instead of "Product". I also think it's a nice way to speak about a product already configured to match some specific needs. Regards, Raphaël. Le 14/05/2020 à 13:52, David Leangen a écrit : Hello! While writing the documentation, I came across a term for which I would like to propose a change. Benoit asked that we hold a discussion here before making a final decision. I am proposing that we change “Product” (as in for example the "James with Guice + Cassandra + RabbitMQ + Swift + ElasticSearch **Product**”) to “Profile”. At the same time, I am also proposing some more general names. For example, the above "James with Guice + Cassandra + RabbitMQ + Swift + ElasticSearch **Product**” would become the “James Distributed Profile”. The explanation I gave previously for my issue with “product" was: Personally, I have a problem with "product”. […] I tend to think of a "product" as something a little different than what is intended here. Mainly, "product" seems to me to have a very commercial connotation in English. I think that is not what is intended here. Secondly, I feel that "product" for the average person probably refers to "Apache James" itself. If anything, these would be more like "sub-products" to me. “Profile” seems a much better fit, if you are willing to make this change. Cheers, =David - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] annotated tag james-project-3.5.0 deleted (was a98edb4)
This is an automated email from the ASF dual-hosted git repository. btellier pushed a change to annotated tag james-project-3.5.0 in repository https://gitbox.apache.org/repos/asf/james-project.git. *** WARNING: tag james-project-3.5.0 was deleted! *** tag was a98edb4 The revisions that were on this annotated tag are still contained in other references; therefore, this change does not discard any commits from the repository. - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] 06/06: Remove leading line breaks
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch 3.5.x in repository https://gitbox.apache.org/repos/asf/james-project.git commit ba532d0d5b9fb45344715ed4f5d6125535b41598 Author: Benoit Tellier AuthorDate: Thu Apr 9 10:00:29 2020 +0700 Remove leading line breaks --- dockerfiles/run/spring/destination/conf/log4j2.xml | 1 - server/app/src/main/resources/log4j2.xml | 1 - 2 files changed, 2 deletions(-) diff --git a/dockerfiles/run/spring/destination/conf/log4j2.xml b/dockerfiles/run/spring/destination/conf/log4j2.xml index c89ad2e..f8c9fad 100644 --- a/dockerfiles/run/spring/destination/conf/log4j2.xml +++ b/dockerfiles/run/spring/destination/conf/log4j2.xml @@ -1,4 +1,3 @@ - diff --git a/server/app/src/main/resources/log4j2.xml b/server/app/src/main/resources/log4j2.xml index c89ad2e..f8c9fad 100644 --- a/server/app/src/main/resources/log4j2.xml +++ b/server/app/src/main/resources/log4j2.xml @@ -1,4 +1,3 @@ - - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] 01/06: JAMES-1541 Remove no longer existing TarpitHandler from configuration
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch 3.5.x in repository https://gitbox.apache.org/repos/asf/james-project.git commit 492e458cda31509e9f894fdf9d7acbafc3bd771e Author: Benoit Tellier AuthorDate: Tue May 19 08:45:15 2020 +0700 JAMES-1541 Remove no longer existing TarpitHandler from configuration --- server/app/src/main/resources/smtpserver.xml | 11 --- 1 file changed, 11 deletions(-) diff --git a/server/app/src/main/resources/smtpserver.xml b/server/app/src/main/resources/smtpserver.xml index a26a06a..d0a5637 100644 --- a/server/app/src/main/resources/smtpserver.xml +++ b/server/app/src/main/resources/smtpserver.xml @@ -247,17 +247,6 @@ 127.0.0.0/8 --> - - - - - - - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] 05/06: JAMES-3187 Provisionned docker: add a cli helper script
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch 3.5.x in repository https://gitbox.apache.org/repos/asf/james-project.git commit 2207d8b3d833cefee9b63b7ecebe6433e5a41fc5 Author: Benoit Tellier AuthorDate: Mon May 18 18:18:55 2020 +0700 JAMES-3187 Provisionned docker: add a cli helper script --- dockerfiles/run/guice/provisioned/Dockerfile | 3 ++- dockerfiles/run/guice/provisioned/james-cli | 3 +++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/dockerfiles/run/guice/provisioned/Dockerfile b/dockerfiles/run/guice/provisioned/Dockerfile index 1acdd8a..a0fdade 100644 --- a/dockerfiles/run/guice/provisioned/Dockerfile +++ b/dockerfiles/run/guice/provisioned/Dockerfile @@ -2,7 +2,7 @@ # # VERSION 1.0 -FROM linagora/james-jpa-guice +FROM linagora/james-jpa-guice:james-project-3.4.0 # Install git RUN apt-get update @@ -15,6 +15,7 @@ RUN cp /root/wait-for-it/wait-for-it.sh /usr/bin/wait-for-it.sh COPY startup.sh /root COPY initialdata.sh /root +COPY james-cli /usr/local/bin/ RUN chmod +x /root/startup.sh RUN chmod +x /root/initialdata.sh diff --git a/dockerfiles/run/guice/provisioned/james-cli b/dockerfiles/run/guice/provisioned/james-cli new file mode 100755 index 000..752e29c --- /dev/null +++ b/dockerfiles/run/guice/provisioned/james-cli @@ -0,0 +1,3 @@ +#!/usr/bin/env bash + +java -jar /root/james-cli.jar -h 127.0.0.1 -p "$@" \ No newline at end of file - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] branch 3.5.x created (now ba532d0)
This is an automated email from the ASF dual-hosted git repository. btellier pushed a change to branch 3.5.x in repository https://gitbox.apache.org/repos/asf/james-project.git. at ba532d0 Remove leading line breaks This branch includes the following new commits: new 492e458 JAMES-1541 Remove no longer existing TarpitHandler from configuration new 04e8ec7 JAMES-1541 Remove no longer existing SuppressDuplicateRcptHandler from configuration new 537af6e JAMES-1541 Fix a typo in MailPriorityHandler in configuration new 3b4828c JAMES-3187 Add webadmin enabled by default to jpa-guice docker image new 2207d8b JAMES-3187 Provisionned docker: add a cli helper script new ba532d0 Remove leading line breaks The 6 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. - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] 02/06: JAMES-1541 Remove no longer existing SuppressDuplicateRcptHandler from configuration
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch 3.5.x in repository https://gitbox.apache.org/repos/asf/james-project.git commit 04e8ec790624066fac48f553a30ad6ae06632cdf Author: Benoit Tellier AuthorDate: Tue May 19 08:45:49 2020 +0700 JAMES-1541 Remove no longer existing SuppressDuplicateRcptHandler from configuration --- server/app/src/main/resources/smtpserver.xml | 6 -- 1 file changed, 6 deletions(-) diff --git a/server/app/src/main/resources/smtpserver.xml b/server/app/src/main/resources/smtpserver.xml index d0a5637..14e16a3 100644 --- a/server/app/src/main/resources/smtpserver.xml +++ b/server/app/src/main/resources/smtpserver.xml @@ -247,12 +247,6 @@ 127.0.0.0/8 --> - - - - - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] 03/06: JAMES-1541 Fix a typo in MailPriorityHandler in configuration
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch 3.5.x in repository https://gitbox.apache.org/repos/asf/james-project.git commit 537af6ed3288a41548e6c7ca41969c30420c7214 Author: Benoit Tellier AuthorDate: Tue May 19 08:46:45 2020 +0700 JAMES-1541 Fix a typo in MailPriorityHandler in configuration --- server/app/src/main/resources/smtpserver.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/app/src/main/resources/smtpserver.xml b/server/app/src/main/resources/smtpserver.xml index 14e16a3..7e6b01f 100644 --- a/server/app/src/main/resources/smtpserver.xml +++ b/server/app/src/main/resources/smtpserver.xml @@ -255,7 +255,7 @@
[james-project] 04/06: JAMES-3187 Add webadmin enabled by default to jpa-guice docker image
This is an automated email from the ASF dual-hosted git repository. btellier pushed a commit to branch 3.5.x in repository https://gitbox.apache.org/repos/asf/james-project.git commit 3b4828c069b3d69d97a64961c32e4c357668df13 Author: Benoit Tellier AuthorDate: Mon May 18 18:53:11 2020 +0700 JAMES-3187 Add webadmin enabled by default to jpa-guice docker image --- .../guice/jpa/destination/conf/webadmin.properties | 48 ++ 1 file changed, 48 insertions(+) diff --git a/dockerfiles/run/guice/jpa/destination/conf/webadmin.properties b/dockerfiles/run/guice/jpa/destination/conf/webadmin.properties new file mode 100644 index 000..ec014ea --- /dev/null +++ b/dockerfiles/run/guice/jpa/destination/conf/webadmin.properties @@ -0,0 +1,48 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +#http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. + +# This template file can be used as example for James Server configuration +# DO NOT USE IT AS SUCH AND ADAPT IT TO YOUR NEEDS + +# Read https://james.apache.org/server/config-webadmin.html for further details + +enabled=true +port=8000 +host=localhost + +# Defaults to false +https.enabled=false + +# Compulsory when enabling HTTPS +#https.keystore=/path/to/keystore +#https.password=password + +# Optional when enabling HTTPS (self signed) +#https.trust.keystore +#https.trust.password + +# Defaults to false +#jwt.enabled=true + +# Defaults to false +#cors.enable=true +#cors.origin + +# List of fully qualified class names that should be exposed over webadmin +# in addition to your product default routes. Routes needs to be located +# within the classpath or in the ./extensions-jars folder. +#extensions.routes= \ No newline at end of file - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] 04/06: Fix a Typo in WebAdmin documentation
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 7d6e6f39289d4c71075f00ceee6e8aa86a77bd95 Author: Andy Tan <18187444+tana...@users.noreply.github.com> AuthorDate: Mon May 25 09:06:59 2020 +0200 Fix a Typo in WebAdmin documentation --- src/site/markdown/server/manage-webadmin.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/site/markdown/server/manage-webadmin.md b/src/site/markdown/server/manage-webadmin.md index 5e6435e..43b3f26 100644 --- a/src/site/markdown/server/manage-webadmin.md +++ b/src/site/markdown/server/manage-webadmin.md @@ -491,7 +491,7 @@ Rerunning the task will *eventually* provide the consistent result. As such we a In order to ensure being offline, stop the traffic on SMTP, JMAP and IMAP ports, for example via re-configuration or firewall rules. -`trustMessageProjection` query parameter can be set to `true`. Cpntent of `messageIdTable` (listing messages by their +`trustMessageProjection` query parameter can be set to `true`. Content of `messageIdTable` (listing messages by their mailbox context) table will be trusted and not compared against content of `imapUidTable` table (listing messages by their messageId mailbox independent identifier). This will result in a better performance running the task at the cost of safety in the face of message denormalization inconsistencies. - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] 05/06: JAMES-3187 Add webadmin enabled by default to jpa-guice docker image
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 37fee88b1875c3c39a3be0091751774f4a69519e Author: Benoit Tellier AuthorDate: Mon May 18 18:53:11 2020 +0700 JAMES-3187 Add webadmin enabled by default to jpa-guice docker image --- .../guice/jpa/destination/conf/webadmin.properties | 48 ++ 1 file changed, 48 insertions(+) diff --git a/dockerfiles/run/guice/jpa/destination/conf/webadmin.properties b/dockerfiles/run/guice/jpa/destination/conf/webadmin.properties new file mode 100644 index 000..ec014ea --- /dev/null +++ b/dockerfiles/run/guice/jpa/destination/conf/webadmin.properties @@ -0,0 +1,48 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +#http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. + +# This template file can be used as example for James Server configuration +# DO NOT USE IT AS SUCH AND ADAPT IT TO YOUR NEEDS + +# Read https://james.apache.org/server/config-webadmin.html for further details + +enabled=true +port=8000 +host=localhost + +# Defaults to false +https.enabled=false + +# Compulsory when enabling HTTPS +#https.keystore=/path/to/keystore +#https.password=password + +# Optional when enabling HTTPS (self signed) +#https.trust.keystore +#https.trust.password + +# Defaults to false +#jwt.enabled=true + +# Defaults to false +#cors.enable=true +#cors.origin + +# List of fully qualified class names that should be exposed over webadmin +# in addition to your product default routes. Routes needs to be located +# within the classpath or in the ./extensions-jars folder. +#extensions.routes= \ No newline at end of file - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] 01/06: JAMES-1541 Remove no longer existing TarpitHandler from configuration
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 2acf1f4c434463219d9607f32d441ff06135d8db Author: Benoit Tellier AuthorDate: Tue May 19 08:45:15 2020 +0700 JAMES-1541 Remove no longer existing TarpitHandler from configuration --- server/app/src/main/resources/smtpserver.xml | 11 --- 1 file changed, 11 deletions(-) diff --git a/server/app/src/main/resources/smtpserver.xml b/server/app/src/main/resources/smtpserver.xml index a26a06a..d0a5637 100644 --- a/server/app/src/main/resources/smtpserver.xml +++ b/server/app/src/main/resources/smtpserver.xml @@ -247,17 +247,6 @@ 127.0.0.0/8 --> - - - - - - - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] 03/06: JAMES-1541 Fix a typo in MailPriorityHandler in configuration
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 301b976e954ff5e51e55a5afcd98d04c47d87c98 Author: Benoit Tellier AuthorDate: Tue May 19 08:46:45 2020 +0700 JAMES-1541 Fix a typo in MailPriorityHandler in configuration --- server/app/src/main/resources/smtpserver.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/app/src/main/resources/smtpserver.xml b/server/app/src/main/resources/smtpserver.xml index 14e16a3..7e6b01f 100644 --- a/server/app/src/main/resources/smtpserver.xml +++ b/server/app/src/main/resources/smtpserver.xml @@ -255,7 +255,7 @@
[james-project] 06/06: JAMES-3187 Provisionned docker: add a cli helper script
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 72165ab8f102599879aff28d165c5f3eaeab2d83 Author: Benoit Tellier AuthorDate: Mon May 18 18:18:55 2020 +0700 JAMES-3187 Provisionned docker: add a cli helper script --- dockerfiles/run/guice/provisioned/Dockerfile | 3 ++- dockerfiles/run/guice/provisioned/james-cli | 3 +++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/dockerfiles/run/guice/provisioned/Dockerfile b/dockerfiles/run/guice/provisioned/Dockerfile index 1acdd8a..a0fdade 100644 --- a/dockerfiles/run/guice/provisioned/Dockerfile +++ b/dockerfiles/run/guice/provisioned/Dockerfile @@ -2,7 +2,7 @@ # # VERSION 1.0 -FROM linagora/james-jpa-guice +FROM linagora/james-jpa-guice:james-project-3.4.0 # Install git RUN apt-get update @@ -15,6 +15,7 @@ RUN cp /root/wait-for-it/wait-for-it.sh /usr/bin/wait-for-it.sh COPY startup.sh /root COPY initialdata.sh /root +COPY james-cli /usr/local/bin/ RUN chmod +x /root/startup.sh RUN chmod +x /root/initialdata.sh diff --git a/dockerfiles/run/guice/provisioned/james-cli b/dockerfiles/run/guice/provisioned/james-cli new file mode 100755 index 000..752e29c --- /dev/null +++ b/dockerfiles/run/guice/provisioned/james-cli @@ -0,0 +1,3 @@ +#!/usr/bin/env bash + +java -jar /root/james-cli.jar -h 127.0.0.1 -p "$@" \ No newline at end of file - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] 02/06: JAMES-1541 Remove no longer existing SuppressDuplicateRcptHandler from configuration
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 47abdb41e972aa5e2217d066bc954931800ef128 Author: Benoit Tellier AuthorDate: Tue May 19 08:45:49 2020 +0700 JAMES-1541 Remove no longer existing SuppressDuplicateRcptHandler from configuration --- server/app/src/main/resources/smtpserver.xml | 6 -- 1 file changed, 6 deletions(-) diff --git a/server/app/src/main/resources/smtpserver.xml b/server/app/src/main/resources/smtpserver.xml index d0a5637..14e16a3 100644 --- a/server/app/src/main/resources/smtpserver.xml +++ b/server/app/src/main/resources/smtpserver.xml @@ -247,12 +247,6 @@ 127.0.0.0/8 --> - - - - - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[james-project] branch master updated (86fcef0 -> 72165ab)
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 86fcef0 JAMES-3158 Deprecate HybridBlobStore new 2acf1f4 JAMES-1541 Remove no longer existing TarpitHandler from configuration new 47abdb4 JAMES-1541 Remove no longer existing SuppressDuplicateRcptHandler from configuration new 301b976 JAMES-1541 Fix a typo in MailPriorityHandler in configuration new 7d6e6f3 Fix a Typo in WebAdmin documentation new 37fee88 JAMES-3187 Add webadmin enabled by default to jpa-guice docker image new 72165ab JAMES-3187 Provisionned docker: add a cli helper script The 6 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: .../destination/conf/webadmin.properties | 0 dockerfiles/run/guice/provisioned/Dockerfile | 3 ++- dockerfiles/run/guice/provisioned/james-cli | 3 +++ server/app/src/main/resources/smtpserver.xml | 19 +-- src/site/markdown/server/manage-webadmin.md | 2 +- 5 files changed, 7 insertions(+), 20 deletions(-) copy dockerfiles/run/guice/{cassandra => jpa}/destination/conf/webadmin.properties (100%) create mode 100755 dockerfiles/run/guice/provisioned/james-cli - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[jira] [Commented] (JAMES-1404) support IPv6 address on server address binding
[ https://issues.apache.org/jira/browse/JAMES-1404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17115843#comment-17115843 ] Scoopta commented on JAMES-1404: I personally agree that the bracket looks weird, another option is the DNS syntax commonly used by dig, dnsmasq, and other DNS stuff which would be something like 2001:db8:85a3:8d3:1319:8a2e:370:7348#443 I personally think it looks a lot better outside of URIs and it's not completely made up either as other things use it. > support IPv6 address on server address binding > -- > > Key: JAMES-1404 > URL: https://issues.apache.org/jira/browse/JAMES-1404 > Project: James Server > Issue Type: Sub-task >Reporter: TzeKai Lee >Priority: Major > Attachments: ipv6.patch, uri_syntax.patch > > > Class org.apache.james.protocols.lib.netty.AbstractConfigurableAsyncServer is > unable to parse IPv6 addresses as current code use split(':') to delimit > address and port. -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[jira] [Commented] (JAMES-3191) Key registration is slow
[ https://issues.apache.org/jira/browse/JAMES-3191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17115825#comment-17115825 ] Benoit Tellier commented on JAMES-3191: --- > Did you run a profiler or use glowroot APM to have some insights on where the > time is spent exactly ? Well, I used the glowroot APM but observation in rabbitMQ admin api shows messages are not consumed fast enough and queues are growing, and Grafana boards also support this analysis. I posted all relevant details in a (private) Linagora operational ticket. > I have serious doubt that raising channel number from 3 to 10 will change > latency from several minutes to some milliseconds > 1. How did you discovered it was related to the channel number? That's a blind proposition that of course need to be fact checked. (I have no real better idea to be honnest) > Why do we need RabbitMQ for a SELECT? Easy. Because: - 1. You need propagating IMAP unsolicitated notifications, that happens upon changes triggered by another protocol session - 2. Maintain the IMAP MSN <-> UID mapping So SelectedMailboxImpl is a mailbox that can be registered to get updates for that given mailbox. > Maybe it is not the channel problem, but the lock time, in this case, we > might do more than just sending a message over RabbitMQ, and we should > prepare the message, then request the channel. That could be related to the change removing the ".block() call" in ReactorRabbitMQChannelPool::borrow(). 4d0da67 `JAMES-2774 Avoid nested block in ReactorRabbitMQChannelPool` is IMO a suspect... > Key registration is slow > > > Key: JAMES-3191 > URL: https://issues.apache.org/jira/browse/JAMES-3191 > Project: James Server > Issue Type: Improvement > Components: eventbus, IMAPServer >Affects Versions: master >Reporter: Benoit Tellier >Priority: Major > Labels: perf > > We noticed that on *master* SELECT IMAP command is significatively slow > (several minutes) on top of the distributed profile. (See attached > instrumentation average time and percentiles) > A quick performance review links this to listener registration (by key). > A code review leads to: > - The low channel number (3) maybe this count can be safely raised to a > higher number? Like 10? Maybe even configurable? > - EventBus::Register operation is handling IO but is synchronous. The > reactor scheduler backing it up is unspecifed (thus is likely the parrallel > one). We should let the caller specify the scheduler he whishes to run. -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[jira] [Commented] (JAMES-3191) Key registration is slow
[ https://issues.apache.org/jira/browse/JAMES-3191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17115814#comment-17115814 ] Gautier DI FOLCO commented on JAMES-3191: - I have a few questions: # How did you discovered it was related to the channel number? # Why do we need RabbitMQ for a SELECT? Maybe it is not the channel problem, but the lock time, in this case, we might do more than just sending a message over RabbitMQ, and we should prepare the message, then request the channel. > Key registration is slow > > > Key: JAMES-3191 > URL: https://issues.apache.org/jira/browse/JAMES-3191 > Project: James Server > Issue Type: Improvement > Components: eventbus, IMAPServer >Affects Versions: master >Reporter: Benoit Tellier >Priority: Major > Labels: perf > > We noticed that on *master* SELECT IMAP command is significatively slow > (several minutes) on top of the distributed profile. (See attached > instrumentation average time and percentiles) > A quick performance review links this to listener registration (by key). > A code review leads to: > - The low channel number (3) maybe this count can be safely raised to a > higher number? Like 10? Maybe even configurable? > - EventBus::Register operation is handling IO but is synchronous. The > reactor scheduler backing it up is unspecifed (thus is likely the parrallel > one). We should let the caller specify the scheduler he whishes to run. -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[jira] [Commented] (JAMES-1404) support IPv6 address on server address binding
[ https://issues.apache.org/jira/browse/JAMES-1404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17115808#comment-17115808 ] Benoit Tellier commented on JAMES-1404: --- I converted this to a subtask of JAMES-401 > support IPv6 address on server address binding > -- > > Key: JAMES-1404 > URL: https://issues.apache.org/jira/browse/JAMES-1404 > Project: James Server > Issue Type: Sub-task >Reporter: TzeKai Lee >Priority: Major > Attachments: ipv6.patch, uri_syntax.patch > > > Class org.apache.james.protocols.lib.netty.AbstractConfigurableAsyncServer is > unable to parse IPv6 addresses as current code use split(':') to delimit > address and port. -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
[jira] [Updated] (JAMES-1404) support IPv6 address on server address binding
[ https://issues.apache.org/jira/browse/JAMES-1404?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benoit Tellier updated JAMES-1404: -- Parent: JAMES-401 Issue Type: Sub-task (was: New Feature) > support IPv6 address on server address binding > -- > > Key: JAMES-1404 > URL: https://issues.apache.org/jira/browse/JAMES-1404 > Project: James Server > Issue Type: Sub-task >Reporter: TzeKai Lee >Priority: Major > Attachments: ipv6.patch, uri_syntax.patch > > > Class org.apache.james.protocols.lib.netty.AbstractConfigurableAsyncServer is > unable to parse IPv6 addresses as current code use split(':') to delimit > address and port. -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
Re: Apache James top-level description
> We could very well change the "e" meaning in the James acronym too. > > Something like `JAva Mail Extensible Server/Solutions` ? Thanks for thinking about this. The “JAM” part of “JAMES” brought to mind “JAMstack”. Taken from this page: https://jamstack.wtf/ [JAM = JavaScript + APIs + Markup] For JAMES, JAM could be something like JVM - API - Mailet == JAM ES could be, as Benoit mentioned “Extensible Server / System / Solutions”
Re: Apache James top-level description
I am lurking a bit within that interesting conversation and might only answer some points I might have something to propose. This thread discusses what we try to do as a project. In my previous post on this list, I (tried to) explain the legacy behind the James project goals. But I of course agree rationalizing the James project goals to align it with what the current contributors try to deliver. Now branding being definitely funny: On 25/05/2020 03:54, Matthieu Baechler wrote: > [..] > Another point is "Enterprise": I think that any "Enterprise" software is > suspicious at best. Free Software should empower anybody. So I would > propose to drop that word. Coffee thoughts: We could very well change the "e" meaning in the James acronym too. Something like `JAva Mail Extensible Server/Solutions` ? > Finally, I think that Java is not a selling point any longer, the JVM is. > So I would rather propose "JVM Mail Server". I globally agree with this point, that converges with the growing usage of Scala programming language within the code base. Do you think we could keep the acronym `James`? A quick search on google for adjective starting by `A` lead to the following results: - `Adaptable` enforces the idea you can adapt James to your needs. However I would see it as a duplicate of the `Extensible` mentioned above. - `Advanced` is a bit more boiler plate - awesome :-) > > Coming back to the documentation: James is pointless without (at least) > Mailets. I would focus the documentation mainly on the Mail Server target > and would promote customization as a top-level concept of it. > > My 2 cents > > Cheers, > - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org