Skip to content
Open
Changes from all commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
8caac67
Pooled annotation like in EJB @Stateless, but for CDI beans
arjantijms Jan 12, 2026
43ad4f8
Add newline
arjantijms Jan 13, 2026
43e1cf1
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 13, 2026
e566893
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 13, 2026
e30f1f8
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 13, 2026
c16730b
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 13, 2026
ee05b78
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 13, 2026
a58dda7
Add exception type to the API for pool timeout
arjantijms Jan 13, 2026
4748e50
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 13, 2026
128621a
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 13, 2026
6659a3b
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 13, 2026
01e5cf4
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 13, 2026
7662842
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 13, 2026
6556b15
Use TimeoutException instead of PoolLockTimeoutException
arjantijms Jan 13, 2026
0786f4c
Change timeout default to seconds instead of minutes
arjantijms Jan 13, 2026
268dbbb
Extended class documentation
arjantijms Jan 14, 2026
3c81743
Checkstyle workaround
arjantijms Jan 14, 2026
cb4724c
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 14, 2026
1b92c64
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 14, 2026
4b2159c
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 14, 2026
9f66a1e
Fix compile error and use JDK non-null
arjantijms Jan 14, 2026
6d51d2e
Alternative for avoiding checkstyle bug
arjantijms Jan 14, 2026
5786e03
Update api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
arjantijms Jan 14, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
317 changes: 317 additions & 0 deletions api/src/main/java/jakarta/enterprise/concurrent/Pooled.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,317 @@
/*
* Copyright (c) 2026 Contributors to the Eclipse Foundation.
*
* This program and the accompanying materials are made available under the
* terms of the Eclipse Public License v. 2.0, which is available at
* http://www.eclipse.org/legal/epl-2.0.
*
* This Source Code may also be made available under the following Secondary
* Licenses when the conditions for such availability set forth in the
* Eclipse Public License v. 2.0 are satisfied: GNU General Public License,
* version 2 with the GNU Classpath Exception, which is available at
* https://www.gnu.org/software/classpath/license.html.
*
* SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0
*/
package jakarta.enterprise.concurrent;

import jakarta.enterprise.context.NormalScope;
import jakarta.enterprise.util.AnnotationLiteral;

import java.lang.annotation.Documented;
import java.lang.annotation.Inherited;
import java.lang.annotation.Retention;
import java.lang.annotation.Target;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.TimeoutException;

import static java.lang.annotation.ElementType.METHOD;
import static java.lang.annotation.ElementType.TYPE;
import static java.lang.annotation.RetentionPolicy.RUNTIME;
import static java.util.Objects.requireNonNull;
import static java.util.concurrent.TimeUnit.SECONDS;

