wgtmac commented on code in PR #3726: URL: https://github.com/apache/parquet-java/pull/3726#discussion_r4216144982
########## parquet-hadoop/README.md: ########## @@ -558,6 +558,26 @@ if not found then the library will use the classic non-vectored reads: it is saf --- +**JVM system property:** `parquet.hadoop.vectored.io.threads` + +**Description:** Maximum number of shared vectored-read workers. Set a positive integer Review Comment: This executor is a JVM-global singleton read once at class initialization, so the limit is process-wide and cannot be changed per reader or per query. Would it make sense to also let the caller supply the executor, with this shared pool as the default? That way an engine could size and share it in the way it already manages its own pools. -- 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]
