tanmayrauth opened a new pull request, #1430:
URL: https://github.com/apache/iceberg-go/pull/1430
Third PR of the metrics reporting framework. Adds reporter selection and
threads it from catalog to scan:
- metrics: a name->factory registry and FromProperties, selecting a reporter
via the metrics-reporter-impl property (the Go analogue of Java's
reflection-based loader); built-in nop and logging reporters.
- table: a WithMetricsReporter option on New/NewFromLocation (variadic,
backward compatible), a MetricsReporter accessor, and a WithReporter scan
option; scans inherit the table's reporter.
- catalog/{rest,sql,glue,hive,hadoop} resolve the reporter from properties
when loading or creating a table.
Reporting stays opt-in: the default reporter is no-op, so there is no
behavior change for existing users. Report emission is added with the scan and
commit instrumentation in later PRs.
Related: #1236
--
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]
