gene-bordegaray commented on PR #23627: URL: https://github.com/apache/datafusion/pull/23627#issuecomment-5012820899
> On the trim, mutations say 4/5/6 are safe but 7, 13, and 14 are each the only test catching a specific branch. @blinding-pixels I am confused what you mea by brnach specific here. The tests I am sugesting to remove are in `range_partitioning.slt` which are mostly there to guarantee subset, file preserving, and target partitions behavior in `enforce_distribution.rs`. They behavior in these are not really specific to an operator rather just showing that intended behavior is happening. Now with the dedicated matrix testing we are locking in a contract of where repartitioning should happen based on these thresholds and the given key distribution. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
