Skip to content

fix(server): fix the scheduler and the scheduler selection logic#3082

Open
Tsukilc wants to merge 9 commits into
apache:masterfrom
hugegraph:fix/scheduler
Open

fix(server): fix the scheduler and the scheduler selection logic#3082
Tsukilc wants to merge 9 commits into
apache:masterfrom
hugegraph:fix/scheduler

Conversation

@Tsukilc

@Tsukilc Tsukilc commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

Replaces #2937 so the source branch is hugegraph/hugegraph:fix/scheduler and org collaborators can push follow-up fixes directly.

New Features

Server IDs now support automatic generation, eliminating the need for manual configuration.

Refactoring

Significantly simplified the task scheduling architecture by adopting a single-node scheduling path by default, removing multi-node scheduling and role-election–related controls.

Streamlined server information and scheduling management logic by eliminating redundant multi-node–related workflows.

Configuration Changes

Removed explicit scheduling type configuration; the scheduling mode is now determined by the runtime environment.

Bug Fixes

Adjusted the handling logic for task cancellation/deletion and missing tasks, improving exception handling and state management.

Tests

Updated test cases to accommodate the new scheduling behavior and serialization differences.

@dosubot dosubot Bot added size:XXL This PR changes 1000+ lines, ignoring generated files. bug Something isn't working tests Add or improve test cases labels Jul 6, 2026
@codecov

codecov Bot commented Jul 6, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 122 lines in your changes missing coverage. Please review.
✅ Project coverage is 1.48%. Comparing base (03e6b8e) to head (abb09aa).

Files with missing lines Patch % Lines
...pache/hugegraph/task/DistributedTaskScheduler.java 0.00% 47 Missing ⚠️
...n/java/org/apache/hugegraph/core/GraphManager.java 0.00% 31 Missing ⚠️
...g/apache/hugegraph/task/StandardTaskScheduler.java 0.00% 15 Missing ⚠️
.../apache/hugegraph/task/TaskAndResultScheduler.java 0.00% 14 Missing ⚠️
...n/java/org/apache/hugegraph/StandardHugeGraph.java 0.00% 7 Missing ⚠️
...a/org/apache/hugegraph/task/ServerInfoManager.java 0.00% 5 Missing ⚠️
.../main/java/org/apache/hugegraph/task/HugeTask.java 0.00% 3 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (03e6b8e) and HEAD (abb09aa). Click for more details.

HEAD has 2 uploads less than BASE
Flag BASE (03e6b8e) HEAD (abb09aa)
3 1
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #3082       +/-   ##
============================================
- Coverage     36.80%   1.48%   -35.32%     
+ Complexity      338      21      -317     
============================================
  Files           805     783       -22     
  Lines         68587   65796     -2791     
  Branches       9029    8528      -501     
============================================
- Hits          25241     975    -24266     
- Misses        40653   64737    +24084     
+ Partials       2693      84     -2609     

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:XXL This PR changes 1000+ lines, ignoring generated files. tests Add or improve test cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants