This is an automated email from the ASF dual-hosted git repository.

fokko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iceberg.git


The following commit(s) were added to refs/heads/master by this push:
     new 949039303c Nessie: Update outdated javadoc (#6678)
949039303c is described below

commit 949039303c6a0f3665fc114a0cad3a861e0a76a4
Author: Ajantha Bhat <[email protected]>
AuthorDate: Mon Jan 30 19:41:12 2023 +0530

    Nessie: Update outdated javadoc (#6678)
    
    * Nessie: Update the incorrect javadoc
    
    Nessie catalog supports namespace properties. Test cases can be found 
[here](https://github.com/apache/iceberg/blob/038091f6b65bf63d028af175dbbbc7285815d6be/nessie/src/test/java/org/apache/iceberg/nessie/TestNamespace.java#L111-L146).
    
    But the  javadoc of `loadNamespaceMetadata` is incorrect (maybe not updated 
when Nessie supported namespace properties)
    
    * self review
    
    * address comments
---
 nessie/src/main/java/org/apache/iceberg/nessie/NessieCatalog.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/nessie/src/main/java/org/apache/iceberg/nessie/NessieCatalog.java 
b/nessie/src/main/java/org/apache/iceberg/nessie/NessieCatalog.java
index f0b2537711..37755f5123 100644
--- a/nessie/src/main/java/org/apache/iceberg/nessie/NessieCatalog.java
+++ b/nessie/src/main/java/org/apache/iceberg/nessie/NessieCatalog.java
@@ -271,11 +271,10 @@ public class NessieCatalog extends BaseMetastoreCatalog
   }
 
   /**
-   * Load the given namespace but return an empty map because namespace 
properties are currently not
-   * supported.
+   * Load the given namespace and return its properties.
    *
    * @param namespace a namespace. {@link Namespace}
-   * @return an empty map
+   * @return a string map of properties for the given namespace
    * @throws NoSuchNamespaceException If the namespace does not exist
    */
   @Override

Reply via email to