-
Notifications
You must be signed in to change notification settings - Fork 623
stream access requires transaction #33212
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
stream access requires transaction #33212
Conversation
|
!build |
|
Your Open Liberty Personal Build request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?pipelineId=643c8106-4360-4730-8df3-9f77dc853756 Note: Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
|
https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?pipelineId=643c8106-4360-4730-8df3-9f77dc853756 - Open Liberty Personal Build completed with errors/failures. |
|
!build |
|
Your Open Liberty Personal Build request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?pipelineId=90021b5b-11bf-46b5-993f-a41a02559d2e Note: Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
|
https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?pipelineId=90021b5b-11bf-46b5-993f-a41a02559d2e - Open Liberty Personal Build completed with warnings. |
|
Build has 1 intermittent test failure in a test bucket that does not use Jakarta Data |
Fixes #33177
Stream operations that continue to access data need to be in the same transaction. If there is not a transaction active, take an approach similar to what EcilpseLink is already doing and obtain all of the data first, creating a stream from that. If a transaction is active, then use getResultStream, so that Hibernate can provide a stream that retrieves data as needed. Enable tests and ensure test coverage for both ways. Also, several additional issues were discovered and opened for separate investigation after tests were able to get further along. They are noted in TODO comments in the code and linked to the user story.
release buglabel if applicable: https://github.com/OpenLiberty/open-liberty/wiki/Open-Liberty-Conventions).