zstan commented on code in PR #13602:
URL: https://github.com/apache/ignite/pull/13602#discussion_r4132200401
##########
modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/CalcitePlanningDumpTest.java:
##########
@@ -36,10 +38,19 @@
* This test assumes no Calcite classes are loaded in the current JVM before
CalciteQueryProcessor.
* Violating this invariant may cause the test to fail due to premature
Calcite initialization.
*/
+@ExtendWith(SystemPropertiesExtension.class)
@WithSystemProperty(key = IGNITE_CALCITE_PLANNER_TIMEOUT, value = "1000")
public class CalcitePlanningDumpTest extends AbstractBasicIntegrationTest {
/** */
- private static final long PLANNER_TIMEOUT =
getLong(IGNITE_CALCITE_PLANNER_TIMEOUT, 0);
+ private static long plannerTimeout;
+
+ /** {@inheritDoc} */
+ @BeforeAll
+ @Override protected void beforeTestsStarted() throws Exception {
+ plannerTimeout = getLong(IGNITE_CALCITE_PLANNER_TIMEOUT, 0);
Review Comment:
try it - as i said earlier - this not work cause sys property is not
initialized yet, you can easilly check it.
--
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]
