moomindani commented on PR #16574: URL: https://github.com/apache/iceberg/pull/16574#issuecomment-4827924749
Not stale — this PR is actively maintained and waiting for reviewer feedback. Rebased onto latest `main` to resolve the merge conflicts in `RESTSessionCatalog` / `TestRESTCatalog` (they overlapped with the recent `RESTMetricsReporter` async-executor change — the table-name filter now wraps the executor-backed `RESTMetricsReporter`). Core compiles and the metrics-filter / REST-catalog tests pass. The design questions raised in the last review round have been addressed (REST-reporter symmetry fix + general catalog test). Happy to incorporate any further feedback whenever a reviewer has a chance to take a look. -- 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]
