uros-b opened a new pull request, #17798: URL: https://github.com/apache/iceberg/pull/17798
In `Partitioning.commonActiveFieldIds`, the loop uses an `int specIndex` counter solely to detect the first spec (the first spec seeds the set with `addAll`, the rest intersect with `retainAll`). Replaces it with a `boolean first` flag, which expresses the intent directly and drops the unused running count. Behavior-preserving; no functional change. -- 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]
