u70b3 opened a new pull request, #68781: URL: https://github.com/apache/doris/pull/68781
Per the updated direction in [#66497 (comment)](https://github.com/apache/doris/issues/66497#issuecomment-6053021847), Lance index mutations move to synchronous SQL with bounded worker execution. A durable asynchronous job framework is no longer part of the initial implementation, so this change reverts it explicitly rather than leaving it disabled or maintaining two execution paths. ### Removed - the durable Lance index job state machine, manager, same-name fences, unresolved-job quotas, and replay infrastructure introduced by #67235 - the job-specific parts of #67630: durable job creation and JobId results, `SHOW LANCE INDEX JOB(S)` with its grammar and plan wiring, and the catalog guards tied to unresolved jobs - the three `lance_index_job_max_unresolved_*` quota configuration keys and the job-specific tests ### Preserved - read-only `SHOW INDEX` (#66637) and the `lance_index_entries` inspection TVF (#66671) - the statically validated DDL surface (#67201); `enable_lance_index_mutation` remains the mutation gate - the privilege, schema, parameter, snapshot, and IF-condition validation from #67630, rewired as pure admission preflight; a mutation that passes the whole preflight now ends in the shared not-supported rejection until the synchronous implementation lands - the schema-contract, name-normalization, and dataset-locator helpers, relocated out of the removed `job` package next to their remaining callers ### Journal and image compatibility `enable_lance_index_mutation` has defaulted to `false` since #67630 merged, and no released build exposes it enabled, so no deployment can hold persisted job records unless someone manually enabled an undocumented, gated feature on an unreleased build. On that basis: the edit-log operation `OP_LANCE_INDEX_JOB_UPSERT` (= 500) reader is removed and the code is not reused; the image chunk the old manager wrote is skipped through the existing deprecated-module list, so images from previous builds still load. An edit log containing job records from such a manual configuration would fail replay — accepted for an unreleased surface. ### Evidence Focused FE suites (`mvn test -pl fe-common,fe-core -am -Dtest='Lance*'`): 411 tests run, 0 failures, 0 errors, 6 skipped (known environment skips), checkstyle included, BUILD SUCCESS. Net change: 60 files, +247/−8517. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
