New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Nessie: Bump Nessie and add Namespace support #4385
Conversation
f8e92b1
to
970f2ca
Compare
Assertions.assertThatThrownBy(() -> catalog.dropNamespace(Namespace.of("test"))) | ||
.isInstanceOf(UnsupportedOperationException.class) | ||
.hasMessage("Cannot drop namespace 'test': dropNamespace is not supported by the NessieCatalog"); | ||
public void testCreatingAndDroppingNamespace() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
additional Namespace testing will be added once we introduce a NessieCatalogTest
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Namespace operations must use the Nessie reference
including the hash. It doesn't matter much for now, but once namespaces have properties, it probably matters.
nessie/src/main/java/org/apache/iceberg/nessie/NessieCatalog.java
Outdated
Show resolved
Hide resolved
nessie/src/main/java/org/apache/iceberg/nessie/NessieCatalog.java
Outdated
Show resolved
Hide resolved
970f2ca
to
fd52086
Compare
@Override | ||
public void createNamespace(Namespace namespace, Map<String, String> metadata) { | ||
try { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I notice metadata is ignored but the javadoc is removed, can you add it back.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a follow-up PR for that part of the code and will add that note there if that's ok?
No description provided.