pvary opened a new pull request #2283: URL: https://github.com/apache/iceberg/pull/2283
#2254 modified how the Timestamp type is handled in `HiveIcebergFilterFactory.microsFromTimestamp` but forgot to update the tests for filter conversion. The original change (#2254) fixed the Timestamp / Date filters to work not only in UTC timezone but other timezones as well. The change was verified by adding new tests which were testing with multiple timezones, but the original `TestHiveIcebergFilterFactory.testTimestampType` test was checking the base behavior which were originally wrong and fixed in #2254, so we need to change the test too. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
