kevinjqliu opened a new pull request, #1350: URL: https://github.com/apache/iceberg-python/pull/1350
When looking into #1346, I was confused by why the `datetime.datetime.utcfromtimestamp` warning only errored for the branch and not for the `main` branch. Turns out there's a bug in the `_deprecation_warning` code where **all warnings** are suppressed when displaying a deprecation warning. The `datetime.datetime.utcfromtimestamp` warning is suppressed in the `main` branch because the test also triggered other deprecation warnings. When those deprecation warnings are removed in the feature branch, the only other warning is shown and causes the test to fail. This PR changes `_deprecation_warning` to temporarily override warning handling. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
