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