jayyyyqwq opened a new pull request, #18719: URL: https://github.com/apache/iceberg/pull/18719
`DynamoDbCatalog.listNamespaces(ns)` queries `begins_with(namespace, "ns")`, so it returns `ns` itself, its descendants and any namespace that shares the name prefix (`db` also matches `dbx`), instead of the direct children that `SupportsNamespaces` documents. `listNamespaces()` returns every namespace at every depth, and a missing namespace returns an empty list instead of throwing. It now queries `ns.`, keeps the first level below `ns`, and throws `NoSuchNamespaceException` when nothing is found and the namespace does not exist. Unit tests cover children only, top level only, a leaf, a missing namespace, and children listed without a parent entry (that one passes before and after). `testListSubNamespaces` expected 4 for 3 children plus the parent, so I changed it to assert the 3 children. -- 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]
