coderfender commented on PR #12901:
URL: https://github.com/apache/iceberg/pull/12901#issuecomment-2869184250
@nastra , It seems like `TestNessieIcebergClient` tests are either broken
(or I am missing an something here). Each of the tests is returning an empty
collection in this line where technically a namespace is created right above
it (ex lines 125-127 in TestNessieIcebergClient). Now ,I could potentially just
fix the unit tests but I believe that wouldnt solve the root cause of Nessie
catalog not being able to find a namespace right after it is created. Do you
think the potential fix for that should be a part of this PR or did you want me
to just to stick raising exceptions whenever a namespace does not exist and
handle the latter in a different PR ?
`client.createNamespace(Namespace.of("a"), Map.of());
assertThat(client.listNamespaces(Namespace.of("a"))).isNotNull(); `
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
