hutiefang76 opened a new pull request, #11877:
URL: https://github.com/apache/gravitino/pull/11877

   ### What changed
   
   This PR lets the JDBC Doris catalog accept the Doris 2.1+ 
`replication_allocation` table property.
   
   It also keeps the existing single-BE fallback behavior for old-style 
`replication_num`, but skips that automatic `replication_num=1` injection when 
the user has already provided `replication_allocation`. Doris treats those two 
properties as mutually exclusive, so adding both can make table creation fail.
   
   ### Why
   
   `replication_allocation` is valid Doris table syntax, but Gravitino did not 
register it in the Doris table property metadata. In single-backend 
environments, Gravitino could also add `replication_num` automatically, which 
conflicts with the user-supplied allocation policy.
   
   ### Tests
   
   ```bash
   JAVA_HOME=$(/usr/libexec/java_home -v 17) ./gradlew 
:catalogs:catalog-jdbc-doris:test --tests 
org.apache.gravitino.catalog.doris.TestDorisCatalog --tests 
org.apache.gravitino.catalog.doris.operation.TestDorisTableOperationsSqlGeneration
 -PskipITs
   git diff --check
   ```
   
   Closes #11829


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