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

Why @MapperScan scan all interface? Maybe only scan annotated with @Mapper is better? #923

Open
sheiy opened this issue Feb 22, 2024 · 2 comments

Comments

@sheiy
Copy link

sheiy commented Feb 22, 2024

No description provided.

@harawata harawata transferred this issue from mybatis/mybatis-3 Feb 22, 2024
@harawata
Copy link
Member

Hello @sheiy ,

I moved the issue to mybatis-spring repo.

@Mapper was added years after @MapperScan.
And the team didn't wan to break backward compatibility, I guess.

It might be worth considering for the next major version.
Cc: @kazuki43zoo

@sheiy
Copy link
Author

sheiy commented Feb 22, 2024

@harawata Got it. Thanks for your reply

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

No branches or pull requests

2 participants