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

   **Cherry-pick Information:**
   - Original commit: `3704b7a5bfe978421e43d53d5472f6c52bc15073`
   - Original PR: #12737
   - Target branch: `branch-1.3`
   - Status: ✅ Conflict resolved manually in the `CatalogDoris4xIT` assertion 
imports
   - Replaces: #12753
   
   ### What changes were proposed in this pull request?
   
   This pull request manually backports #12737 to `branch-1.3`. It preserves 
the shared Doris CREATE/ALTER index field-shape validation and its focused unit 
and Doris 4.x integration coverage.
   
   ### Why are the changes needed?
   
   The automated cherry-pick PR #12753 contains unresolved conflict markers in 
`CatalogDoris4xIT` and cannot be updated from the contributor's fork. This PR 
resolves that conflict while preserving the original fix.
   
   Fix: #12736
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. Doris CREATE TABLE and ALTER TABLE ADD INDEX requests containing 
multiple fields, nested field paths, empty field shapes, or blank field names 
now fail with an explicit `IllegalArgumentException` instead of being truncated 
or producing invalid SQL. Valid single-field index operations are unchanged, 
and no public API or property key is added or removed.
   
   ### How was this patch tested?
   
   - `./gradlew :catalogs:catalog-jdbc-doris:spotlessCheck`
   - `./gradlew rat`
   - `./gradlew :catalogs:catalog-jdbc-doris:test --tests 
'org.apache.gravitino.catalog.doris.operation.TestDorisTableOperationsSqlGeneration'
 -PskipITs`
   - `./gradlew :catalogs:catalog-jdbc-doris:test --tests 
'org.apache.gravitino.catalog.doris.integration.test.CatalogDoris4xIT.testAddAndDropInvertedIndex'
 -PskipDockerTests=false -PdorisMultiVersionTest` (1 test, 0 skipped, 0 
failures, 0 errors)
   - `./gradlew :catalogs:catalog-jdbc-doris:build -x test`
   


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