gaborkaszab commented on PR #14808: URL: https://github.com/apache/iceberg/pull/14808#issuecomment-3632865482
When we get the path in the REST catalog tests we use e.g. `TestRESTCatalog.RESOURCE_PATHS` that uses `%1F` as namespace separator. However, the test REST catalog use `%2E` as NS separator, so verification on paths being called fail if there is a multi-level namespace, or metadata table name involved. One workaround can be to create a new ResourcePaths instance with the particular NS-separator for each such test (one example is in testNotModified()) but I think it's more future-proof to configure the RESOURCE_PATHS member to use such separator. For instance I ran into this issue when rebasing my freshness-aware loading PR and then some of my path checks started failing. -- 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]
