jiangxt2 commented on PR #11731: URL: https://github.com/apache/gravitino/pull/11731#issuecomment-4779579919
Fixed. The CI failure was caused by `CatalogDorisIT` asserting `PRIMARY_KEY` for all indexes — a behavior inherited from the upstream `getIndexes()` which hardcodes `PRIMARY_KEY` regardless of actual index type. The fix reads the `Index_type` column from `SHOW INDEX` using `ResultSetMetaData` to detect column availability, then maps via `mapDorisIndexType()`: - `Key_name = "PRIMARY"` → `PRIMARY_KEY` - `Index_type = "INVERTED"` → `INVERTED` - `Index_type = "BITMAP"` → `INVERTED` (cross-version compat) - `Index_type = null` (Doris 1.2.x) → inferred from index name Also addressed: catch block narrowed to use `ResultSetMetaData` instead of swallowing all `SQLException`, and added index existence assertion in 1.2.x IT. Local CI results: ``` Unit tests: 12/12 ✅ CatalogDorisIT: 15/15 ✅ CatalogDoris3xIT: 5/5 ✅ CatalogDoris4xIT: 5/5 ✅ ``` -- 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]
