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


##########
docs/lakehouse-paimon-catalog.md:
##########
@@ -47,15 +47,25 @@ Builds with Apache Paimon `1.2`.
 | `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 |
+| `dlf-token-provider`                               | The token provider type 
for Aliyun DLF (Data Lake Formation).                                           
                                                                                
                    | (none)                                                    
                     | required if the value of `catalog-backend` is `rest` and 
accessing Aliyun DLF Paimon REST server.                                        
                            | 1.1.1            |
+| `dlf-access-key-id`                                | The access key ID for 
Aliyun DLF.                                                                     
                                                                                
                      | (none)                                                  
                       | required if the value of `catalog-backend` is `rest` 
and accessing Aliyun DLF Paimon REST server.                                    
                                | 1.1.1            |
+| `dlf-access-key-secret`                            | The access key secret 
for Aliyun DLF.                                                                 
                                                                                
                      | (none)                                                  
                       | required if the value of `catalog-backend` is `rest` 
and accessing Aliyun DLF Paimon REST server.                                    
                                | 1.1.1            |
+| `dlf-security-token`                               | The security token for 
Aliyun DLF.                                                                     
                                                                                
                     | (none)                                                   
                      | No                                                      
                                                                                
                             | 1.1.1            |
+| `dlf-token-path`                                   | The token path for 
Aliyun DLF.                                                                     
                                                                                
                         | (none)                                               
                          | No                                                  
                                                                                
                                 | 1.1.1            |
+| `dlf-token-loader`                                 | The token loader for 
Aliyun DLF.                                                                     
                                                                                
                       | (none)                                                 
                        | No                                                    
                                                                                
                               | 1.1.1            |
 
 :::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/).
+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/1.2/project/download/#filesystem-jars).
 :::
 
 :::note
 The hive backend does not support the kerberos authentication now.
 :::
 
+:::note
+If you want to use REST backend, Gravitino Paimon catalog supports Aliyun DLF 
(Data Lake Formation) as the REST catalog service. You need to configure the 
DLF-related properties like `dlf-token-provider`, `dlf-access-key-id`, and 
`dlf-access-key-secret` to connect to Aliyun DLF, more information can be found 
in the [Paimon REST 
Catalog](https://paimon.apache.org/docs/master/concepts/rest/overview/).

Review Comment:
   I would suggest you try to merge these three `note` blocks as it's really 
tedious to have so many subsequent extra text blocks here. 



##########
docs/lakehouse-paimon-catalog.md:
##########
@@ -47,15 +47,25 @@ Builds with Apache Paimon `1.2`.
 | `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 |
+| `dlf-token-provider`                               | The token provider type 
for Aliyun DLF (Data Lake Formation).                                           
                                                                                
                    | (none)                                                    
                     | required if the value of `catalog-backend` is `rest` and 
accessing Aliyun DLF Paimon REST server.                                        
                            | 1.1.1            |
+| `dlf-access-key-id`                                | The access key ID for 
Aliyun DLF.                                                                     
                                                                                
                      | (none)                                                  
                       | required if the value of `catalog-backend` is `rest` 
and accessing Aliyun DLF Paimon REST server.                                    
                                | 1.1.1            |
+| `dlf-access-key-secret`                            | The access key secret 
for Aliyun DLF.                                                                 
                                                                                
                      | (none)                                                  
                       | required if the value of `catalog-backend` is `rest` 
and accessing Aliyun DLF Paimon REST server.                                    
                                | 1.1.1            |
+| `dlf-security-token`                               | The security token for 
Aliyun DLF.                                                                     
                                                                                
                     | (none)                                                   
                      | No                                                      
                                                                                
                             | 1.1.1            |

Review Comment:
   I'm afraid that the version should be `1.2.0` as this commit will not be 
cherry-picked to `branch-1.1`, so the next version that contains this content 
will be 1.2.0. 



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