Skip to content

Improve GraphQlWebMvcAutoConfiguration to respect order of customizers#50908

Open
quaff wants to merge 1 commit into
spring-projects:mainfrom
quaff:patch-224
Open

Improve GraphQlWebMvcAutoConfiguration to respect order of customizers#50908
quaff wants to merge 1 commit into
spring-projects:mainfrom
quaff:patch-224

Conversation

@quaff

@quaff quaff commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 3, 2026
@snicoll

snicoll commented Jul 3, 2026

Copy link
Copy Markdown
Member

@quaff thanks, but those two are very different. The change on DataSourcePoolMetadataProvider is less conflictual, I think. The one for GraphQL should be treated as a bug though. Can you please open a separate PR so that each have a single change? Thank you.

@snicoll snicoll added the status: waiting-for-feedback We need additional information before we can continue label Jul 3, 2026
Signed-off-by: Yanming Zhou <zhouyanming@gmail.com>
@quaff quaff changed the title Use orderedStream() to respect order of beans Improve GraphQlWebMvcAutoConfiguration to respect order of customizers Jul 3, 2026
@quaff

quaff commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

@quaff thanks, but those two are very different. The change on DataSourcePoolMetadataProvider is less conflictual, I think. The one for GraphQL should be treated as a bug though. Can you please open a separate PR so that each have a single change? Thank you.

@snicoll Updated this PR and created #50911.

@spring-projects-issues spring-projects-issues added status: feedback-provided Feedback has been provided and removed status: waiting-for-feedback We need additional information before we can continue labels Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: feedback-provided Feedback has been provided status: waiting-for-triage An issue we've not yet triaged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants