Red Hat Bugzilla – Bug 824818
[eap6] setting Keepalive Time when adding a new ThreadPool
Last modified: 2015-11-01 19:45:31 EST
Description of problem:When creating a new threadPool (or overall configuring Keepalive Time) It is not simple to guess Time unit. It would be nice if UI suggested time units by enumerating java.util.concurrent.TimeUnit. Moreover, Keepalive Time must be set, otherwise resource is not created. UI does not handle this.
Version-Release number of selected component (if applicable):
JON 3.1.ER4 EAP6.ER8
Steps to Reproduce:
1.have imported EAP6 in standalone mode
2.go to threads
3.add child of type 'ThreadPool'
Actual results: User is not required to set Keepalive Time, but it is required to create the resource. Time Unit is a text field and it is not straightforward to guess correct value
Expected results: UI notifies me, that Keepalive Time is required and pops-up a combo for selecting Unit.
This has now been fixed in master branch by the commits for bug 811288.
The commits relevant to this bug from master branch:
The keepalive-time validation is complicated on the EAP6 side.
Here are the rules from the actual resource description:
1) The keepalive-time can be null
2) If either time or unit are non-null then both of them need to be non-null
Currently there is no support to implement the complex validation rules described by EAP6 without lists that support min and max. In this case it would be implemented with a list with min=0 and max=1 and if a single element is present then translate that into the EAP6 map having a value, if list has a size of zero set the property to null.
However, these rules are not enforced by EAP6 (tested all the resources that have this property). Users can set either time or unit to null and the configuration is accepted by the server. Until the more complex solution is implemented given the weak enforcement from EAP6 side, all the required flags have been removed from the resource descriptor. This will avoid user confusion and prevent false configuration errors.
master branch commit for required removal: