-
Notifications
You must be signed in to change notification settings - Fork 16
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
Fully bring in springs code and update any spring logic as needed #61
Comments
Hi, I apologize for the trouble. |
I think your concern is quite different. The library here has long allowed the code to run with spring 4 and 5 as that is its purpose and it works well for that. You don't need to update anything.
This issue has to do with how we make this compatible with spring 6 at some point as it is likely to break given we use underlying spring from spring 3 which really is just direct code so it was decoupled from spring itself.
But again you are able to use now without issue with spring 5.
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: agharta ***@***.***>
Sent: Monday, October 9, 2023 3:14:47 AM
To: mybatis/ibatis-spring ***@***.***>
Cc: Jeremy Landis ***@***.***>; Assign ***@***.***>
Subject: Re: [mybatis/ibatis-spring] Fully bring in springs code and upgraded to spring 5 (Issue #61)
Hi, I apologize for the trouble.
Is there any news about that? Unfortunately, it is not easy for us to migrate more than 8000 queries and the solution must necessarily be gradual. We use Spring only to handle transactions and connection pooling. If ibatis-spring could be compatible with Spring 5.x it would allow us to have a gradual and painless migration.
Thank you for your precious support,
Agharta
—
Reply to this email directly, view it on GitHub<#61 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAHODI6T3VT4YI7L2RN5SRLX6OP6PAVCNFSM6AAAAAAS3DKGD2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJSGQ2TONJUHA>.
You are receiving this because you were assigned.Message ID: ***@***.***>
|
Oh, great! |
I'll update the readme when I have a chance to make that more clear and will update this ticket to clarify what the intention was. Thanks for pointing that out. |
Usage of ibatis just doesn't want to fully go away. I'm well aware of many usages up through jdk 17 using spring 5 latest. This logic was originally pulling the source but then switched to doing so without copying springs code in case they made updates. Well, its been many years, there won't be updates, so pom needs reworked to slurp in source more closely to how it works now with ability to do that one of if ever needed (unlikely) and then code needs managed up to spring 5 (internals). That will further allow feasibility of going to spring 6 in a subsequent release.
The text was updated successfully, but these errors were encountered: