Re: RFR: 8341735: Rewrite the build/AbsPathsInImage.java test to not load the entire file at once [v2]

2025-07-08 Thread Daniel Hu
> These changes should prevent entire binary files from being loaded into > memory for build/AbsPathsInImage.java test. I chose a default buffer size of > 8KB since BufferedInputStream uses that, but open to alternative solutions. > GHA passes and test passes on linux x64. D

RFR: 8341735: Rewrite the build/AbsPathsInImage.java test to not load the entire file at once

2025-07-07 Thread Daniel Hu
These changes should prevent entire binary files from being loaded into memory for build/AbsPathsInImage.java test. I chose a default buffer size of 8KB since BufferedInputStream uses that, but open to alternative solutions. GHA passes and test passes on linux x64. - Commit message

Re: RFR: 8341735: Rewrite the build/AbsPathsInImage.java test to not load the entire file at once [v3]

2025-07-09 Thread Daniel Hu
> These changes should prevent entire binary files from being loaded into > memory for build/AbsPathsInImage.java test. I chose a default buffer size of > 8KB since BufferedInputStream uses that, but open to alternative solutions. > GHA passes and test passes on linux x64. D

Re: RFR: 8341735: Rewrite the build/AbsPathsInImage.java test to not load the entire file at once [v2]

2025-07-09 Thread Daniel Hu
On Tue, 8 Jul 2025 21:03:40 GMT, Mikael Vidstedt wrote: > Sorry for the potentially stupid question but: what happens if the pattern > spans a chunk? Thanks for catching that! I should've been more careful. Rewrote the byte matching completely to accommodate, but now test takes twice as long..

Re: RFR: 8341735: Rewrite the build/AbsPathsInImage.java test to not load the entire file at once [v4]

2025-07-09 Thread Daniel Hu
> These changes should prevent entire binary files from being loaded into > memory for build/AbsPathsInImage.java test. I chose a default buffer size of > 8KB since BufferedInputStream uses that, but open to alternative solutions. > GHA passes and test passes on linux x64. D