[jira] [Updated] (CAMEL-25064) camel-management - EIP and service MBeans: fix bugs found in a deep review

["Claus Ibsen (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/CAMEL-25064?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Claus Ibsen updated CAMEL-25064:
--------------------------------
    Fix Version/s: 4.23.0

> camel-management - EIP and service MBeans: fix bugs found in a deep review
> --------------------------------------------------------------------------
>
>                 Key: CAMEL-25064
>                 URL: https://issues.apache.org/jira/browse/CAMEL-25064
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-core
>            Reporter: Claus Ibsen
>            Assignee: Claus Ibsen
>            Priority: Major
>             Fix For: 4.23.0
>
>
> A deep review of the EIP and service MBeans in camel-management found the bugs below. Each one was reproduced against 4.23.0-SNAPSHOT and has a test in ManagedTabularDataEdgeCasesTest that fails without the fix.
> Most of them are tabular data keyed by a value that is not unique, so the JMX operation failed with KeyAlreadyExistsException:
> # *extendedInformation of a Choice EIP with two whens that have the same predicate.*
> # *extendedInformation of a doTry with two doCatch of the same exception* (such as one with onWhen, the documented pattern) once both have caught it.
> # *listTasks of the task manager registry with two tasks of the same name* (such as the reconnect tasks of a Kafka consumer with consumersCount > 1, or tasks without a name).
> # *extendedInformation of toD, enrich, pollEnrich, recipient list, routing slip, dynamic router, wire tap and poll* (Extended statistics) when endpoints only differ in a secret, which is masked. Their hits are now merged per masked url.
> # *listStatistics of the exchange factory manager with two routes consuming the same uri.*
> # *listEndpoints of the endpoint registry with endpoints that only differ in a secret.* It also reported static=false and dynamic=false for every endpoint with a secret, as it looked them up by the masked uri.
> And:
> # *extendedInformation of a Choice EIP pairs the wrong predicate with the matches when a when is disabled* (the processor has no filter for a disabled when), and shows a disabled otherwise.
> # *ForceCompletionOnStop of the aggregate MBean returns CompletionFromBatchConsumer.*
> The tabular data of choice, doTry and listTasks have a new index item as their key; the endpoint utilization code of the eight EIP MBeans is now in one helper.
> *Not changed (for a later look)*
> * listEndpointServices of the endpoint service registry can have the same key for two routes consuming the same service.
> * The component verifier maps INCOMPLETE_PARAMETER_GROUP to ILLEGAL_PARAMETER_GROUP_COMBINATION.
> * The minimum/maximum processing time statistics can lose an update under concurrency.
> * The notification types advertised by the event notifier MBean do not match the notifications it sends.
> _Claude Code on behalf of Claus Ibsen_



--
This message was sent by Atlassian Jira
(v8.20.10#820010)