[CAMEL-14414] Aggregation completion has inconsistent property/header handling

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.24.3, 3.x
• Fix Version/s: 3.1.0, 3.0.2, 2.25.1
• Component/s: camel-core
• Estimated Complexity: Unknown
• Assignee: Unassigned
• Reporter: Gianandrea Rigoni
• Created: 18/Jan/20 14:14
• Updated: 08/Feb/20 08:29
• Resolved: 23/Jan/20 08:55

Description

following headers or properties are handled in an inconsistent way
sometimes setted as prop then removed as headers or viceversa
documentation in describing how to set them is also slightly inconsistent

affected flags are (I am on purpose avoiding naming them header or property)
AGGREGATION_COMPLETE_CURRENT_GROUP
AGGREGATION_COMPLETE_ALL_GROUPS
AGGREGATION_COMPLETE_ALL_GROUPS_INCLUSIVE

also chaining aggregations is not working when using
AGGREGATION_COMPLETE_CURRENT_GROUP
AGGREGATION_COMPLETE_ALL_GROUPS

because setting is done on prop and removal is done on header level

I have created a PR proposing a change

Issue Links

• links to: GitHub Pull Request #3494
• links to: GitHub Pull Request #3496
• links to: GitHub Pull Request #3497
• links to: GitHub Pull Request #3554