Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(RouteDiagram): Proper visualisation of parallel processing of the multicast #1022

Merged
merged 1 commit into from
Jul 12, 2024

Conversation

mmelko
Copy link
Collaborator

@mmelko mmelko commented Jul 12, 2024

Also I added tests for the multicast rendering and did small variables refactor.

Route visualisation without parallelProcessing=true:
Screenshot 2024-07-12 at 09 46 17

with parallelProcessing=true:
Screenshot 2024-07-12 at 09 45 50

@tadayosi
Copy link
Member

tadayosi commented Jul 12, 2024

@mmelko Don't forget to point to the issue that the pull req is supposed to fix from here.

No worries. It's a SO issue :-p

Copy link

Test Results

  8 files  ±0    8 suites  ±0   36m 36s ⏱️ +2s
 66 tests ±0   64 ✅ ±0   2 💤 ±0  0 ❌ ±0 
470 runs  ±0  445 ✅  - 1  25 💤 +1  0 ❌ ±0 

Results for commit abdaaee. ± Comparison against base commit e6a61a1.

@hawtio-ci
Copy link

hawtio-ci bot commented Jul 12, 2024

Test results

Run attempt: 1999
Detailed summary

NAME TESTS PASSED ✅ SKIPPED 💤 FAILED ❌ ERRORS 🚫 TIME 🕖
results-quarkus-node(18)-java(17)-firefox 67 64 3 0 0 317.078
results-quarkus-node(18)-java(21)-firefox 1 1 0 0 0 7.076
results-quarkus-node(20)-java(17)-firefox 67 64 3 0 0 308.876
results-quarkus-node(20)-java(21)-firefox 67 64 3 0 0 311.383
results-springboot-node(18)-java(17)-firefox 67 63 4 0 0 311.961
results-springboot-node(18)-java(21)-firefox 67 63 4 0 0 318.173
results-springboot-node(20)-java(17)-firefox 67 63 4 0 0 310.152
results-springboot-node(20)-java(21)-firefox 67 63 4 0 0 312.237

@mmelko mmelko merged commit 736cce6 into hawtio:main Jul 12, 2024
13 checks passed
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