kgusakov commented on code in PR #1872:
URL: https://github.com/apache/ignite-3/pull/1872#discussion_r1165907504


##########
examples/src/main/java/org/apache/ignite/example/storage/StorageEngineExample.java:
##########
@@ -65,14 +65,18 @@ void run() throws SQLException {
             
//--------------------------------------------------------------------------------------
 
             try (Statement stmt = conn.createStatement()) {
+                stmt.executeUpdate(
+                        "CREATE ZONE ACCOUNTS_ZONE "
+                                + "ENGINE "  + engineName
+                                + " WITH DATAREGION='" + dataRegionName + "'"
+                );
                 stmt.executeUpdate(
                         "CREATE TABLE ACCOUNTS ( "
                                 + "ACCOUNT_ID INT PRIMARY KEY,"
                                 + "FIRST_NAME VARCHAR, "
                                 + "LAST_NAME  VARCHAR, "
                                 + "BALANCE    DOUBLE) "
-                                + " ENGINE " + engineName
-                                + " WITH dataRegion='" + dataRegionName + "'"
+                                + "WITH PRIMARY_ZONE = 'ACCOUNTS_ZONE'"

Review Comment:
   In general, this keyword was introduced some months ago, not in this PR.
   
   But current plan: to have the 2 different zones for primary and secondary 
storages. This topic is under discussion, though, I think



-- 
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]

Reply via email to