caican00 commented on code in PR #5092:
URL: https://github.com/apache/gravitino/pull/5092#discussion_r1800568074


##########
docs/lakehouse-paimon-catalog.md:
##########
@@ -22,38 +22,40 @@ Builds with Apache Paimon `0.8.0`.
 
 ### Catalog capabilities
 
-- Works as a catalog proxy, supporting `FilesystemCatalog` and `JdbcCatalog`.
+- Works as a catalog proxy, supporting `FilesystemCatalog`, `JdbcCatalog` and 
`HiveCatalog`.
 - Supports DDL operations for Paimon schemas and tables.
 
-- Doesn't support `HiveCatalog` catalog backend now.
 - Doesn't support alterSchema.
 
 ### Catalog properties
 
-| Property name                                      | Description             
                                                                                
                                                                                
                    | Default value          | Required                         
                               | Since Version     |
-|----------------------------------------------------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|------------------------|-----------------------------------------------------------------|-------------------|
-| `catalog-backend`                                  | Catalog backend of 
Gravitino Paimon catalog. Supports `filesystem` and `jdbc` now.                 
                                                                                
                         | (none)                 | Yes                         
                                    | 0.6.0-incubating  |
-| `uri`                                              | The URI configuration 
of the Paimon catalog. `thrift://127.0.0.1:9083` or 
`jdbc:postgresql://127.0.0.1:5432/db_name` or 
`jdbc:mysql://127.0.0.1:3306/metastore_db`. It is optional for 
`FilesystemCatalog`. | (none)                 | required if the value of 
`catalog-backend` is not `filesystem`. | 0.6.0-incubating  |
-| `warehouse`                                        | Warehouse directory of 
catalog. `file:///user/hive/warehouse-paimon/` for local fs, 
`hdfs://namespace/hdfs/path` for HDFS , `s3://{bucket-name}/path/` for S3 or 
`oss://{bucket-name}/path` for Aliyun OSS  | (none)                 | Yes       
                                                      | 0.6.0-incubating  |
-| `authentication.type`                              | The type of 
authentication for Paimon catalog backend, currently Gravitino only supports 
`Kerberos` and `simple`.                                                        
                                   | `simple`               | No                
                                              | 0.6.0-incubating  |
-| `authentication.kerberos.principal`                | The principal of the 
Kerberos authentication.                                                        
                                                                                
                       | (none)                 | required if the value of 
`authentication.type` is Kerberos.     | 0.6.0-incubating  |
-| `authentication.kerberos.keytab-uri`               | The URI of The keytab 
for the Kerberos authentication.                                                
                                                                                
                      | (none)                 | required if the value of 
`authentication.type` is Kerberos.     | 0.6.0-incubating  |
-| `authentication.kerberos.check-interval-sec`       | The check interval of 
Kerberos credential for Paimon catalog.                                         
                                                                                
                      | 60                     | No                             
                                 | 0.6.0-incubating  |
-| `authentication.kerberos.keytab-fetch-timeout-sec` | The fetch timeout of 
retrieving Kerberos keytab from `authentication.kerberos.keytab-uri`.           
                                                                                
                       | 60                     | No                            
                                  | 0.6.0-incubating  |
-| `oss-endpoint`                                     | The endpoint of the 
Aliyun oss.                                                                     
                                                                                
                        | (none)                 | required if the value of 
`warehouse` is a oss path              | 0.7.0-incubating  |
-| `oss-access-key-id`                                | The access key of the 
Aliyun oss.                                                                     
                                                                                
                      | (none)                 | required if the value of 
`warehouse` is a oss path              | 0.7.0-incubating  |
-| `oss-accesss-key-secret`                           | The secret key the 
Aliyun s3.                                                                      
                                                                                
                         | (none)                 | required if the value of 
`warehouse` is a oss path              | 0.7.0-incubating  |
-| `s3-endpoint`                                      | The endpoint of the AWS 
s3.                                                                             
                                                                                
                    | (none)                 | required if the value of 
`warehouse` is a S3 path               | 0.7.0-incubating  |
-| `s3-access-key-id`                                 | The access key of the 
AWS s3.                                                                         
                                                                                
                      | (none)                 | required if the value of 
`warehouse` is a S3 path               | 0.7.0-incubating  |
-| `s3-secret-access-key`                             | The secret key of the 
AWS s3.                                                                         
                                                                                
                      | (none)                 | required if the value of 
