AMashenkov commented on code in PR #7564:
URL: https://github.com/apache/ignite-3/pull/7564#discussion_r2798670152
##########
modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/systemviews/ItZonesSystemViewTest.java:
##########
@@ -49,14 +49,6 @@ public class ItZonesSystemViewTest extends
AbstractSystemViewTest {
@Test
public void systemViewDefaultZone() {
IgniteImpl node = unwrapIgniteImpl(CLUSTER.aliveNode());
-
- // Check that there is no default zone yet before test table is
created.
- assertQuery("SELECT COUNT(*) FROM SYSTEM.ZONES").returns(0L).check();
- // Table for lazy default zone creation.
- createTableOnly("test_table");
- // Check that the default zone was created and is presented on zone
view.
- assertQuery("SELECT COUNT(*) FROM SYSTEM.ZONES").returns(1L).check();
-
Review Comment:
Ok.
However, the first check looks useful (ensure system views works well with
no default zone exists),
as long as we allow the case when no default zone exists.
--
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]
