dossett opened a new pull request, #3926: URL: https://github.com/apache/parquet-java/pull/3926
### Rationale for this change With the Java 17 baseline, internal lists that are only read after construction can use `Stream.toList()`. This shortens stream expressions and makes their read-only use explicit. ### What changes are included in this PR? Replace seven `collect(Collectors.toList())` calls across `ReflectClassValidator`, `FilteringRecordMaterializer`, `ParquetFileReader`, and `ParquetRewriter`, and remove unused imports. The selected lists hold private validation prefixes, lookup-map keys, directory listings, and temporary logging, metadata, and row-count results. ### Are these changes tested? Validated in the Java 17 developer Docker image: - Nine existing Avro string-behavior and reflection read/write tests. - 37 existing record-level filtering, rewrite/join, and footer metadata tests. - Affected reactor builds and Spotless checks passed. ### Are there any user-facing changes? No. The changed lists are internal and are not modified after construction. Publicly returned lists and lists passed to Hadoop implementations retain their existing behavior. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
