jx2lee opened a new pull request, #1621: URL: https://github.com/apache/iceberg-go/pull/1621
## Summary - `properties get` always reported "on namespace" in its error message even when the target was a table, since the message was hardcoded instead of using the actual entity type. - Switched to `fmt.Errorf` with `get.Type` so the message reflects "namespace" or "table" correctly. ## Test plan Ran against a local hadoop catalog, requesting a non-existent property: **Before** ``` $ iceberg properties get table ns1.t1 nonexistent_prop could not find property nonexistent_prop on namespace ns1.t1 # wrong: says "namespace" for a table ``` **After** ``` $ iceberg properties get table ns1.t1 nonexistent_prop could not find property nonexistent_prop on table ns1.t1 # correct $ iceberg properties get namespace ns1 nonexistent_prop could not find property nonexistent_prop on namespace ns1 # unchanged, still correct -- 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]
