Re: RFR: 8292698: Improve performance of DataInputStream [v4]

2022-10-16 Thread Jaikiran Pai
On Mon, 29 Aug 2022 08:46:20 GMT, Сергей Цыпанов wrote: >> I found out that reading from `DataInputStream` wrapping >> `ByteArrayInputStream` (as well as `BufferedInputStream` or any >> `InputStream` relying on `byte[]`) can be significantly improved by >> accessing volatile `in` field only on

Re: RFR: 8292698: Improve performance of DataInputStream [v4]

2022-10-16 Thread Jaikiran Pai
On Wed, 5 Oct 2022 15:24:10 GMT, Сергей Цыпанов wrote: > Anyone to sponsor, or should I get more approves? I see that both Alan and Daniel have approved these changes and there hasn't been a commit in this PR, after the approval (which is a good thing). I'll run some internal tests and if all

Re: RFR: 8292698: Improve performance of DataInputStream [v4]

2022-10-05 Thread Сергей Цыпанов
On Mon, 29 Aug 2022 08:46:20 GMT, Сергей Цыпанов wrote: >> I found out that reading from `DataInputStream` wrapping >> `ByteArrayInputStream` (as well as `BufferedInputStream` or any >> `InputStream` relying on `byte[]`) can be significantly improved by >> accessing volatile `in` field only on

Re: RFR: 8292698: Improve performance of DataInputStream [v4]

2022-09-16 Thread Сергей Цыпанов
On Mon, 29 Aug 2022 08:46:20 GMT, Сергей Цыпанов wrote: >> I found out that reading from `DataInputStream` wrapping >> `ByteArrayInputStream` (as well as `BufferedInputStream` or any >> `InputStream` relying on `byte[]`) can be significantly improved by >> accessing volatile `in` field only on

Re: RFR: 8292698: Improve performance of DataInputStream [v4]

2022-09-14 Thread Roger Riggs
On Mon, 29 Aug 2022 08:46:20 GMT, Сергей Цыпанов wrote: >> I found out that reading from `DataInputStream` wrapping >> `ByteArrayInputStream` (as well as `BufferedInputStream` or any >> `InputStream` relying on `byte[]`) can be significantly improved by >> accessing volatile `in` field only on

Re: RFR: 8292698: Improve performance of DataInputStream [v4]

2022-09-12 Thread Daniel Fuchs
On Mon, 29 Aug 2022 08:46:20 GMT, Сергей Цыпанов wrote: >> I found out that reading from `DataInputStream` wrapping >> `ByteArrayInputStream` (as well as `BufferedInputStream` or any >> `InputStream` relying on `byte[]`) can be significantly improved by >> accessing volatile `in` field only on

Re: RFR: 8292698: Improve performance of DataInputStream [v4]

2022-08-29 Thread Сергей Цыпанов
> I found out that reading from `DataInputStream` wrapping > `ByteArrayInputStream` (as well as `BufferedInputStream` or any `InputStream` > relying on `byte[]`) can be significantly improved by accessing volatile `in` > field only once per operation. > > Current implementation does it for each