Skip to content

refactor: enhance Vermeer concurrent OLAP preview - #892

Merged
imbajin merged 55 commits into
apache:masterfrom
hugegraph:fix/vermeer-concurrent-olap-preview
Nov 3, 2026
Merged

imbajin merged 55 commits into
apache:masterfrom
hugegraph:fix/vermeer-concurrent-olap-preview

Conversation

@imbajin

@imbajin imbajin commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Before → After

Before: concurrent OLAP task results are difficult to distinguish and preview.
After: allow switching among recent tasks, keep the selected task's status and preview, and show connected paths with a more responsive layout.

Validation boundary

The source PR has ongoing review findings. Independent upstream validation and live deployment acceptance are not established.

Source: hugegraph#136

@imbajin
imbajin force-pushed the fix/vermeer-concurrent-olap-preview branch from b4ab7f0 to 329e89b Compare November 3, 2026 19:00

@imbajin imbajin left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[P2] Keep restored pending tasks connected to their terminal state

When a user submits an OLAP algorithm, navigates away while it is pending, returns, and expands that same pending algorithm, selection disables restoration and sets LOADING. When the previous request finishes, the session registry updates, but the new Home's restore effect remains gated, so it does not receive the terminal success or failure state. The loading display can persist until another selection or remount.

Please keep pending restoration eligible or connect the selected in-flight task to its terminal state, and add a regression for this exact navigation and selection sequence. This is static UI-state analysis, not a browser reproduction or evidence that the background task fails.

@codecov

codecov Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.78%. Comparing base (7e1683f) to head (00912aa).
⚠️ Report is 7 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #892      +/-   ##
============================================
- Coverage     56.98%   56.78%   -0.21%     
+ Complexity     6325     3947    -2378     
============================================
  Files           622      286     -336     
  Lines         33292    20451   -12841     
  Branches       4383     3187    -1196     
============================================
- Hits          18972    11613    -7359     
+ Misses        12656     7604    -5052     
+ Partials       1664     1234     -430     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@imbajin
imbajin merged commit 0be65bf into apache:master Nov 3, 2026
20 checks passed
@imbajin
imbajin deleted the fix/vermeer-concurrent-olap-preview branch November 3, 2026 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants