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

Add test for result streaming with data-jpa #171

Conversation

christophstrobl
Copy link
Member

See: spring-projects/spring-data-jpa#2848

The test will fail until spring-projects/spring-data-jpa#2880 is merged to main and back ported to the 3.0.x maintenance branch.

@christophstrobl
Copy link
Member Author

The mentioned PR has been merged to main and backported to 3.0.x.

@wilkinsona

This comment was marked as resolved.

@wilkinsona

This comment was marked as resolved.

@wilkinsona
Copy link
Member

wilkinsona commented Mar 24, 2023

The new test will fail until Boot's switched to Spring Data snapshots again (in ~2 weeks time). I'm inclined to either wait for that to happen or to temporarily override the version of Spring Data that's used. WDYT, @christophstrobl?

@christophstrobl
Copy link
Member Author

christophstrobl commented Mar 24, 2023

2 more weeks should be fine, we'd still have enough time to revisit the issue if something (unexpectedly) breaks. Same for #161 .

@wilkinsona wilkinsona changed the title Add test for result streaming with data-jpa. Add test for result streaming with data-jpa Mar 28, 2023
@christophstrobl
Copy link
Member Author

Boot already picked up the Spring Data BOM for 2022.0.5 (spring-projects/spring-boot#34903). Please let me know if there's anything I can help with.

@wilkinsona
Copy link
Member

Thanks for the nudge, Christoph.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants