kgyrtkirk commented on code in PR #15465: URL: https://github.com/apache/druid/pull/15465#discussion_r1455544270
########## sql/src/test/java/org/apache/druid/sql/calcite/BaseCalciteQueryTest.java: ########## @@ -633,16 +632,16 @@ public QueryLogHook getQueryLogHook() @ClassRule public static SqlTestFrameworkConfig.ClassRule queryFrameworkClassRule = new SqlTestFrameworkConfig.ClassRule(); - @Rule + @Rule(order = 3) Review Comment: I've described the main problem in [this comment](https://github.com/apache/druid/pull/15465#discussion_r1455534365) this order=3 ensures that this rule must follow the tempfolder rule so that the `beforeNewFrameworkCreateHook` could use the `tempFolder` -- 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]
