QuakeWang opened a new pull request, #8272:
URL: https://github.com/apache/paimon/pull/8272

   ### Purpose
   
   When a nested-key was configured together with a count limit, `agg` 
short-circuited with `if (acc.size() >= countLimit) return accumulator;`. This 
early return was shared with the no-key path, so once the nested array reached 
the limit, updates to existing keys (and `nested-sequence-field` updates) were 
silently dropped — the limit was meant to bound the number of distinct keys, 
not block in-place updates.
   
   This PR splits the two paths: the no-key path keeps the count-limit 
truncation, while the keyed path merges by key so existing keys are always 
updated and the limit only caps the number of new keys.
   
   ### Tests
   
   - `mvn test -pl paimon-core -Dtest=FieldAggregatorTest`
   


-- 
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]

Reply via email to