Skip to content

Conversation

@andrew8er
Copy link
Contributor

Short Description

Proposed Changes

Side Effects

Testing

Resolved Issues

Fixes: #2637

@andrew8er andrew8er changed the base branch from main to 2449-spring-boot-migration November 5, 2025 17:31
@andrew8er andrew8er changed the title 2637 use webflux 2637: Use WebFlux Nov 5, 2025
@github-actions
Copy link

github-actions bot commented Nov 5, 2025

Coverage report for administration

St.
Category Percentage Covered / Total
🟡 Statements 75.13% 2565/3414
🔴 Branches 54.33% 552/1016
🔴 Functions 56.22% 434/772
🟡 Lines 74.35% 2385/3208

Test suite run success

330 tests passing in 56 suites.

Report generated by 🧪jest coverage report action from f52e91f

@f1sh1918
Copy link
Contributor

Are you sure we really need this if we use virtual threads?
I mean performance improvements do never harm.
Not sure how much additional adjustments have to be done for webflux
I just mean we should think if the effort is it worth (which i cannot estimate)

@andrew8er
Copy link
Contributor Author

andrew8er commented Nov 13, 2025

I was just trying to get a feel of how much work this would be. Turns out that for the most part, it is very simple, but the MultipartGraphQLHandler is quite complicated. I will experiment with this a little more, when there is time. In the meantime, we should be good to go with virtual threads.

@seluianova seluianova force-pushed the 2449-spring-boot-migration branch from 95ed803 to b4331d3 Compare November 17, 2025 12:03
Base automatically changed from 2449-spring-boot-migration to main November 18, 2025 09:43
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.

Use Spring Webflux

3 participants