(james-site) branch asf-staging updated: Site checkin for project Apache James: Jenkins Tools
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 5dc9b1e83 Site checkin for project Apache James: Jenkins Tools 5dc9b1e83 is described below commit 5dc9b1e8392fbfc5272a15d8d039eb58adf55c66 Author: jenkins AuthorDate: Fri Mar 22 22:31:36 2024 + Site checkin for project Apache James: Jenkins Tools --- sitemap-james-distributed-app.xml | 140 +-- sitemap-james-project.xml | 286 +++--- sitemap-james-site.xml| 4 +- 3 files changed, 215 insertions(+), 215 deletions(-) diff --git a/sitemap-james-distributed-app.xml b/sitemap-james-distributed-app.xml index e4c2f1629..0e558e2a7 100644 --- a/sitemap-james-distributed-app.xml +++ b/sitemap-james-distributed-app.xml @@ -2,282 +2,282 @@ http://www.sitemaps.org/schemas/sitemap/0.9;> https://james.apache.org/james-distributed-app/3.8.1/architecture/consistency-model.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/architecture/implemented-standards.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/architecture/index.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/architecture/specialized-instances.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/benchmark/db-benchmark.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/benchmark/index.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/benchmark/james-benchmark.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/batchsizes.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/blobstore.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/cassandra.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/collecting-contacts.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/collecting-events.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/dns.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/domainlist.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/dsn.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/extensions.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/healthcheck.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/imap.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/index.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/jmap.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/jmx.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/jvm.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/listeners.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/mailetcontainer.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/mailets.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/mailrepositorystore.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/matchers.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/opensearch.html -2024-03-22T10:44:04.434Z +2024-03-22T22:30:31.355Z https://james.apache.org/james-distributed-app/3.8.1/configure/pop3.html -2024-03-22T10:44:04.434Z
Re: [PR] [UPDATE] more javax APIs migrated to jakarta [james-project]
romainmoreau commented on code in PR #2141: URL: https://github.com/apache/james-project/pull/2141#discussion_r1535772288 ## server/apps/cassandra-app/src/assemble/license-for-binary.txt: ## @@ -1103,37 +1103,3 @@ Within the lib directory implementations of the Specification to develop, distribute and use a Compliant Implementation. - -licensed under the BSD (3-clause style) http://jetm.void.fm/license.html (as follows) - -Copyright (c) 2004, 2005, 2006, 2007 void.fm -All rights reserved. - -Redistribution and use in source and binary forms, with or without modification, -are permitted provided that the following conditions are met: - -* Redistributions of source code must retain the above copyright notice, this list - of conditions and the following disclaimer. - -* Redistributions in binary form must reproduce the above copyright notice, this - list of conditions and the following disclaimer in the documentation and/or - other materials provided with the distribution. - -* Neither the name void.fm nor the names of its contributors may be - used to endorse or promote products derived from this software without specific - prior written permission. - -THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND -ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED -WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. -IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, -INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, -BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, -DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF -LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE -OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED -OF THE POSSIBILITY OF SUCH DAMAGE. - -from JETM http://jetm.void.fm -jetm-1.2.3.jar -jetm-optional-1.2.3.jar Review Comment: My understanding is that it was removed a long time ago in 91848703279fa96cd1ab0b202b63554655e02fe7, and I suspect the files changed in this PR were overlooked. -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] [UPDATE] more javax APIs migrated to jakarta [james-project]
chibenwa merged PR #2141: URL: https://github.com/apache/james-project/pull/2141 -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] [UPDATE] more javax APIs migrated to jakarta [james-project]
chibenwa commented on PR #2141: URL: https://github.com/apache/james-project/pull/2141#issuecomment-2015253772 I am merging this in order to reduce the rebase pain... -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] [UPDATE] more javax APIs migrated to jakarta [james-project]
chibenwa commented on code in PR #2141: URL: https://github.com/apache/james-project/pull/2141#discussion_r1535691336 ## server/apps/cassandra-app/src/assemble/license-for-binary.txt: ## @@ -1103,37 +1103,3 @@ Within the lib directory implementations of the Specification to develop, distribute and use a Compliant Implementation. - -licensed under the BSD (3-clause style) http://jetm.void.fm/license.html (as follows) - -Copyright (c) 2004, 2005, 2006, 2007 void.fm -All rights reserved. - -Redistribution and use in source and binary forms, with or without modification, -are permitted provided that the following conditions are met: - -* Redistributions of source code must retain the above copyright notice, this list - of conditions and the following disclaimer. - -* Redistributions in binary form must reproduce the above copyright notice, this - list of conditions and the following disclaimer in the documentation and/or - other materials provided with the distribution. - -* Neither the name void.fm nor the names of its contributors may be - used to endorse or promote products derived from this software without specific - prior written permission. - -THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND -ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED -WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. -IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, -INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, -BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, -DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF -LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE -OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED -OF THE POSSIBILITY OF SUCH DAMAGE. - -from JETM http://jetm.void.fm -jetm-1.2.3.jar -jetm-optional-1.2.3.jar Review Comment: We are no longer using this, correct? -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] [Netty] Size estimator for ChunkedStream [james-project]
chibenwa commented on PR #2145: URL: https://github.com/apache/james-project/pull/2145#issuecomment-2015166917 See https://github.com/apache/james-project/pull/2149 Instead of inferring the size of the stream, which needs knowledge on the fact it is memory based is infered. Preserving the knowledge that it is memory backed all along the chain, along with a tiny fix, enables accurate back pressure. -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] [Netty] Size estimator for ChunkedStream [james-project]
chibenwa closed pull request #2145: [Netty] Size estimator for ChunkedStream URL: https://github.com/apache/james-project/pull/2145 -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
(james-project) branch master updated: [FIX] Avoid sending bounce when reprocessing (#2139)
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 The following commit(s) were added to refs/heads/master by this push: new 4e2bb0b57b [FIX] Avoid sending bounce when reprocessing (#2139) 4e2bb0b57b is described below commit 4e2bb0b57bd949709a92978ba312bdfb87a04727 Author: Benoit TELLIER AuthorDate: Fri Mar 22 14:55:42 2024 +0100 [FIX] Avoid sending bounce when reprocessing (#2139) --- server/apps/cassandra-app/sample-configuration/mailetcontainer.xml | 5 - .../apps/distributed-app/sample-configuration/mailetcontainer.xml | 5 - server/apps/distributed-app/src/test/resources/mailetcontainer.xml | 5 - .../distributed-pop3-app/sample-configuration/mailetcontainer.xml | 5 - .../distributed-pop3-app/src/test/resources/mailetcontainer.xml | 5 - server/apps/jpa-app/sample-configuration/mailetcontainer.xml| 5 - server/apps/jpa-app/src/test/resources/mailetcontainer.xml | 5 - server/apps/jpa-smtp-app/sample-configuration/mailetcontainer.xml | 5 - server/apps/jpa-smtp-app/src/test/resources/mailetcontainer.xml | 5 - server/apps/memory-app/sample-configuration/mailetcontainer.xml | 5 - server/apps/memory-app/src/test/resources/mailetcontainer.xml | 5 - .../scaling-pulsar-smtp/sample-configuration/mailetcontainer.xml| 5 - .../apps/scaling-pulsar-smtp/src/test/resources/mailetcontainer.xml | 5 - server/apps/spring-app/src/main/resources/mailetcontainer.xml | 6 +- 14 files changed, 57 insertions(+), 14 deletions(-) diff --git a/server/apps/cassandra-app/sample-configuration/mailetcontainer.xml b/server/apps/cassandra-app/sample-configuration/mailetcontainer.xml index f9c62031f0..bcd4a99f11 100644 --- a/server/apps/cassandra-app/sample-configuration/mailetcontainer.xml +++ b/server/apps/cassandra-app/sample-configuration/mailetcontainer.xml @@ -46,10 +46,13 @@ + + + mailetContainerErrors - + ignore diff --git a/server/apps/distributed-app/sample-configuration/mailetcontainer.xml b/server/apps/distributed-app/sample-configuration/mailetcontainer.xml index f9c62031f0..7885ff1796 100644 --- a/server/apps/distributed-app/sample-configuration/mailetcontainer.xml +++ b/server/apps/distributed-app/sample-configuration/mailetcontainer.xml @@ -46,10 +46,13 @@ + + + mailetContainerErrors - + ignore diff --git a/server/apps/distributed-app/src/test/resources/mailetcontainer.xml b/server/apps/distributed-app/src/test/resources/mailetcontainer.xml index 62a33ce870..119f0f89c1 100644 --- a/server/apps/distributed-app/src/test/resources/mailetcontainer.xml +++ b/server/apps/distributed-app/src/test/resources/mailetcontainer.xml @@ -40,7 +40,10 @@ - + + + + ignore diff --git a/server/apps/distributed-pop3-app/sample-configuration/mailetcontainer.xml b/server/apps/distributed-pop3-app/sample-configuration/mailetcontainer.xml index 945deba91f..19a839cdc1 100644 --- a/server/apps/distributed-pop3-app/sample-configuration/mailetcontainer.xml +++ b/server/apps/distributed-pop3-app/sample-configuration/mailetcontainer.xml @@ -46,10 +46,13 @@ + + + mailetContainerErrors - + ignore diff --git a/server/apps/distributed-pop3-app/src/test/resources/mailetcontainer.xml b/server/apps/distributed-pop3-app/src/test/resources/mailetcontainer.xml index 16cf49b911..cb0a08af66 100644 --- a/server/apps/distributed-pop3-app/src/test/resources/mailetcontainer.xml +++ b/server/apps/distributed-pop3-app/src/test/resources/mailetcontainer.xml @@ -40,7 +40,10 @@ - + + + + ignore diff --git a/server/apps/jpa-app/sample-configuration/mailetcontainer.xml b/server/apps/jpa-app/sample-configuration/mailetcontainer.xml index 90cbcedef1..55e84ba625 100644 --- a/server/apps/jpa-app/sample-configuration/mailetcontainer.xml +++ b/server/apps/jpa-app/sample-configuration/mailetcontainer.xml @@ -46,10 +46,13 @@ + + + mailetContainerErrors - + ignore diff
Re: [PR] [FIX] Avoid sending bounce when reprocessing [james-project]
chibenwa merged PR #2139: URL: https://github.com/apache/james-project/pull/2139 -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
(james-project) branch dependabot/maven/org.apache.commons-commons-configuration2-2.10.1 deleted (was a7368dcd45)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/org.apache.commons-commons-configuration2-2.10.1 in repository https://gitbox.apache.org/repos/asf/james-project.git was a7368dcd45 Bump org.apache.commons:commons-configuration2 from 2.9.0 to 2.10.1 The revisions that were on this branch are still contained in other references; therefore, this change does not discard any commits from the repository. - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
(james-project) branch master updated: Bump org.apache.commons:commons-configuration2 from 2.9.0 to 2.10.1 (#2147)
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 The following commit(s) were added to refs/heads/master by this push: new f038aa737d Bump org.apache.commons:commons-configuration2 from 2.9.0 to 2.10.1 (#2147) f038aa737d is described below commit f038aa737d58d4581d1a3f90df81684ff1ab3ec0 Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> AuthorDate: Fri Mar 22 14:55:00 2024 +0100 Bump org.apache.commons:commons-configuration2 from 2.9.0 to 2.10.1 (#2147) Bumps org.apache.commons:commons-configuration2 from 2.9.0 to 2.10.1. --- updated-dependencies: - dependency-name: org.apache.commons:commons-configuration2 dependency-type: direct:production ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index cad9398faf..be1301d2b8 100644 --- a/pom.xml +++ b/pom.xml @@ -2585,7 +2585,7 @@ org.apache.commons commons-configuration2 -2.9.0 +2.10.1 commons-logging - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] Bump org.apache.commons:commons-configuration2 from 2.9.0 to 2.10.1 [james-project]
chibenwa merged PR #2147: URL: https://github.com/apache/james-project/pull/2147 -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] POC IMAP FETCH with large body should not rely on in-memory buffering [james-project]
chibenwa commented on PR #2137: URL: https://github.com/apache/james-project/pull/2137#issuecomment-2015113954 Blocking reads is a major issue I do not feel confident solving for the AES blobstore (which I **need** to do) See https://github.com/apache/james-project/pull/2149 as an alternative for reasonable memory usage on IMAP FETCH -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] POC IMAP FETCH with large body should not rely on in-memory buffering [james-project]
chibenwa closed pull request #2137: POC IMAP FETCH with large body should not rely on in-memory buffering URL: https://github.com/apache/james-project/pull/2137 -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
[PR] JAMES-3977 Accurate backpressure for IMAP FETCH [james-project]
chibenwa opened a new pull request, #2149: URL: https://github.com/apache/james-project/pull/2149 Backpressure results. Set up: ``` channel.config().setWriteBufferWaterMark(new WriteBufferWaterMark(100 * 1024, 500 * 1014)); ``` (100 KB low level watermark, 500 KB high level watermark) higher than the defaults to fasten overall streaming. Results: Message size (bytes) | Count at which backpressure trigger | Total memory loaded | -|-|-| 4532 | NEVER | 2.3 MB | 9032 | 349 | 3.2 MB | 18032| 176 | 3.2 MB | 36032| 86 | 3.1 MB | 72032| 44 | 3.2 MB | 144032 | 22 | 3.2 MB | 288032 | 11 | 3.2 MB | 576032 | 5 | 2.9 MB | 1152032 | 3 | 4.6 MB | 2304032 | 2 (min)| 4.6 MB | 4608032 | 2 | 9.2 MB | -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] [Netty] Size estimator for ChunkedStream [james-project]
chibenwa commented on code in PR #2145: URL: https://github.com/apache/james-project/pull/2145#discussion_r1535378432 ## server/protocols/protocols-imap4/src/main/java/org/apache/james/imapserver/netty/ChunkedStreamWithSize.java: ## @@ -0,0 +1,150 @@ +/ + * 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. * + / + +package org.apache.james.imapserver.netty; + +import java.io.InputStream; +import java.io.PushbackInputStream; + +import io.netty.buffer.ByteBuf; +import io.netty.buffer.ByteBufAllocator; +import io.netty.channel.ChannelHandlerContext; +import io.netty.handler.stream.ChunkedInput; +import io.netty.util.internal.ObjectUtil; + +/** + * A clone of io.netty.handler.stream.ChunkedStream class, with getChunkSize method added + */ +public class ChunkedStreamWithSize implements ChunkedInput { + +static final int DEFAULT_CHUNK_SIZE = 8192; + +private final PushbackInputStream in; +private final int chunkSize; +private long offset; +private boolean closed; + +/** + * Creates a new instance that fetches data from the specified stream. + */ +public ChunkedStreamWithSize(InputStream in) { +this(in, DEFAULT_CHUNK_SIZE); +} + +/** + * Creates a new instance that fetches data from the specified stream. + * + * @param chunkSize the number of bytes to fetch on each + * {@link #readChunk(ChannelHandlerContext)} call + */ +public ChunkedStreamWithSize(InputStream in, int chunkSize) { +ObjectUtil.checkNotNull(in, "in"); +ObjectUtil.checkPositive(chunkSize, "chunkSize"); + +if (in instanceof PushbackInputStream) { +this.in = (PushbackInputStream) in; +} else { +this.in = new PushbackInputStream(in); +} +this.chunkSize = chunkSize; +} + +/** + * Returns the number of transferred bytes. + */ +public long transferredBytes() { +return offset; +} + +@Override +public boolean isEndOfInput() throws Exception { +if (closed) { +return true; +} +if (in.available() > 0) { +return false; +} + +int b = in.read(); +if (b < 0) { +return true; +} else { +in.unread(b); +return false; +} +} + +@Override +public void close() throws Exception { +closed = true; +in.close(); +} + +@Deprecated +@Override +public ByteBuf readChunk(ChannelHandlerContext ctx) throws Exception { +return readChunk(ctx.alloc()); +} + +@Override +public ByteBuf readChunk(ByteBufAllocator allocator) throws Exception { +if (isEndOfInput()) { +return null; +} + +final int availableBytes = in.available(); +final int chunkSize; +if (availableBytes <= 0) { +chunkSize = this.chunkSize; +} else { +chunkSize = Math.min(this.chunkSize, in.available()); +} + +boolean release = true; +ByteBuf buffer = allocator.buffer(chunkSize); +try { +// transfer to buffer +int written = buffer.writeBytes(in, chunkSize); +if (written < 0) { +return null; +} +offset += written; +release = false; +return buffer; +} finally { +if (release) { +buffer.release(); +} +} +} + +@Override +public long length() { +return -1; +} + +@Override +public long progress() { +return offset; +} + +public int getChunkSize() { +return chunkSize; +} Review Comment: Well James do actually chucks some kinda transformations of "ByteArrayInputStream".
(james-project) 01/05: JAMES-2586 Avoid declare jooq and r2dbc-postgresql version in multiple places
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 commit 617d6e10355be6056f6f9a3c35cf9e367c24a769 Author: Quan Tran AuthorDate: Thu Mar 21 10:58:21 2024 +0700 JAMES-2586 Avoid declare jooq and r2dbc-postgresql version in multiple places Was declared in the backend-common postgres module, and it is enough. --- server/blob/blob-postgres/pom.xml | 20 1 file changed, 20 deletions(-) diff --git a/server/blob/blob-postgres/pom.xml b/server/blob/blob-postgres/pom.xml index 09ab43e02b..d5bb4bfd06 100644 --- a/server/blob/blob-postgres/pom.xml +++ b/server/blob/blob-postgres/pom.xml @@ -31,11 +31,6 @@ Apache James :: Server :: Blob :: Postgres - -3.16.22 -1.0.2.RELEASE - - ${james.groupId} @@ -106,26 +101,11 @@ awaitility test - -org.jooq -jooq -${jooq.version} - - -org.jooq -jooq-postgres-extensions -${jooq.version} - org.mockito mockito-core test - -org.postgresql -r2dbc-postgresql -${r2dbc.postgresql.version} - org.testcontainers junit-jupiter - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
(james-project) 05/05: JAMES-2586 Postgres RewriteTablesTest should not fail unstable test phase
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 commit d7a19ef054aeec7d2ef22e7800acef8d37077c53 Author: Quan Tran AuthorDate: Fri Mar 22 12:34:37 2024 +0700 JAMES-2586 Postgres RewriteTablesTest should not fail unstable test phase cf: https://github.com/apache/james-project/pull/1963/commits/c6f2462a8d8cf7b90c1509ca9c13ae16ecc74ba5 --- .../src/test/java/org/apache/james/rrt/postgres/RewriteTablesTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/data/data-postgres/src/test/java/org/apache/james/rrt/postgres/RewriteTablesTest.java b/server/data/data-postgres/src/test/java/org/apache/james/rrt/postgres/RewriteTablesTest.java index ee1e00e3f5..e6f3e2cef2 100644 --- a/server/data/data-postgres/src/test/java/org/apache/james/rrt/postgres/RewriteTablesTest.java +++ b/server/data/data-postgres/src/test/java/org/apache/james/rrt/postgres/RewriteTablesTest.java @@ -25,7 +25,7 @@ import org.junit.platform.suite.api.IncludeEngines; import org.junit.platform.suite.api.SelectClasspathResource; import org.junit.platform.suite.api.Suite; -@Suite +@Suite(failIfNoTests = false) @IncludeEngines("cucumber") @SelectClasspathResource("cucumber") @ConfigurationParameter(key = GLUE_PROPERTY_NAME, value = "org.apache.james.rrt.lib,org.apache.james.rrt.postgres") - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
(james-project) 02/05: JAMES-2586 Bump jOOQ to 3.19.6
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 commit 990446c65c239fd430defedca3374370ca4170a4 Author: Quan Tran AuthorDate: Thu Mar 21 10:58:43 2024 +0700 JAMES-2586 Bump jOOQ to 3.19.6 --- backends-common/postgres/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/backends-common/postgres/pom.xml b/backends-common/postgres/pom.xml index 2ec6e658a5..2b4ec8797e 100644 --- a/backends-common/postgres/pom.xml +++ b/backends-common/postgres/pom.xml @@ -29,7 +29,7 @@ Apache James :: Backends Common :: Postgres -3.16.23 +3.19.6 1.0.3.RELEASE - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
(james-project) 03/05: JAMES-2586 Bump r2dbc-postgresql to 1.0.4
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 commit c19bcb378befe1594950713c8cc298cbad898162 Author: Quan Tran AuthorDate: Thu Mar 21 11:59:30 2024 +0700 JAMES-2586 Bump r2dbc-postgresql to 1.0.4 --- backends-common/postgres/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/backends-common/postgres/pom.xml b/backends-common/postgres/pom.xml index 2b4ec8797e..437c49bd53 100644 --- a/backends-common/postgres/pom.xml +++ b/backends-common/postgres/pom.xml @@ -30,7 +30,7 @@ 3.19.6 -1.0.3.RELEASE +1.0.4.RELEASE - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
(james-project) 04/05: JAMES-2586 Adapt jooq 3.19.6 change
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 commit e4ddaf88e24843abf7d0473a142252fdb19f843d Author: Quan Tran AuthorDate: Fri Mar 22 10:07:29 2024 +0700 JAMES-2586 Adapt jooq 3.19.6 change --- .../org/apache/james/backends/postgres/utils/PostgresExecutor.java| 3 +-- .../java/org/apache/james/sieve/postgres/PostgresSieveRepository.java | 2 +- .../java/org/apache/james/sieve/postgres/PostgresSieveScriptDAO.java | 4 ++-- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/backends-common/postgres/src/main/java/org/apache/james/backends/postgres/utils/PostgresExecutor.java b/backends-common/postgres/src/main/java/org/apache/james/backends/postgres/utils/PostgresExecutor.java index 37d3726e14..4bfb730ab0 100644 --- a/backends-common/postgres/src/main/java/org/apache/james/backends/postgres/utils/PostgresExecutor.java +++ b/backends-common/postgres/src/main/java/org/apache/james/backends/postgres/utils/PostgresExecutor.java @@ -138,10 +138,9 @@ public class PostgresExecutor { .map(record -> record.get(0, Boolean.class)); } -public Mono executeReturnAffectedRowsCount(Function> queryFunction) { +public Mono executeReturnAffectedRowsCount(Function> queryFunction) { return dslContext() .flatMap(queryFunction) -.cast(Long.class) .retryWhen(Retry.backoff(MAX_RETRY_ATTEMPTS, MIN_BACKOFF) .filter(preparedStatementConflictException())); } diff --git a/server/data/data-postgres/src/main/java/org/apache/james/sieve/postgres/PostgresSieveRepository.java b/server/data/data-postgres/src/main/java/org/apache/james/sieve/postgres/PostgresSieveRepository.java index f9b09e8eab..0fb63a018f 100644 --- a/server/data/data-postgres/src/main/java/org/apache/james/sieve/postgres/PostgresSieveRepository.java +++ b/server/data/data-postgres/src/main/java/org/apache/james/sieve/postgres/PostgresSieveRepository.java @@ -196,7 +196,7 @@ public class PostgresSieveRepository implements SieveRepository { @Override public void renameScript(Username username, ScriptName oldName, ScriptName newName) throws DuplicateException, ScriptNotFoundException { try { -long renamedScripts = postgresSieveScriptDAO.renameScript(username, oldName, newName).block(); +int renamedScripts = postgresSieveScriptDAO.renameScript(username, oldName, newName).block(); if (renamedScripts == 0) { throw new ScriptNotFoundException(); } diff --git a/server/data/data-postgres/src/main/java/org/apache/james/sieve/postgres/PostgresSieveScriptDAO.java b/server/data/data-postgres/src/main/java/org/apache/james/sieve/postgres/PostgresSieveScriptDAO.java index 88ff9c4034..92e81ce347 100644 --- a/server/data/data-postgres/src/main/java/org/apache/james/sieve/postgres/PostgresSieveScriptDAO.java +++ b/server/data/data-postgres/src/main/java/org/apache/james/sieve/postgres/PostgresSieveScriptDAO.java @@ -53,7 +53,7 @@ public class PostgresSieveScriptDAO { this.postgresExecutor = postgresExecutor; } -public Mono upsertScript(PostgresSieveScript sieveScript) { +public Mono upsertScript(PostgresSieveScript sieveScript) { return postgresExecutor.executeReturnAffectedRowsCount(dslContext -> Mono.from(dslContext.insertInto(TABLE_NAME) .set(SCRIPT_ID, sieveScript.getId().getValue()) .set(USERNAME, sieveScript.getUsername()) @@ -128,7 +128,7 @@ public class PostgresSieveScriptDAO { IS_ACTIVE.eq(true; } -public Mono renameScript(Username username, ScriptName oldName, ScriptName newName) { +public Mono renameScript(Username username, ScriptName oldName, ScriptName newName) { return postgresExecutor.executeReturnAffectedRowsCount(dslContext -> Mono.from(dslContext.update(TABLE_NAME) .set(SCRIPT_NAME, newName.getValue()) .where(USERNAME.eq(username.asString()), - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
(james-project) branch postgresql updated (e9dc71cf97 -> d7a19ef054)
This is an automated email from the ASF dual-hosted git repository. rcordier pushed a change to branch postgresql in repository https://gitbox.apache.org/repos/asf/james-project.git from e9dc71cf97 JAMES-2586 Fix MailboxSetMethodContract new 617d6e1035 JAMES-2586 Avoid declare jooq and r2dbc-postgresql version in multiple places new 990446c65c JAMES-2586 Bump jOOQ to 3.19.6 new c19bcb378b JAMES-2586 Bump r2dbc-postgresql to 1.0.4 new e4ddaf88e2 JAMES-2586 Adapt jooq 3.19.6 change new d7a19ef054 JAMES-2586 Postgres RewriteTablesTest should not fail unstable test phase The 5 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: backends-common/postgres/pom.xml | 4 ++-- .../backends/postgres/utils/PostgresExecutor.java| 3 +-- server/blob/blob-postgres/pom.xml| 20 .../sieve/postgres/PostgresSieveRepository.java | 2 +- .../james/sieve/postgres/PostgresSieveScriptDAO.java | 4 ++-- .../apache/james/rrt/postgres/RewriteTablesTest.java | 2 +- 6 files changed, 7 insertions(+), 28 deletions(-) - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] JAMES-2586 Bump jOOQ to 3.19.6 [james-project]
Arsnael merged PR #2146: URL: https://github.com/apache/james-project/pull/2146 -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] JAMES-2586 Bump jOOQ to 3.19.6 [james-project]
quantranhong1999 commented on PR #2146: URL: https://github.com/apache/james-project/pull/2146#issuecomment-2014666777 @Arsnael can we merge this? ^^ -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
Re: [PR] [Netty] Size estimator for ChunkedStream [james-project]
quantranhong1999 commented on PR #2145: URL: https://github.com/apache/james-project/pull/2145#issuecomment-2014640384 > With this we could potentially increase watermarks without risking things to blow up... Likely yes. I tested with a lower `WriteBufferWaterMark` => more back-pressure triggered, while a higher `WriteBufferWaterMark` => less back-pressure triggered. Selecting a big enough `WriteBufferWaterMark` likely would avoid being throttled too soon. -- 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: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org