On Fri, 18 Apr 2025 13:05:24 GMT, Daniel Fuchs wrote:
> Hi,
>
> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the HTTP
> Client API](https://openjdk.org/jeps/517).
>
> The CSR can be viewed at [JDK-8350588: Implement JEP 517: HTTP/3 for the HTTP
&
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Thu, 11 Sep 2025 09:34:59 GMT, Raffaello Giulietti
wrote:
>> test/jdk/java/math/BigInteger/BigIntegerTest.java line 32:
>>
>>> 30: * @library /test/lib
>>> 31: * @build jdk.test.lib.RandomFactory
>>> 32: * @run main/timeout=480 BigIntegerTest
>>
>> Should the timeoutFactor change have ad
On Thu, 11 Sep 2025 08:55:02 GMT, Raffaello Giulietti
wrote:
> This PR
> * Moves sqrt() and nthRoot() tests to a new subset.
> * Removes a leftover `@run` tag without a timeout.
> * Increases the explicit timeout from 400 to 480.
> * Sorts the jtreg tags according to standard conventions.
LGTM
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
Hi,
Not immediately obvious but you can create a Stream> using
Stream.of and reduce that using Stream::concat to obtain a Stream.
Something along those lines:
```
var stream = Stream.of(Stream.of(1,2,3), Stream.of(4), Stream.of(5, 6,
7, 8)).reduce(Stream.empty(), Stream::concat, Stream::concat
On Sat, 13 Sep 2025 09:35:54 GMT, Jaikiran Pai wrote:
>> Can I please get a review of this change which addresses the issue noted in
>> https://bugs.openjdk.org/browse/JDK-8367597?
>>
>> As noted in that issue, on certain occasions, during shutdown of the JVM,
>> the logging in `Runtime.exit(
On Fri, 12 Sep 2025 06:36:31 GMT, Jaikiran Pai wrote:
>> Can I please get a review of this change which proposes to address the issue
>> noted in https://bugs.openjdk.org/browse/JDK-8357708?
>>
>> As noted in the issue, the current code in
>> `com.sun.jndi.ldap.Connection.readReply()` is susce
On Tue, 9 Sep 2025 15:31:40 GMT, David Beaumont wrote:
>> The writer field of the handler needs to be re-checked for null inside the
>> locked region before use.
>
> David Beaumont has updated the pull request incrementally with one additional
> commit since the last revision:
>
> License fi
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Tue, 9 Sep 2025 08:11:15 GMT, Jaikiran Pai wrote:
>> Can I please get a review of this change which proposes to address the issue
>> noted in https://bugs.openjdk.org/browse/JDK-8357708?
>>
>> As noted in the issue, the current code in
>> `com.sun.jndi.ldap.Connection.readReply()` is suscep
On Tue, 9 Sep 2025 19:19:22 GMT, David Beaumont wrote:
>> The writer field of the handler needs to be re-checked for null inside the
>> locked region before use.
>
> David Beaumont has updated the pull request with a new target base due to a
> merge or a rebase. The incremental webrev excludes
On Tue, 9 Sep 2025 15:25:05 GMT, David Beaumont wrote:
>> The writer field of the handler needs to be re-checked for null inside the
>> locked region before use.
>
> David Beaumont has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Adding tes
On Tue, 9 Sep 2025 14:25:00 GMT, David Beaumont wrote:
> The writer field of the handler needs to be re-checked for null inside the
> locked region before use.
LGTM. Please run tier 2 before integrating.
-
Marked as reviewed by dfuchs (Reviewer).
PR Review: https://git.openjdk.or
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Mon, 25 Aug 2025 15:58:08 GMT, Artur Barashev wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 628 commits:
>>
>> - merge latest changes from master branch
>> - http3:
On Wed, 20 Aug 2025 15:32:55 GMT, Artur Barashev wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 616 commits:
>>
>> - merge latest changes from master branch
>> - merge l
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Wed, 20 Aug 2025 16:21:42 GMT, Jaikiran Pai wrote:
>> src/java.base/share/classes/sun/security/ssl/Finished.java line 852:
>>
>>> 850: QuicTLSEngineImpl engine =
>>> 851: (QuicTLSEngineImpl)
>>> shc.conContext.transport;
>>> 852:
On Wed, 20 Aug 2025 13:55:38 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Wed, 23 Jul 2025 11:00:57 GMT, Daniel Jeliński wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 550 commits:
>>
>> - merge latest changes from master branch
>> - htt
On Thu, 24 Jul 2025 12:21:56 GMT, Daniel Jeliński wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 550 commits:
>>
>> - merge latest changes from master branch
>> - htt
On Thu, 24 Jul 2025 19:28:16 GMT, Daniel Jeliński wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 550 commits:
>>
>> - merge latest changes from master branch
>> - htt
On Thu, 24 Jul 2025 17:04:52 GMT, Daniel Jeliński wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 550 commits:
>>
>> - merge latest changes from master branch
>> - htt
On Thu, 24 Jul 2025 16:02:44 GMT, Daniel Jeliński wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 550 commits:
>>
>> - merge latest changes from master branch
>> - htt
On Thu, 24 Jul 2025 15:31:21 GMT, Daniel Jeliński wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 550 commits:
>>
>> - merge latest changes from master branch
>> - htt
On Thu, 24 Jul 2025 15:20:37 GMT, Daniel Jeliński wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 550 commits:
>>
>> - merge latest changes from master branch
>> - htt
On Mon, 18 Aug 2025 16:34:21 GMT, Leo Korinth wrote:
>> This changes the timeout factor from 4 to 1. Most of the changes add
>> timeouts to individual test cases so that I am able to run them with a
>> timeout factor of 0.7 (some margin to the checked in factor of one)
>>
>> In addition to cha
On Fri, 15 Aug 2025 11:43:33 GMT, Leo Korinth wrote:
>> This changes the timeout factor from 4 to 1. Most of the changes add
>> timeouts to individual test cases so that I am able to run them with a
>> timeout factor of 0.7 (some margin to the checked in factor of one)
>>
>> In addition to cha
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Tue, 1 Jul 2025 15:20:07 GMT, Jaikiran Pai wrote:
>> The uniqueness comes not just from the timestamp but also from the random
>> data in the other bytes that are generated for each new UUID.
>
> Hello Roger, that's true about the uniqueness semantics. However, from what I
> understand of RF
On Tue, 1 Jul 2025 11:48:30 GMT, Jaikiran Pai wrote:
> Can I please get a review of this backport of a test-only fix?
>
> This backports the fix for https://bugs.openjdk.org/browse/JDK-8359337 into
> JDK 25. The original fix was integrated into mainline a few minutes back
> https://github.com
On Mon, 30 Jun 2025 09:34:48 GMT, Daniel Jeliński wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 525 commits:
>>
>> - merge latest changes from master branch
>> - http3: run
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Fri, 27 Jun 2025 11:37:00 GMT, Daniel Jeliński wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 525 commits:
>>
>> - merge latest changes from master branch
>> - http3: run
On Fri, 27 Jun 2025 11:22:51 GMT, Daniel Jeliński wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 525 commits:
>>
>> - merge latest changes from master branch
>> - http3: run
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Thu, 26 Jun 2025 17:36:21 GMT, Daniel Jeliński wrote:
>> src/java.base/share/classes/jdk/internal/net/quic/QuicTLSContext.java line
>> 70:
>>
>>> 68: if (!(underlyingImpl instanceof SSLContextImpl ssci)) {
>>> 69: return false;
>>> 70: }
>>
>> Would there be a wa
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Thu, 26 Jun 2025 16:36:40 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Thu, 26 Jun 2025 11:04:33 GMT, Daniel Fuchs wrote:
>> Thank you Artur.
>>
>> @dfuch this conversation can be resolved too.
>
> Done
> Hi Jaikiran! Sounds good. It's likely we are going to re-work this code
> anyhow when we make QUIC engine pub
On Thu, 26 Jun 2025 13:22:28 GMT, Jaikiran Pai wrote:
>> src/java.base/share/classes/sun/security/ssl/X509Authentication.java line
>> 226:
>>
>>> 224: chc.peerSupportedAuthorities == null ? null :
>>> 225: chc.peerSupportedAuthorities.clone(),
>>>
On Thu, 26 Jun 2025 01:04:41 GMT, Jaikiran Pai wrote:
>> Hi Jaikiran! Sounds good. It's likely we are going to re-work this code
>> anyhow when we make QUIC engine public in the next iteration. We had a
>> discussion with Daniel about it today.
>
> Thank you Artur.
>
> @dfuch this conversation
On Wed, 4 Jun 2025 17:43:16 GMT, Daniel Fuchs wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 506 commits:
>>
>> - merge latest changes from master branch
>> - http3: update
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Thu, 15 May 2025 23:59:41 GMT, Brent Christian wrote:
>> Please review this change to replace the finalizer in
>> `AbstractLdapNamingEnumeration` with Cleaner.
>>
>> (The [first PR](https://github.com/openjdk/jdk/pull/8311) for this fix
>> started some substantial discussions, leading to, a
On Fri, 6 Jun 2025 18:35:53 GMT, Phil Race wrote:
> The fix for JDK-8344235: Revisit SecurityManager usage in java.logging after
> JEP 486 and JEP 491 integration
> removed use of AppContext from java/util/logging/LogManager.java.
> That was the only place in the JDK that used
> jdk.internal.ac
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Wed, 4 Jun 2025 15:46:36 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Fri, 16 May 2025 10:26:11 GMT, Daniel Fuchs wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 422 commits:
>>
>> - merge latest changes from master branch
>> - Undo
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Wed, 4 Jun 2025 15:03:11 GMT, Jaikiran Pai wrote:
>> Can I please get a review of this trivial doc-only change to the
>> `jdk.zipfs`'s documentation? This moves the `accessMode` property listing to
>> the top of the table instead of being at the bottom. With this change, the
>> text about t
On Wed, 4 Jun 2025 06:47:59 GMT, Jaikiran Pai wrote:
> Can I please get a review of this trivial doc-only change to the
> `jdk.zipfs`'s documentation? This moves the `accessMode` property listing to
> the top of the table instead of being at the bottom. With this change, the
> text about throw
On Tue, 3 Jun 2025 14:23:54 GMT, Michael McMahon wrote:
>> Hi,
>>
>> Enhanced exception messages are designed to hide sensitive information such
>> as hostnames, IP
>> addresses from exception message strings, unless the enhanced mode for the
>> specific category
>> has been explicitly enabl
x27; into 8348986-exceptions
> - doc update to java.security
> - removed jmod/Handler change
> - doc update to java.security
> - Fixed problem with j.n.HostPortRange
> - typo in suggestions and other issues
> - Merge branch 'master' into 8348986-exceptions
> -
On Thu, 29 May 2025 14:35:11 GMT, Michael McMahon wrote:
>> Hi,
>>
>> Enhanced exception messages are designed to hide sensitive information such
>> as hostnames, IP
>> addresses from exception message strings, unless the enhanced mode for the
>> specific category
>> has been explicitly enab
On Mon, 26 May 2025 08:16:03 GMT, Matthias Baesken wrote:
>> On a 'german' Ubuntu 24 (LANG="de_DE.UTF-8") the jtreg test
>> java/lang/System/LoggerFinder/internal/BootstrapLogger/BootstrapLoggerTest
>> fails with
>>
>>
>> FEIN: hi now!
>> java.lang.RuntimeException: System.err does not contai
On Mon, 19 May 2025 12:24:28 GMT, Jaikiran Pai wrote:
>> Hello David, I had another look at this code and after going through it, it
>> looked like `readOnly` field can in fact be made `final` because of the
>> refactoring changes that you did in this PR. I checked out your latest PR
>> locall
On Mon, 26 May 2025 16:16:44 GMT, Jaikiran Pai wrote:
> Additionally, the addRequest(), close() and cancel() methods of this
> LdapRequet get called by a single thread managed by the Connection class, so
> there isn't expected to be concurrent calls across these methods.
I am not seeing that
On Mon, 26 May 2025 12:28:16 GMT, Jaikiran Pai wrote:
> Can I please get a review of this change which proposes to address the issue
> noted in https://bugs.openjdk.org/browse/JDK-8357708?
>
> As noted in the issue, the current code in
> `com.sun.jndi.ldap.Connection.readReply()` is susceptibl
On Mon, 26 May 2025 10:31:39 GMT, Michael McMahon wrote:
>> Hi,
>>
>> Enhanced exception messages are designed to hide sensitive information such
>> as hostnames, IP
>> addresses from exception message strings, unless the enhanced mode for the
>> specific category
>> has been explicitly enab
On Mon, 26 May 2025 08:16:03 GMT, Matthias Baesken wrote:
>> On a 'german' Ubuntu 24 (LANG="de_DE.UTF-8") the jtreg test
>> java/lang/System/LoggerFinder/internal/BootstrapLogger/BootstrapLoggerTest
>> fails with
>>
>>
>> FEIN: hi now!
>> java.lang.RuntimeException: System.err does not contai
On Sat, 17 May 2025 19:42:39 GMT, Nizar Benalla wrote:
> Please review this patch to fix some `javadoc` bugs in `java.base`.
> Certain `@link` tags used to refer to private fields instead of public APIs.
>
> A couple of `@see` tags in the [serialization
> page](https://download.java.net/java/ea
On Fri, 16 May 2025 11:42:08 GMT, Michael McMahon wrote:
>> Hi,
>>
>> Enhanced exception messages are designed to hide sensitive information such
>> as hostnames, IP
>> addresses from exception message strings, unless the enhanced mode for the
>> specific category
>> has been explicitly enab
On Fri, 9 May 2025 14:39:53 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of [JEP 517: HTTP/3 for the
>> HTTP Client API](https://openjdk.org/jeps/517).
>>
>> The CSR can be viewed at [JDK-8350588: Implement JEP 517
On Fri, 9 May 2025 14:14:57 GMT, Magnus Ihse Bursie wrote:
> A handful of html and xml files in the JDK source tree claims to have
> encodings like `ISO-8859-1`, when they are in fact pure US-ASCII files.
>
> While perhaps technically correct, this is misleading, and goes contrary to
> the eff
350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull request with a new
On Fri, 9 May 2025 08:40:44 GMT, Leo Korinth wrote:
> The whole idea of running with a timeout factor of `0.7` is to remove
> intermittent failures. (I had it close to 0.5 or maybe less to begin with
> until I found and reported CODETOOLS-7903937: JTREG uses timeout factor on
> socket timeout
On Thu, 8 May 2025 17:41:02 GMT, Daniel Fuchs wrote:
> Thank you. I have imported your PR locally and running some HTTP client tests
> in the CI.
> Tests have not finished running - but I already see one intermittent failure:
> `java/net/httpclient/RedirectTimeoutTest.java` i
On Thu, 8 May 2025 14:51:24 GMT, Leo Korinth wrote:
> This change tries to add timeout to individual testcases so that I am able to
> run them with a timeout factor of 1 in the future (JDK-8260555).
>
> The first commit changes the timeout factor to 0.7, so that I can run tests
> and test the
On Thu, 8 May 2025 14:51:24 GMT, Leo Korinth wrote:
> This change tries to add timeout to individual testcases so that I am able to
> run them with a timeout factor of 1 in the future (JDK-8260555).
>
> The first commit changes the timeout factor to 0.7, so that I can run tests
> and test the
On Tue, 6 May 2025 14:40:46 GMT, Per Minborg wrote:
>> This sketch shows how "Stable Updaters" can be used to create stable
>> computations of `@Stable` fields. Only one updater is needed per class,
>> similar to `AtomicIntegerFieldUpdater`.
>
> Per Minborg has updated the pull request incremen
On Thu, 1 May 2025 17:26:48 GMT, Joe Darcy wrote:
>> Note, the timeout factor also adjusts the jtreg timeout for the entire test.
>> The adjustTimeout() method allows internal test timeouts to scale along with
>> the jtreg timeout.
>
>> Hi Joe, yes `adjustTimeout` will scale based on the jtreg
rg/browse/JDK-8350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull reques
On Wed, 30 Apr 2025 10:19:54 GMT, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of JEP [JDK-8291976: HTTP/3 for
>> the HTTP Client API](https://bugs.openjdk.org/browse/JDK-8291976).
>>
>> The CSR can be viewed at [JDK-83
rg/browse/JDK-8350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull reques
On Sat, 26 Apr 2025 17:07:33 GMT, Markus KARG wrote:
>> This Pull Request proposes an implementation for
>> [JDK-8343110](https://bugs.openjdk.org/browse/JDK-8343110): Adding the new
>> method `public void getChars(int srcBegin, int srcEnd, char[] dst, int
>> dstBegin)` to the `CharSequence` i
rg/browse/JDK-8350588)
>
> This JEP proposes to enhance the HttpClient implementation to support HTTP/3.
> It adds a non-exposed / non-exported internal implementation of the QUIC
> protocol based on DatagramChannel and the SunJSSE SSLContext provider.
Daniel Fuchs has updated the pull reques
On Wed, 16 Apr 2025 17:37:07 GMT, David Beaumont wrote:
>> Increasing timeout for deadlock detection and adjusting for the timeout
>> factor in higher tiers.
>
> David Beaumont has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Removing test
On Tue, 22 Apr 2025 18:48:06 GMT, Chen Liang wrote:
>> PushId is an HTTP/3 concept. It would be strange to have a generic method
>> (sendAsync) that you could call with any requests, but with a parameter that
>> could only be used for HTTP/3. We don't have pushIds for HTTP/2. I don't
>> think
On Fri, 18 Apr 2025 18:49:19 GMT, Chen Liang wrote:
>> Hi,
>>
>> Please find here a PR for the implementation of JEP [JDK-8291976: HTTP/3 for
>> the HTTP Client API](https://bugs.openjdk.org/browse/JDK-8291976).
>>
>> The CSR can be viewed at [JDK-8350588: Implement HTTP/3 for the HTTP Client
On Mon, 21 Apr 2025 19:05:54 GMT, Joe Darcy wrote:
>> David Beaumont has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Removing test from the problem list.
>
> test/jdk/java/util/logging/LoggingDeadlock5.java line 127:
>
>> 125: /
Hi,
Please find here a PR for the implementation of JEP [JDK-8291976: HTTP/3 for
the HTTP Client API](https://bugs.openjdk.org/browse/JDK-8291976).
The CSR can be viewed at [JDK-8350588: Implement HTTP/3 for the HTTP Client
API](https://bugs.openjdk.org/browse/JDK-8350588)
This JEP proposes to
On Wed, 16 Apr 2025 17:37:07 GMT, David Beaumont wrote:
>> Increasing timeout for deadlock detection and adjusting for the timeout
>> factor in higher tiers.
>
> David Beaumont has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Removing test
On Wed, 16 Apr 2025 12:12:14 GMT, David Beaumont wrote:
> Increasing timeout for deadlock detection and adjusting for the timeout
> factor in higher tiers.
Hard to tell whether 1s would be enough. I'd be tempted to bump that up.
You should also take the test out of the problem list.
--
On Wed, 16 Apr 2025 11:20:36 GMT, Jaikiran Pai wrote:
> This brings in the CPU25_04 changes into the master branch.
LGTM
-
Marked as reviewed by dfuchs (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/24683#pullrequestreview-2772164693
On Tue, 15 Apr 2025 14:35:28 GMT, Michael McMahon wrote:
>> Hi,
>>
>> Enhanced exception messages are designed to hide sensitive information such
>> as hostnames, IP
>> addresses from exception message strings, unless the enhanced mode for the
>> specific category
>> has been explicitly enab
On Mon, 14 Apr 2025 10:56:45 GMT, David Beaumont wrote:
> Add null pointer guard in test formatter (since it can be called with a log
> record without parameters).
Trivial fix. Looks like the right thing to do.
-
Marked as reviewed by dfuchs (Reviewer).
PR Review: https://git.ope
1 - 100 of 524 matches
Mail list logo
