fix(mcp): remove temp-table lineage refs on 1.12.10 (schema not backported) - #28644
Conversation
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
Code Review ✅ ApprovedRemoves unreachable TempLineageTable references in GetLineageTool to fix compilation failures on the 1.12.10 branch. No issues found. OptionsDisplay: compact → Showing less information. Comment with these commands to change:
Was this helpful? React with 👍 / 👎 | Gitar |
|
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
🔴 Playwright Results — 15 failure(s), 23 flaky✅ 3452 passed · ❌ 15 failed · 🟡 23 flaky · ⏭️ 97 skipped
Genuine Failures (failed on all attempts)❌
|
Problem
OM
1.12.10doesn't compile.GetLineageToolreferencesTempLineageTable, but the schema that generates it (#26487) was never backported to 1.12.10 (intentional — no 1.13 UI, per @ulixius9).Breaks every build on
1.12.10(OM #28636) and Collate builds pulling the OM submodule (#4380).Fix
Remove the temp-table-lineage refs from
GetLineageTooland its test. No schema changed → no codegen needed.Verification
mvn clean install -DskipTests -pl '!openmetadata-ui'→ BUILD SUCCESSGetLineageToolTest: 8/8 passmainis unaffected.