RFR: 8278268 - (ch) InputStream returned by Channels.newInputStream should have fast path for FileChannel targets [v19]
Brian Burkhalter
bpb at openjdk.org
Fri Mar 31 21:23:33 UTC 2023
On Fri, 17 Mar 2023 16:58:00 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:
>> Markus KARG has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
>>
>> - updated copyright
>> - Implementing 8278268
>>
>> Signed-off-by: Markus Karg <markus at headcrashing.eu>
>
>> So can you please briefly **confirm** that I _shall not_ merge `TransferTo2` into `TransferTo` as _Lance_ proposed (hence `TransferTo2` will not be a _separate_ class anymore) but **instead** I _only shall_ abstract `testNullPointerException` and `testStreamContents` as _you_ proposed (hence keep `TransferTo2` as a _separate_ class)? Thanks.
>
> That is fine for now. It will be possible to revisit merging at a later time if it appears reasonable.
>
>> > ...I think the two tests `*_2GB*` should remain separate as they likely each have a sufficiently long run time.
>>
>> I confirm that we had intentionally kept these two tests in separate classes due to the long run time of each, so I agree to keep them _as separate_ classes, but I will reduce _duplicate_ code.
>
> Sounds good.
>
>> So can you please briefly **confirm** that I am correct with this objection and the 2GB files _should not_ be renamed?
>
> That is fine.
> @bplb Your requested changes are done since one week. Kindly asking to proceed with this PR. Thanks. :-)
Indeed I was intending to approve this afternoon. I read through it all and the tests look much better now.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/6711#issuecomment-1492620222
More information about the nio-dev
mailing list