ajantha-bhat commented on code in PR #4675: URL: https://github.com/apache/iceberg/pull/4675#discussion_r865522097
########## docs/api/java-api-quickstart.md: ########## @@ -66,6 +66,17 @@ Table table = catalog.createTable(name, schema, spec); The logs [schema](#create-a-schema) and [partition spec](#create-a-partition-spec) are created below. +HiveCatalog supports registering the table using an existing metadata JSON file as below: Review Comment: As you mentioned in the proposal, definitely having one doc per each catalog makes sense. Again, now keeping just this PR change in a new file `hive_catalog.md` looks odd to me. a) I would rather have this merged in quick-start now and refactor later when separate catalog page is ready. b) Or keep this PR open till separate catalog page is ready and move this change on top of it. -- 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]
