sundapeng commented on PR #1014: URL: https://github.com/apache/paimon-rust/pull/1014#issuecomment-6417585714
@JingsongLi both findings from the 10-25 review are addressed and the threads answered: bbc4a693 routes only the files under a table's own location, so PK external descriptor reads go to the origin, and 04517aa7 gives every table and branch copy, including `copy_with_branch` to a catalog-only REST branch, its io-cache view with that branch's current prefixes. CI is green on 04517aa7. I also checked the Rust client against the class raised in the latest Java and Python reviews, a component routing or caching with options other than the effective ones: `RESTTokenFileIO` merges the catalog options with the token and applies the catalog's overrides once per instance, nothing caches an endpoint-bound client across catalogs, and the Python binding goes through the same merge, so no change was needed. Could you take another look when you have time? -- 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]