/**
* Specify that a bean is to be pooled.
*
* <p>
* A pooled bean provides <em>max concurrency</em> (throttling) semantics. Instead of allowing an unbounded number of
* concurrent method invocations, the container exposes a single contextual reference (a normal-scoped client proxy)
* backed by a bounded pool of underlying bean instances. Each underlying instance is used by at most one thread at a
* time.
* </p>
*
* <p>
* For each invocation of a business method through the pooled bean proxy, the container must first obtain an available
* underlying instance from the pool. If an instance is available, the invocation proceeds using that instance and the
* instance remains reserved for the duration of the invocation. When the invocation completes (normally or
* exceptionally), the instance is returned to the pool and becomes available for other callers, or, depending on the
* configured exception policy, is destroyed and replaced.
* </p>
*
* <p>
* The pool size, configured by {@link #value()}, defines the maximum number of concurrent invocations that may execute
* at any one time. If more callers invoke the pooled bean proxy concurrently than there are underlying instances available,
* callers must wait for an instance to become available, or fail with a {@link TimeoutException} if the configured
* {@link #accessTimeout()} expires. This makes {@code @Pooled} suitable for protecting concurrency-sensitive resources
* (for example, rate-limited downstream APIs, or non thread-safe objects). In effect, {@code value()} acts as a concurrency
* limit (similar to a semaphore with {@code value()} permits) for invocations of the pooled bean.
* </p>
*
* <p>
* To avoid waiting, use {@link #IMMEDIATE}, which causes a {@link TimeoutException} to be thrown if no instance is
* immediately available. To wait without timing out, use {@link #UNLIMITED}.
* </p>
*
* <p>
* Example: limit concurrency to 20 invocations, failing fast when the pool is exhausted.
*
* {@snippet lang="java" :
* @Pooled(value = 20, accessTimeout = Pooled.IMMEDIATE)
* public class ReportService {
* public Report generate(...) { ... }
* }
* }
* </p>
*
* <p>
* Example: limit concurrency to 5 invocations, applying bounded backpressure.
*
* {@snippet lang="java" :
* @Pooled(value = 5, accessTimeout = 2, unit = TimeUnit.SECONDS)
* public class LegacyApiClient {
* public Response call(...) { ... }
* }
* }
* </p>
*
* <p>
* Example: two successive calls through the same injected proxy may be serviced by different underlying instances.
*
* {@snippet lang="java" :
* @jakarta.enterprise.concurrent.Pooled
* public class SomeBean {
* public void myMethod() { }
* }
*
* public class UsingBean {
*
* @jakarta.inject.Inject
* SomeBean someBean;
*
* public void run() {
*
* // myMethod call is directed to a free underlying instance for the duration of the call.
* someBean.myMethod();
*
* // A subsequent call may be directed to a different underlying instance.
* someBean.myMethod();
* }
* }
* }
* </p>
*
* <p>
* When configured via {@link #destroyOn()} and {@link #keepOn()}, the container may destroy an underlying instance after
* an exceptional invocation in order to avoid reusing an instance that may have been left in an inconsistent state.
* </p>
*
* <p>
* The {@code @Pooled} annotation can only be applied to a bean class or a producer method that defines a bean.
* It must not annotate individual bean methods.
* </p>
*
* @since 3.2
*/
@Inherited
@NormalScope
@Documented
@Target({TYPE, METHOD})
@Retention(RUNTIME)
public @interface Pooled {
Copy link
Contributor

Choose a reason for hiding this comment

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

I would add "name" parameter to give the bean a name, which could be used in configuration to specify the values externally (via vendor-specific means, until Jakarta Config is available).


/**
* Value for {@link #accessTimeout} that indicates that the bean instance
* running a bean method must be obtained immediately.
* Otherwise, a {@link TimeoutException} is thrown.
*/
long IMMEDIATE = 0;

/**
* Value for {@link #accessTimeout} that indicates to wait as long as necessary
* to obtain the bean instance that permits running a bean method.
*/
long UNLIMITED = -1;

/**
* {@return the maximum number of instances in the pool.}
*/
int value() default 10;
Copy link
Contributor

Choose a reason for hiding this comment

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

I suggest that default should be some magic constant, e.g. -1, so that implementations can distinguish between the default value and value that's explicitly set.

In case of the default, implementations would be able to change the default in an external configuration.

Copy link
Contributor

Choose a reason for hiding this comment

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

Same for the accessTimeout parameter - it should be possible to detect whether the value is the default or explicitly set.


/**
* <p>The maximum amount of time to wait to obtain a bean instance instance
* from the pool to run a single bean method against.
* If the lock cannot be obtained within the specified amount of time, a
* {@link TimeoutException} is thrown.</p>
*
* <p>Use the {@link #IMMEDIATE} constant to avoid waiting.</p>
*
* <p>Use the {@link #UNLIMITED} constant to avoid timing out.</p>
*
* <p>The supplied timeout value must be positive or one of the constants
* described above.</p>
*
* @return the time to wait to obtain a bean instance for the duration of
* a single method call.
*/
long accessTimeout() default 5;

/**
* {@return the {@link TimeUnit} to use for the {@link #accessTimeout()}}
*/
TimeUnit unit() default SECONDS;

/**
* The types of {@link Throwable Throwables} which must cause the destruction of a pooled bean instance.
* <p>
* If the pooled bean instances throws any Throwable which is an instance of a type set in the destroyOn property,
* then the bean will be destroyed, except if the throwable is also an instance of a type set in the {@link
* #keepOn()} property.
* </p>
*
* @return The types of {@link Throwable Throwables} which must cause the destruction of a pooled bean instance
*/
Class<? extends Throwable>[] destroyOn() default {};

/**
* The types of {@link Throwable Throwables} which must not cause the destruction of a pooled bean instance.
* <p>
* If the {@link #destroyOn()} property is empty, but the keepOn property is not, then the bean will be
* destroyed for any Throwable that isn't an instance of a type in the keepOn property.
* When the {@link #destroyOn()} property is not empty, then the keepOn property takes precedence.
* If a pooled bean instance throws a throwable which is an instance of a type in both the
* destroyOn and keepOn properties, then the bean will not be destroyed.
* </p>
*
* @return The types of {@link Throwable Throwables} which must not cause the destruction of a pooled bean
* instance.
*/
Class<? extends Throwable>[] keepOn() default {};

/**
* Supports inline instantiation of the Pooled annotation.
*
* @since 3.2
*/
public static final class Literal extends AnnotationLiteral<Pooled> implements Pooled {

private static final long serialVersionUID = 1L;

private final int value;
private final long accessTimeout;
private final TimeUnit unit;
private final Class<? extends Throwable>[] destroyOn;
private final Class<? extends Throwable>[] keepOn;

/**
* Default instance of the {@link Pooled} annotation.
*/
@SuppressWarnings("unchecked")
public static final Literal INSTANCE = of(
10,
5,
SECONDS,
new Class[]{},
new Class[]{}
);

/**
* Instance of the {@link Pooled} annotation.
*
* @param value The maximum number of instances in the pool.
* @param accessTimeout The maximum amount of time to wait attempting to retrieve an instance from the pool.
* @param unit The {@link TimeUnit} to use for the {@link #accessTimeout()}
* @param destroyOn The types of {@link Throwable Throwables} which must cause the destruction of a pooled bean instance.
* @param keepOn The types of {@link Throwable Throwables} which must not cause the destruction of a pooled bean instance.

* @return instance of the {@link Pooled} annotation
*/
public static Literal of(
final int value,
final long accessTimeout,
final TimeUnit unit,
final Class<? extends Throwable>[] destroyOn,
final Class<? extends Throwable>[] keepOn
) {

requireNonNull(unit, "unit: null");
requireNonNull(destroyOn, "destroyOn: null");
requireNonNull(keepOn, "keepOn: null");

return new Literal(
value,
accessTimeout,
unit,
destroyOn,
keepOn
);
}

private Literal(
final int value,
final long accessTimeout,
final TimeUnit unit,
final Class<? extends Throwable>[] destroyOn,
final Class<? extends Throwable>[] keepOn
) {

this.value = value;
this.accessTimeout = accessTimeout;
this.unit = unit;
this.destroyOn = destroyOn;
this.keepOn = keepOn;
}


/**
* {@return the maximum number of instances in the pool.}
*/
@Override
public int value() {
return value;
}

/**
* {@return the maximum amount of time to wait attempting to retrieve an instance from the pool.}
*/
@Override
public long accessTimeout() {
return accessTimeout;
}

/**
* {@return the {@link TimeUnit} to use for the {@link #accessTimeout()}}
*/
@Override
public TimeUnit unit() {
return unit;
}

/**
* {@return the types of {@link Throwable Throwables} which must cause the destruction of a pooled bean instance.}
*/
@Override
public Class<? extends Throwable>[] destroyOn() {
return destroyOn;
}

/**
* {@return the types of {@link Throwable Throwables} which must not cause the destruction of a pooled bean instance.}
*/
@Override
public Class<? extends Throwable>[] keepOn() {
return keepOn;
}

}
}