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

   ### Purpose
   
   Allow Partial Update tables to configure one aggregate function for every 
value field protected by a sequence group, using a sequence-centered option:
   
   ```sql
   'fields.seq.sequence-group' = 'a,b,c',
   'fields.seq.sequence-group.aggregate-function' = 'sum'
   ```
   
   Per-field aggregate functions still take precedence, followed by the 
sequence-group aggregate function and then the default aggregate function. 
Ordering and primary-key fields are never aggregated.
   
   This change also validates orphan sequence-group aggregate options, 
preserves the option during column renames, and documents the new syntax.
   
   ### Tests
   
   - `mvn -pl paimon-core -am -DfailIfNoTests=false -DwildcardSuites=none 
-Dtest='CoreOptionsTest#testSequenceGroupAggregateFunction+testSequenceGroupAggregateFunctionWithoutGroup,SchemaManagerTest#testRenameColumnInSequenceGroupAggregateFunction'
 test`
   - The new merge behavior test and the broader related test classes passed on 
the implementation baseline. On current `master`, local execution of both the 
new test and the existing `PartialUpdateMergeFunctionTest#testSequenceGroup` is 
blocked by the same JDK 8 `CodeGenLoader` / `URLClassPath` NPE before merge 
assertions run.
   


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