abstractdog commented on code in PR #5530: URL: https://github.com/apache/hive/pull/5530#discussion_r1830965341
########## ql/src/test/org/apache/hadoop/hive/ql/udf/generic/TestGenericUDTFGetSQLSchema.java: ########## @@ -84,6 +96,8 @@ public void testWithSimpleTypes() throws Exception { @Test public void testWithDDL() throws Exception { + // Set the execution engine to mr to avoid the NPE exception in stats flow + conf.set("hive.execution.engine", "mr"); Review Comment: in general, if a test fails with tez, and it's not convenient and easy to fix, it can stay on mr, but a ticket under HIVE-28027 is needed + comment like: ``` //TODO: HIVE-27998: TestGenericUDTFGetSQLSchema to run on Tez ``` -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org