Re: Re: Why common-compress skip data by reading and discard?

2024-05-24 Thread sebb
On Fri, 24 May 2024 at 14:51, iseki zero  wrote:
>
> Thank you. It will better if we can do a detection before we call it, to
> improve the performance.

Is there a performance issue here?
If so, is it enough to be worth the effort of updating the code?

Alternatively, it would be easy to update the Javadoc to advise users
who are on Java12+ to update their code to use the new methods...


> On 2024/05/23 11:21:33 Gary Gregory wrote:
>  > Hello iseki,
>  >
>  > The Javadoc explains all of this in detail.
>  >
>  > Java 12 adds skipNBytes and IO is still based on Java 8.
>  >
>  > Gary
>  >
>  > On Thu, May 23, 2024, 6:30 AM iseki zero  wrote:
>  >
>  > > Hello,
>  > >
>  > > When I reading the code of common-compress, I found it skip data using
>  > > the `IOUtils.skip` in `commons-io` module.
>  > >
>  > > I found this method just reading the data and discard it directly. But
>  > > in JDK there's an more effeciency method `InputStream.skip` and
>  > > `InputStream.skipNBytes`.
>  > >
>  > > So why we read & discard data instead of call `skip*` directly?
>  > >
>  > > Thank you.
>  > >
>  > > iseki zero.
>  > >
>  > >
>  > > -
>  > > To unsubscribe, e-mail: user-unsubscr...@commons.apache.org
>  > > For additional commands, e-mail: user-h...@commons.apache.org
>  > >
>  > >
>  >
>
> -
> To unsubscribe, e-mail: user-unsubscr...@commons.apache.org
> For additional commands, e-mail: user-h...@commons.apache.org
>

-
To unsubscribe, e-mail: user-unsubscr...@commons.apache.org
For additional commands, e-mail: user-h...@commons.apache.org



RE: Re: Why common-compress skip data by reading and discard?

2024-05-24 Thread iseki zero
Thank you. It will better if we can do a detection before we call it, to 
improve the performance.


On 2024/05/23 11:21:33 Gary Gregory wrote:
> Hello iseki,
>
> The Javadoc explains all of this in detail.
>
> Java 12 adds skipNBytes and IO is still based on Java 8.
>
> Gary
>
> On Thu, May 23, 2024, 6:30 AM iseki zero  wrote:
>
> > Hello,
> >
> > When I reading the code of common-compress, I found it skip data using
> > the `IOUtils.skip` in `commons-io` module.
> >
> > I found this method just reading the data and discard it directly. But
> > in JDK there's an more effeciency method `InputStream.skip` and
> > `InputStream.skipNBytes`.
> >
> > So why we read & discard data instead of call `skip*` directly?
> >
> > Thank you.
> >
> > iseki zero.
> >
> >
> > -
> > To unsubscribe, e-mail: user-unsubscr...@commons.apache.org
> > For additional commands, e-mail: user-h...@commons.apache.org
> >
> >
>

-
To unsubscribe, e-mail: user-unsubscr...@commons.apache.org
For additional commands, e-mail: user-h...@commons.apache.org