omarsmak commented on a change in pull request #3162:
URL: https://github.com/apache/iceberg/pull/3162#discussion_r713185595
##########
File path: site/docs/hive.md
##########
@@ -98,8 +98,8 @@ To globally register different catalogs, set the following
Hadoop configurations
| Config Key | Description
|
| --------------------------------------------- |
------------------------------------------------------ |
-| iceberg.catalog.<catalog_name\>.type | type of catalog: `hive` or
`hadoop` |
-| iceberg.catalog.<catalog_name\>.catalog-impl | catalog implementation, must
not be null if type is null |
+| iceberg.catalog.<catalog_name\>.type | type of catalog: `hive`,
`hadoop` or empty if `catalog-impl` will be set. |
Review comment:
I don't think it is necessary since this is the original intention to
leave the `type` empty in case the `catalog-impl ` will be set.
--
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]
