nastra commented on issue #13178: URL: https://github.com/apache/iceberg/issues/13178#issuecomment-2934269419
> DangerousParallelStreamUsage in BigQueryMetastoreClientImpl#491: [Link to code](https://github.com/apache/iceberg/blob/a4b2a0dab092821d4843749b8abc30208622e164/bigquery/src/main/java/org/apache/iceberg/gcp/bigquery/BigQueryMetastoreClientImpl.java#L491) Context: I noticed that .parallel() is mostly used in test files throughout the codebase. The ParallelIterable class seems not designed for this situation. Question: Should I simply remove the .parallel() call to align with the project patterns? @talatuyarer was the `.parallel()` on the Stream added on purpose in `BigQueryMetastoreClientImpl` or can we remove it? -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