`warehouse` is a S3 path               | 0.7.0-incubating  |
+| Property name                                      | Description             
                                                                                
                                                                                
                     | Default value          | Required                        
                                | Since Version     |
+|----------------------------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|------------------------|-----------------------------------------------------------------|-------------------|
+| `catalog-backend`                                  | Catalog backend of 
Gravitino Paimon catalog. Supports `filesystem`, `jdbc` and `hive`.             
                                                                                
                          | (none)                 | Yes                        
                                     | 0.6.0-incubating  |
+| `uri`                                              | The URI configuration 
of the Paimon catalog. `thrift://127.0.0.1:9083` or 
`jdbc:postgresql://127.0.0.1:5432/db_name` or 
`jdbc:mysql://127.0.0.1:3306/metastore_db`. It is optional for 
`FilesystemCatalog`.  | (none)                 | required if the value of 
`catalog-backend` is not `filesystem`. | 0.6.0-incubating  |
+| `warehouse`                                        | Warehouse directory of 
catalog. `file:///user/hive/warehouse-paimon/` for local fs, 
`hdfs://namespace/hdfs/path` for HDFS , `s3://{bucket-name}/path/` for S3 or 
`oss://{bucket-name}/path` for Aliyun OSS   | (none)                 | Yes      
                                                       | 0.6.0-incubating  |
+| `authentication.type`                              | The type of 
authentication for Paimon catalog backend, currently Gravitino only supports 
`Kerberos` and `simple`.                                                        
                                    | `simple`               | No               
                                               | 0.6.0-incubating  |
+| `authentication.kerberos.principal`                | The principal of the 
Kerberos authentication.                                                        
                                                                                
                        | (none)                 | required if the value of 
`authentication.type` is Kerberos.     | 0.6.0-incubating  |
+| `authentication.kerberos.keytab-uri`               | The URI of The keytab 
for the Kerberos authentication.                                                
                                                                                
                       | (none)                 | required if the value of 
`authentication.type` is Kerberos.     | 0.6.0-incubating  |
+| `authentication.kerberos.check-interval-sec`       | The check interval of 
Kerberos credential for Paimon catalog.                                         
                                                                                
                       | 60                     | No                            
                                  | 0.6.0-incubating  |
+| `authentication.kerberos.keytab-fetch-timeout-sec` | The fetch timeout of 
retrieving Kerberos keytab from `authentication.kerberos.keytab-uri`.           
                                                                                
                        | 60                     | No                           
                                   | 0.6.0-incubating  |
+| `oss-endpoint`                                     | The endpoint of the 
Aliyun oss.                                                                     
                                                                                
                         | (none)                 | required if the value of 
`warehouse` is a oss path              | 0.7.0-incubating  |
+| `oss-access-key-id`                                | The access key of the 
Aliyun oss.                                                                     
                                                                                
                       | (none)                 | required if the value of 
`warehouse` is a oss path              | 0.7.0-incubating  |
+| `oss-accesss-key-secret`                           | The secret key the 
Aliyun s3.                                                                      
                                                                                
                          | (none)                 | required if the value of 
`warehouse` is a oss path              | 0.7.0-incubating  |
+| `s3-endpoint`                                      | The endpoint of the AWS 
s3.                                                                             
                                                                                
                     | (none)                 | required if the value of 
`warehouse` is a S3 path               | 0.7.0-incubating  |
+| `s3-access-key-id`                                 | The access key of the 
AWS s3.                                                                         
                                                                                
                       | (none)                 | required if the value of 
`warehouse` is a S3 path               | 0.7.0-incubating  |
+| `s3-secret-access-key`                             | The secret key of the 
AWS s3.                                                                         
                                                                                
                       | (none)                 | required if the value of 
`warehouse` is a S3 path               | 0.7.0-incubating  |
 
 :::note
 If you want to use the `oss` or `s3` warehouse, you need to place related jars 
in the `catalogs/lakehouse-paimon/lib` directory, more information can be found 
in the [Paimon S3](https://paimon.apache.org/docs/master/filesystems/s3/). 
 :::
 
-
 Any properties not defined by Gravitino with `gravitino.bypass.` prefix will 
pass to Paimon catalog properties and HDFS configuration. For example, if 
specify `gravitino.bypass.table.type`, `table.type` will pass to Paimon catalog 
properties.
 
+:::caution

Review Comment:
   updated to `note`



-- 
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]

Reply via email to