rdblue commented on a change in pull request #1870:
URL: https://github.com/apache/iceberg/pull/1870#discussion_r536325472
##########
File path: core/src/main/java/org/apache/iceberg/hadoop/HadoopCatalog.java
##########
@@ -56,16 +56,16 @@
import org.apache.iceberg.relocated.com.google.common.collect.Sets;
/**
- * HadoopCatalog provides a way to use table names like db.table to work with
path-based tables under a common
- * location. It uses a specified directory under a specified filesystem as the
warehouse directory, and organizes
- * multiple levels directories that mapped to the database, namespace and the
table respectively. The HadoopCatalog
- * takes a location as the warehouse directory. When creating a table such as
$db.$tbl, it creates $db/$tbl
- * directory under the warehouse directory, and put the table metadata into
that directory.
- *
- * The HadoopCatalog now supports {@link
org.apache.iceberg.catalog.Catalog#createTable},
- * {@link org.apache.iceberg.catalog.Catalog#dropTable}, the {@link
org.apache.iceberg.catalog.Catalog#renameTable}
- * is not supported yet.
- *
+ * HadoopCatalog provides a way to use table names like db.table to work with
path-based tables under a common location.
+ * It uses a specified directory under a specified filesystem as the warehouse
directory, and organizes multiple levels
+ * directories that mapped to the database, namespace and the table
respectively. The HadoopCatalog takes a location as
+ * the warehouse directory. When creating a table such as $db.$tbl, it creates
$db/$tbl directory under the warehouse
+ * directory, and put the table metadata into that directory.
+ * <p>
+ * The HadoopCatalog now supports {@link
org.apache.iceberg.catalog.Catalog#NoSuchNamespaceExceptioncreateTable}, {@link
+ * org.apache.iceberg.catalog.Catalog#dropTable}, the {@link
org.apache.iceberg.catalog.Catalog#renameTable} is not
+ * supported yet.
+ * <p>
Review comment:
These changes don't look related. Could you add them in a separate PR?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
