pvary commented on PR #13627:
URL: https://github.com/apache/iceberg/pull/13627#issuecomment-3106640504

   > LGTM, but I would be cautious with having too much logging output now. 
Maybe we can try this for a week and see if it adds too much noise or not
   
   Added a CI check before the config change, so user builds are not affected:
   ```
         // Check if running on CI environment
         if (System.getenv('CI') != null) {
           // More verbose logging for CI environments
           events "started", "passed", "skipped", "failed"
         } else {
           events "failed"
         }
   ```


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