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

Add Spring Pulsar container property customizers #37559

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

onobc
Copy link
Contributor

@onobc onobc commented Sep 24, 2023

This commit adds the ability for users to register property customizers for the auto-configured Spring Pulsar listener containers.

The motivation is to ultimately give users the ability to set the AsyncTaskExecutors on the underlying listener containers to support Java virtual threads capability.

See #36347

This commit adds the ability for users to register
property customizers for the auto-configured Spring
Pulsar listener containers.

See spring-projects#36347
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Sep 24, 2023
* @since 3.2.0
*/
@FunctionalInterface
public interface PulsarContainerPropertiesCustomizer {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was originally going to put these container props customizers in the framework. However, the container factories do not need to accept a customizer as they already accept the container props. Non-boot users will already have to create the container props in order to create the container factory. They will therefore just adjust the props accordingly - no need to also have a customizer.

IOW - the framework does not need the concept of container props customizers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting-for-triage An issue we've not yet triaged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants