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

fix: DefaultResultSetHandler memory waste #3113 #3114

Merged
merged 2 commits into from
Mar 20, 2024

Conversation

whojes-toss
Copy link
Contributor

java.lang.reflect.Constructor.getParameterTypes method clones objects

public Class<?>[] getParameterTypes() {
     return parameterTypes.clone();
 }

this change will reduce not necessary memory allocations.

@coveralls
Copy link

Coverage Status

coverage: 87.117% (+0.001%) from 87.116%
when pulling 926c451 on whojes-toss:issue-3113
into b044200 on mybatis:master.

@harawata harawata added this to the 3.5.16 milestone Mar 20, 2024
@harawata harawata merged commit 0ec8fcb into mybatis:master Mar 20, 2024
19 checks passed
@harawata
Copy link
Member

Thank you, @whojes-toss !

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