raunaqmorarka commented on issue #14490: URL: https://github.com/apache/iceberg/issues/14490#issuecomment-3483663446
@singhpk234 it is indeed possible to pass in metrics reporter as a constructor argument in `org.apache.iceberg.BaseTable#BaseTable(org.apache.iceberg.TableOperations, java.lang.String, org.apache.iceberg.metrics.MetricsReporter)`. I think we can use this by creating new BaseTable objects with our metrics reporter where needed, but the API is not as nice and it looks like this would replace any existing metrics reporter in BaseTable. E.g. In case of REST catalog, the MetricsReporter is already set by `org.apache.iceberg.rest.RESTSessionCatalog` and we would like to plug in our custom reporter in addition to that rather than replacing 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: [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]
