damansingh1313 opened a new pull request, #17905:
URL: https://github.com/apache/iceberg/pull/17905

   Several places nest an `if` whose entire body is another `if` (no else on 
either), which is equivalent to a single `if` combined with `&&`:
   
     - BaseEntriesTable.in(): matches the sibling notIn() which already uses &&
     - CachingCatalog metadata-table invalidation on cache eviction
     - Files.LocalOutputFile.createOrOverwrite() delete-on-overwrite check
     - VariantShreddingAnalyzer.observe() decimal precision/scale tracking
   
   No behavior 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]

Reply via email to