alexandrefimov commented on PR #16664: URL: https://github.com/apache/iceberg/pull/16664#issuecomment-4691071862
The concrete use case I know of is the one from #14875 / #14654: users reading through IcebergGenerics directly want to attach an InMemoryMetricsReporter or custom MetricsReporter to a specific generic scan, then consume the emitted ScanReport for telemetry/logging after the read. ScanBuilder owns the underlying TableScan today, so there is no way to configure that per scan without this delegating method. I do not know of an in-repo production call site that currently needs it. The usages I found in this repository are mostly data tests and Flink helper/test paths. The motivation here is for standalone/generic read users rather than a Spark/Flink integration path, and the PR keeps the surface limited to the one existing TableScan hook needed for that use case. -- 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]
