On Mon, 6 Sep 2021 07:20:11 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 17 commits:
>
> - Merge bran
> 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 21 commits:
- remove trailing whitespaces
- fix build warnings
- Merge branch 'master' into gzip-fi
On Mon, 6 Sep 2021 07:20:11 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 17 commits:
>
> - Merge bran
> 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 17 commits:
- Merge branch 'master' into gzip-field
- delete trailing spaces
- refine code
- Merge bra
> 4890732: GZIPOutputStream doesn't support optional GZIP fields
Lin Zang has updated the pull request incrementally with one additional commit
since the last revision:
delete trailing spaces
-
Changes:
- all: https://git.openjdk.java.net/jdk/pull/3072/files
- new: ht
> 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 15 commits:
- refine code
- Merge branch 'master' into gzip-field
- change since version to 18
- Me
On Mon, 26 Jul 2021 20:47:59 GMT, Lance Andersen wrote:
>> 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
On Mon, 26 Jul 2021 20:55:00 GMT, Lance Andersen wrote:
>> 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
On Tue, 27 Jul 2021 15:52:23 GMT, Alan Bateman wrote:
>> Thank you for reviving the discussion.
>>
>> I have not gone through the latest update in detail but there are some
>> changes that are needed. Before moving forward with the CSR, I would like to
>> give time for additional feedback on
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:
>
> - chang
> 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 'mas
> 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 11 commits:
- Merge branch 'master' into gzip-field
- Add api in GZIPInputStream to get header data
- Me
> 4890732: GZIPOutputStream doesn't support optional GZIP fields
Lin Zang has updated the pull request incrementally with one additional commit
since the last revision:
Add api in GZIPInputStream to get header data
-
Changes:
- all: https://git.openjdk.java.net/jdk/pull/3
On Wed, 24 Mar 2021 10:25:44 GMT, Lance Andersen wrote:
>> Lin Zang has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - update copyright
>> - reuse arguments constructor for non-argument one.
>
> Hi L
On Thu, 15 Apr 2021 12:41:17 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 incremental webrev excludes the unrelated changes brought in
On Thu, 8 Apr 2021 08:54:06 GMT, Alan Bateman wrote:
>> Dear All,
>> May I ask your help to review this change? Thanks!
>>
>> BRs,
>> Lin
>
>> Dear All,
>> May I ask your help to review this change? Thanks!
>
> @LanceAndersen Do you have cycles to help Lin? This proposal will require
>
> 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 incremental webrev excludes the unrelated changes brought in by
the merge/rebase. The pull request contains ten additional commits si
On Wed, 24 Mar 2021 10:25:44 GMT, Lance Andersen wrote:
>> Lin Zang has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - update copyright
>> - reuse arguments constructor for non-argument one.
>
> Hi L
> 4890732: GZIPOutputStream doesn't support optional GZIP fields
Lin Zang has updated the pull request incrementally with one additional commit
since the last revision:
Use record and Builder pattern
-
Changes:
- all: https://git.openjdk.java.net/jdk/pull/3072/files
-
On Thu, 8 Apr 2021 08:54:06 GMT, Alan Bateman wrote:
>> Dear All,
>> May I ask your help to review this change? Thanks!
>>
>> BRs,
>> Lin
>
>> Dear All,
>> May I ask your help to review this change? Thanks!
>
> @LanceAndersen Do you have cycles to help Lin? This proposal will require
>
On Fri, 2 Apr 2021 08:53:20 GMT, Lin Zang wrote:
>> Hi Lin,
>>
>> On Mar 24, 2021, at 2:51 AM, Lin Zang ***@***.**@***.***>> wrote:
>>
>>
>>
>> Hi Lance,
>> Thanks a lot for your review. I will update the PR ASAP.
>> May I as
On Wed, 24 Mar 2021 10:25:44 GMT, Lance Andersen wrote:
>> Hi Lance,
>> Thanks a lot for your review. I will update the PR ASAP.
>> May I ask your help to also review the CSR? Thanks!
>>
>> BRs,
>> Lin
>
> Hi Lin,
>
> On Mar 24, 2021, at
> 4890732: GZIPOutputStream doesn't support optional GZIP fields
Lin Zang has updated the pull request incrementally with one additional commit
since the last revision:
add class GZIPHeaderData, refine testcases
-
Changes:
- all: https://git.openjdk.java.net/jdk/pull/3
On Mon, 22 Mar 2021 20:18:37 GMT, Lance Andersen wrote:
>> Lin Zang has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - update copyright
>> - reuse arguments constructor for non-argument one.
>
> src/j
On Mon, 22 Mar 2021 20:15:38 GMT, Lance Andersen wrote:
> on operating systems using
> EBCDIC or any other character set for file names, the name
> must be translated to the ISO LATIN-1 character set. This
> is the original name of the file being compressed, with any
>
On Mon, 22 Mar 2021 20:06:33 GMT, Lance Andersen wrote:
>> Lin Zang has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - update copyright
>> - reuse arguments constructor for non-argument one.
>
> src/j
On Wed, 24 Mar 2021 10:25:44 GMT, Lance Andersen wrote:
>> Hi Lance,
>> Thanks a lot for your review. I will update the PR ASAP.
>> May I ask your help to also review the CSR? Thanks!
>>
>> BRs,
>> Lin
>
> Hi Lin,
>
> On Mar 24, 2021, at
On Mon, 22 Mar 2021 21:37:27 GMT, Lance Andersen wrote:
>> Lin Zang has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - update copyright
>> - reuse arguments constructor for non-argument one.
>
> Hi Li
> 4890732: GZIPOutputStream doesn't support, in fact thwarts, use of optional
> GZIP fields
Lin Zang has updated the pull request incrementally with two additional commits
since the last revision:
- update copyright
- reuse arguments constructor for non-argument one.
-
C
> 4890732: GZIPOutputStream doesn't support, in fact thwarts, use of optional
> GZIP fields
Lin Zang has updated the pull request incrementally with one additional commit
since the last revision:
add test case
-
Changes:
- all: https://git.openjdk.java.net/jdk/pull/3072
On Thu, 18 Mar 2021 10:50:05 GMT, Lin Zang wrote:
>> 4890732: GZIPOutputStream doesn't support, in fact thwarts, use of optional
>> GZIP fields
>
> Dear All,
> This PR introduce new constructor of GZIPOutputStream to support adding extra
> field info in gzip file he
On Thu, 18 Mar 2021 10:46:04 GMT, Lin Zang wrote:
> 4890732: GZIPOutputStream doesn't support, in fact thwarts, use of optional
> GZIP fields
Dear All,
This PR introduce new constructor of GZIPOutputStream to support adding extra
field info in gzip file header, as decribed in RFC 195
4890732: GZIPOutputStream doesn't support, in fact thwarts, use of optional
GZIP fields
-
Commit messages:
- remove trailing spaces
- 4890732: support optional GZIP fields in GZIPOutputStream
Changes: https://git.openjdk.java.net/jdk/pull/3072/files
Webrev:
On Thu, 11 Feb 2021 02:36:01 GMT, Lin Zang wrote:
> 8261482: Adjust jmap histo command to accept noparallel option to inspect
> heap serially
Dear All,
Stories and discussions related with this PR could be found at #2379 and
#2261. Just FYI.
BRs,
Lin
-
PR:
8261482: Adjust jmap -histo to accept noparallel option to inspect heap serially
-
Commit messages:
- 8261482: Adjust jmap -histo to accept noparallel option to inspect heap
serially
Changes: https://git.openjdk.java.net/jdk/pull/2519/files
Webrev:
On Fri, 29 Jan 2021 05:33:19 GMT, Serguei Spitsyn wrote:
> You are right.
> So, in my suggestion just replace this block:
>
> ```
> if (gzlevel <= 0 || gzlevel > 9) {
> err.println("Invalid "gz=" option: " + option);
>
8252842: Extend jmap to support parallel heap dump
-
Commit messages:
- fix white space trailing issue
- 8252842: Extend jmap to support parallel heap dump
Changes: https://git.openjdk.java.net/jdk/pull/2261/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk=2261=00
The implementation of jmap tool depends on the implementation of object
iteration by different GC heap.
This patch extend the BasicJMapTest to cover differet GC Heap.
-
Commit messages:
- 8255982: Extend BasicJMapTest to test with different GC Heap
Changes:
38 matches
Mail list logo