tkalkirill commented on code in PR #13602:
URL: https://github.com/apache/ignite/pull/13602#discussion_r4104786434
##########
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:
Maybe do it this way?
```
private static final long PLANNER_TINEOUT =
getLong(IGNITE_CALCITE_PLANNER_TIMEOUT, 0);
```
--
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]
