Re: RFR: 4890732: GZIPOutputStream doesn't support optional GZIP fields [v9]

2021-07-25 Thread Lin Zang
On Mon, 26 Jul 2021 03:28:44 GMT, Lin Zang wrote: >> 4890732: GZIPOutputStream doesn't support optional GZIP fields > > Lin Zang has updated the pull request with a new target base due to a merge > or a rebase. The pull request now contains 13 commits: > > - change since version to 18 > -

Re: RFR: 4890732: GZIPOutputStream doesn't support optional GZIP fields [v9]

2021-07-25 Thread Lin Zang
> 4890732: GZIPOutputStream doesn't support optional GZIP fields Lin Zang has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 13 commits: - change since version to 18 - Merge branch 'master' into gzip-field - Merge branch 'master'

RFR: 8251329: (zipfs) Files.walkFileTree walks infinitely if zip has dir named "." inside

2021-07-25 Thread Lance Andersen
Hi, As discussed in the https://mail.openjdk.java.net/pipermail/core-libs-dev/2021-July/079621.html thread, this is the revised patch to address the use of '.' and '..' within Zip FS Zip FS needs to use "." and ".." as links to the current and parent directories and cannot reliably support

Re: RFR: 8270321: Startup regressions in 18-b5 caused by JDK-8266310

2021-07-25 Thread Alan Bateman
On Sun, 25 Jul 2021 12:06:49 GMT, Sergey Chernyshev wrote: > I don't understand your analysis, you are testing the startup time with -Xint > which disable JITs, but there is no mention of -Xint in the bug report. > It's obvious to me that there is a regression with -Xint given that the >

Re: RFR: 8270321: Startup regressions in 18-b5 caused by JDK-8266310

2021-07-25 Thread Sergey Chernyshev
On Fri, 23 Jul 2021 18:03:31 GMT, Sergey Chernyshev wrote: > Dear colleagues, > > Please review the patch that replaces the lambdas with anonymous classes > which solves the startup time regression as shown below. > > I attached the Bytestacks flamegraphs for both original (regression) and