yuqi1129 commented on code in PR #11354:
URL: https://github.com/apache/gravitino/pull/11354#discussion_r3395447356


##########
docs/apache-hive-catalog.md:
##########
@@ -29,19 +29,19 @@ The Hive catalog supports creating, updating, and deleting 
databases and tables
 
 Besides the [common catalog 
properties](./gravitino-server-config.md#catalog-properties-configuration), the 
Hive catalog has the following properties:
 
-| Property Name                            | Description                       
                                                                                
                                                                                
                                                  | Default Value | Required    
                 | Since Version |
-|------------------------------------------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|---------------|------------------------------|---------------|
-| `metastore.uris`                         | The Hive metastore service URIs, 
separate multiple addresses with commas. Such as `thrift://127.0.0.1:9083`      
                                                                                
                                                   | (none)        | Yes        
                  | 0.2.0         |
-| `client.pool-size`                       | The maximum number of Hive 
metastore clients in the pool for Gravitino.                                    
                                                                                
                                                         | 1             | No   
                        | 0.2.0         |
-| `gravitino.bypass.`                      | Property name with this prefix 
passed down to the underlying HMS client for use. Such as 
`gravitino.bypass.hive.metastore.failure.retries = 3` indicate 3 times of 
retries upon failure of Thrift metastore calls                                  
 | (none)        | No                           | 0.2.0         |
-| `client.pool-cache.eviction-interval-ms` | The cache pool eviction interval. 
                                                                                
                                                                                
                                                  | 300000        | No          
                 | 0.4.0         |
-| `impersonation-enable`                   | Enable user impersonation for 
Hive catalog.                                                                   
                                                                                
                                                      | false         | No      
                     | 0.4.0         |
-| `kerberos.principal`                     | The Kerberos principal for the 
catalog. You should configure 
`gravitino.bypass.hadoop.security.authentication`, 
`gravitino.bypass.hive.metastore.kerberos.principal` and 
`gravitino.bypass.hive.metastore.sasl.enabled`if you want to use Kerberos. | 
(none)        | required if you use kerberos | 0.4.0         |
-| `kerberos.keytab-uri`                    | The uri of key tab for the 
catalog. Now supported protocols are `https`, `http`, `ftp`, `file`.            
                                                                                
                                                         | (none)        | 
required if you use kerberos | 0.4.0         |
-| `kerberos.check-interval-sec`            | The interval to check validness 
of the principal                                                                
                                                                                
                                                    | 60            | No        
                   | 0.4.0         |
-| `kerberos.keytab-fetch-timeout-sec`      | The timeout to fetch key tab      
                                                                                
                                                                                
                                                  | 60            | No          
                 | 0.4.0         |
-| `list-all-tables`                        | Whether to list all tables in a 
database, including non-Hive tables such as Iceberg, Paimon, and Hudi. When 
false, non-Hive tables are filtered out on a best-effort basis; see the note 
below for known limitations.                            | false         | No    
                       | 0.5.1         |
-| `default.catalog`                        | The default catalog name for the 
Hive3 metastore backend; this configuration is ignored when using a Hive2 
metastore.                                                                      
                                                         | hive          | No   
                        | 1.1.0         |
+| Property Name                            | Description                       
                                                                                
                                                                                
                                                  | Default Value  | Required   
                  | Since Version |

Review Comment:
   I removed some empty lines in the table by the way.



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