ammachado commented on PR #24122:
URL: https://github.com/apache/camel/pull/24122#issuecomment-4760587075
Closing this PR. @davsclaus is correct: the collision this change targets
does not exist.
CAMEL-17073 (commit `72ebeb85bdf`) already prefixes every expression-level
cache key with `@SIMPLE@`, while `SimpleFunctionExpression` writes raw function
names. Those are the only two writers to the shared `cacheExpression` map, so a
raw function name can never equal `@SIMPLE@${...}` and the keyspaces are
already disjoint. `SimpleFunctionExpression` also always writes the correct
expression for a given function name, so function-to-function reuse is correct
as well.
The `@FUNC@` prefix is therefore redundant, and the regression test
reproduced an artificial scenario (a manually seeded poison entry) that no
framework path produces. CAMEL-23800 will be resolved as "Not a Problem".
_Claude Code on behalf of Adriano Machado_
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
