[ https://issues.apache.org/jira/browse/DRILL-8523?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17950002#comment-17950002 ]
ASF GitHub Bot commented on DRILL-8523: --------------------------------------- jnturton commented on code in PR #2987: URL: https://github.com/apache/drill/pull/2987#discussion_r2077669175 ########## exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/TestNestedDateTimeTimestamp.java: ########## @@ -46,6 +48,13 @@ public class TestNestedDateTimeTimestamp extends BaseTestQuery { private static final String DATAFILE = "cp.`datetime.parquet`"; private static final Map<String, Object> expectedRecord = new TreeMap<String, Object>(); + @BeforeClass + public static void setUpTimeZone() { Review Comment: I think the correct way to do it is to understand if these failing tests are telling us that something is genuinely broken in other time zones. If it is then the tests stay as is and the code gets fixed. Or, perhaps the test code is being dumb about time zones in which case that code gets a fix. Last resort for me would be "sorry it only works in UTC". Whichever it is, I think merits its own issue and PR so that its documented and searchable while in this PR it's a bit concealed. > Remove Support for Java 8 > ------------------------- > > Key: DRILL-8523 > URL: https://issues.apache.org/jira/browse/DRILL-8523 > Project: Apache Drill > Issue Type: Task > Affects Versions: 1.21.2 > Reporter: Charles Givre > Assignee: Charles Givre > Priority: Major > Fix For: 1.22.0 > > > This PR removes CI actions for Hadoop 2 and Java 8. -- This message was sent by Atlassian Jira (v8.20.10#820010)