Fix duplicate scheduling in procedure execution

[Caideyipi]

Description

Prevent duplicate procedure execution

• Add a per-procedure execution guard so a duplicated scheduler entry cannot run the same Procedure instance concurrently.
• Release the guard, active executor count, and worker state in the worker finally path.
• Release locks when a scheduled procedure is no longer RUNNABLE.

Avoid duplicate lock wakeups

• De-duplicate LockQueue waiters by procedure id before adding a waiting procedure.
• Keep PipeTaskCoordinatorLock waiting until a permit is actually acquired, even if the thread is already interrupted.

Tests

• Add coverage for duplicate procedure scheduling, duplicate LockQueue wakeups, interrupted PipeTaskCoordinatorLock acquisition, and completed PipeHandleLeaderChangeProcedure equality.

---

This PR has:

• been self-reviewed.
  • concurrent read
  • concurrent write
  • concurrent read and write
• added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage.

---

Key changed/added classes (or packages if there are too many classes) in this PR

• org.apache.iotdb.confignode.procedure.Procedure
• org.apache.iotdb.confignode.procedure.ProcedureExecutor
• org.apache.iotdb.confignode.procedure.scheduler.LockQueue
• org.apache.iotdb.confignode.manager.pipe.coordinator.task.PipeTaskCoordinatorLock

Local verification

• PASS: mvn -pl iotdb-core/confignode spotless:apply
• FAIL: mvn -pl iotdb-core/confignode -Dtest=TestLockRegime,TestProcedureExecutor,PipeHandleLeaderChangeProcedureTest,PipeTaskCoordinatorLockTest test failed during compile before test execution because local generated/snapshot dependencies were missing classes such as TGetCommitProgressReq, TPullCommitProgressReq, CommitProgressKeeper, and RegionProgress.

[codecov]

Codecov Report

❌ Patch coverage is 70.50360% with 41 lines in your changes missing coverage. Please review.
✅ Project coverage is 41.17%. Comparing base (8fad784) to head (bca866a).
⚠️ Report is 7 commits behind head on master.

Files with missing lines	Patch %	Lines
.../iotdb/confignode/procedure/ProcedureExecutor.java	68.51%	17 Missing ⚠️
...ode/procedure/impl/node/AbstractNodeProcedure.java	0.00%	6 Missing ⚠️
...dure/impl/pipe/AbstractOperatePipeProcedureV2.java	0.00%	6 Missing ⚠️
...db/confignode/procedure/TimeoutExecutorThread.java	75.00%	4 Missing ⚠️
...otdb/confignode/procedure/scheduler/LockQueue.java	66.66%	2 Missing ⚠️
...otdb/db/schemaengine/table/DataNodeTableCache.java	50.00%	2 Missing ⚠️
...g/apache/iotdb/confignode/procedure/Procedure.java	80.00%	1 Missing ⚠️
.../impl/schema/AlterTimeSeriesDataTypeProcedure.java	0.00%	1 Missing ⚠️
...ntime/SubscriptionHandleLeaderChangeProcedure.java	0.00%	1 Missing ⚠️
.../procedure/scheduler/SimpleProcedureScheduler.java	94.44%	1 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## master #17902 +/- ##
============================================
+ Coverage 41.15% 41.17% +0.02% 
Complexity 318 318 ============================================
Files 5258 5258 Lines 365613 365743 +130 Branches 47282 47300 +18 ============================================
+ Hits 150460 150596 +136 + Misses 215153 215147 -6 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jt2594838]

line 112 is weird together with line 115.

[sonarqubecloud]

Quality Gate failed

Failed conditions
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE