jerqi commented on code in PR #9779:
URL: https://github.com/apache/gravitino/pull/9779#discussion_r2745097175


##########
docs/manage-statistics-in-gravitino.md:
##########
@@ -234,11 +234,73 @@ table.dropPartitionStatistics(statisticsToDrop);
 
 ### Server configuration
 
-| Configuration item                              | Description                
                                                                                
                                                                                
                                          | Default value                       
                                       | Required  | Since version |
-|-------------------------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|----------------------------------------------------------------------------|-----------|---------------|
-| `gravitino.stats.partition.storageFactoryClass` | The storage factory class 
for partition statistics, which is used to store partition statistics in the 
different storage. The 
`org.apache.gravitino.stats.storage.MemoryPartitionStatsStorageFactory`  can 
only be used for testing. | 
`org.apache.gravitino.stats.storage.LancePartitionStatisticStorageFactory` |  
No       | 1.0.0         |
+| Configuration item                              | Description                
                                                                                
                                                                                
                                          | Default value                       
                                     | Required  | Since version |
+|-------------------------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|--------------------------------------------------------------------------|-----------|---------------|
+| `gravitino.stats.partition.storageFactoryClass` | The storage factory class 
for partition statistics, which is used to store partition statistics in the 
different storage. The 
`org.apache.gravitino.stats.storage.MemoryPartitionStatsStorageFactory`  can 
only be used for testing. | 
`org.apache.gravitino.stats.storage.JdbcPartitionStatisticStorageFactory` |  No 
      | 1.0.0         |
 
 
+#### JDBC Storage (Default)
+
+Starting from version 1.2.0, Gravitino uses JDBC-based storage as the default 
partition statistics storage backend.
+This provides a reliable, production-ready solution that supports multiple 
database backends:
+
+- **MySQL** (recommended for production)
+- **PostgreSQL**
+- **H2** (suitable for testing and development)
+
+To use JDBC storage, configure the following options by adding the prefix 
`gravitino.stats.partition.storageOption.`:
+
+| Configuration item                                          | Description    
                                                    | Default value             
 | Required | Since version |
+|-------------------------------------------------------------|--------------------------------------------------------------------|----------------------------|----------|---------------|
+| `gravitino.stats.partition.storageOption.jdbc-url`          | JDBC 
connection URL (e.g., jdbc:mysql://localhost:3306/gravitino) | None             
          | Yes      | 1.2.0         |

Review Comment:
   Maybe we should change this style to camel style
   `jdbc-url` to `jdbcUrl`.



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