Repository navigation
refactor: enhance Vermeer concurrent OLAP preview - #892
Conversation
b4ab7f0 to
329e89b
Compare
imbajin
left a comment
There was a problem hiding this comment.
[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 Report✅ All modified and coverable lines are covered by tests. 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. 🚀 New features to boost your workflow:
|
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