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

cacheEnabled bypasses BATCH mode #3408

Open
TrentonAdams opened this issue Feb 12, 2025 · 1 comment
Open

cacheEnabled bypasses BATCH mode #3408

TrentonAdams opened this issue Feb 12, 2025 · 1 comment

Comments

@TrentonAdams
Copy link

cacheEnabled is true by default

When specifying a default executor type of BATCH, it is not used if cacheEnabled is true, and instead creates a cached executor.

To me this seems like a bug. If they are mutually exclusive, there should just be an ExecutorType.CACHED, because it took me hours to figure this out. I finally dived into the code, not thinking it would be a mybatis code problem. I should have done it earlier, but my claim stands, I consider that a bug.

@TrentonAdams TrentonAdams changed the title batch mode enabled but not performant cacheEnabled bypasses BATCH mode Feb 13, 2025
@harawata
Copy link
Member

Hello @TrentonAdams ,

We need the details about the problem you are experiencing.
Please provide a test case [1] or small demo project [2].
We basically need to understand both 1) what you expect and 2) what actually happens.

[1] https://github.com/mybatis/mybatis-3/wiki/Unit-Test
[2] https://github.com/harawata/mybatis-issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants