Skip to content

Avoid multiple DeferredRepositoryInitializationListener registrations in RepositoryConfigurationDelegate #3219

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

Conversation

xeounxzxu
Copy link
Contributor

  • You have read the Spring Data contribution guidelines.
  • You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.
  • You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).

@mp911de

hello.

I changed the content for the following reasons.

issue: #3287

If you use two or more EnableJpaRepositories and use DEFERRED bootstrap mode, an error that the bean cannot be registered will occur.

Logic was added to check the presence of a bean. And we ran a repository check as a test case.

@xeounxzxu xeounxzxu changed the title Fix Add Multiple DeferredRepositoryInitializationListener in `Reposit… Fix Add Multiple DeferredRepositoryInitializationListener in RepositoryConfigurationDelegate Dec 9, 2024
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Dec 9, 2024
@xeounxzxu
Copy link
Contributor Author

@odrotbohm Could you please check this PR?

@mp911de mp911de added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels May 9, 2025
@mp911de mp911de assigned mp911de and unassigned odrotbohm May 9, 2025
@mp911de mp911de added this to the 3.4.6 (2024.1.6) milestone May 9, 2025
@mp911de mp911de changed the title Fix Add Multiple DeferredRepositoryInitializationListener in RepositoryConfigurationDelegate Avoid multiple DeferredRepositoryInitializationListener registrations in RepositoryConfigurationDelegate May 9, 2025
@mp911de mp911de linked an issue May 9, 2025 that may be closed by this pull request
mp911de pushed a commit that referenced this pull request May 9, 2025
…ns in `RepositoryConfigurationDelegate`.

Closes #3287
Original pull request: #3219
mp911de added a commit that referenced this pull request May 9, 2025
Reformat code. Simplify flow. Add ticket reference to test.

See #3287
Original pull request: #3219
mp911de pushed a commit that referenced this pull request May 9, 2025
…ns in `RepositoryConfigurationDelegate`.

Closes #3287
Original pull request: #3219
mp911de added a commit that referenced this pull request May 9, 2025
Reformat code. Simplify flow. Add ticket reference to test.

See #3287
Original pull request: #3219
mp911de pushed a commit that referenced this pull request May 9, 2025
…ns in `RepositoryConfigurationDelegate`.

Closes #3287
Original pull request: #3219
mp911de added a commit that referenced this pull request May 9, 2025
Reformat code. Simplify flow. Add ticket reference to test.

See #3287
Original pull request: #3219
@mp911de
Copy link
Member

mp911de commented May 9, 2025

Thank you for your contribution. That's merged, polished, and backported now. Please make sure to use to use our formatter configuration on your next contribution to introduce less formatting-related changes. I fixed those.

@mp911de mp911de closed this May 9, 2025
mp911de pushed a commit that referenced this pull request May 9, 2025
…ns in `RepositoryConfigurationDelegate`.

Closes #3287
Original pull request: #3219
mp911de added a commit that referenced this pull request May 9, 2025
Reformat code. Simplify flow. Add ticket reference to test.

See #3287
Original pull request: #3219
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DeferredRepositoryInitializationListener is registered multiple times
5 participants