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

Incorrect bean configuration in DefaultBatchConfiguration #4543

Closed
fmbenhassine opened this issue Feb 5, 2024 · 0 comments
Closed

Incorrect bean configuration in DefaultBatchConfiguration #4543

fmbenhassine opened this issue Feb 5, 2024 · 0 comments

Comments

@fmbenhassine
Copy link
Contributor

As of 5.0.4, bean method proxying is disabled in DefaultBatchConfiguration (inherited from the previous AbstractBatchConfiguration and which is required for native compilation to work). However, some bean definition methods call other bean definition methods to define dependencies, which is incorrect. Dependency injection through method parameters should be used instead.

This issue will introduce a minor breaking change in method signatures and should be included in a minor release.

@fmbenhassine fmbenhassine added this to the 5.2.0 milestone Feb 5, 2024
@fmbenhassine fmbenhassine modified the milestones: 5.2.0, 5.2.0-M1 Apr 8, 2024
FBibonne pushed a commit to FBibonne/spring-batch that referenced this issue Feb 2, 2025
Before this commit, the dependency injection style used to
define some infrastructure beans was incompatible with the
bean method proxying being disabled.

This commit fixes the issue by injecting dependencies through
the parameters of bean definition methods.

Resolves spring-projects#4543

Signed-off-by: Fabrice Bibonne <[email protected]>
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

1 participant