You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ignite.apache.org by "IgGusev (via GitHub)" <gi...@apache.org> on 2025/10/27 11:10:36 UTC
[PR] IGNITE-26824 Add migration tools doc [ignite-3]
IgGusev opened a new pull request, #6849:
URL: https://github.com/apache/ignite-3/pull/6849
Thank you for submitting the pull request.
To streamline the review process of the patch and ensure better code quality
we ask both an author and a reviewer to verify the following:
### The Review Checklist
- [ ] **Formal criteria:** TC status, codestyle, mandatory documentation. Also make sure to complete the following:
\- There is a single JIRA ticket related to the pull request.
\- The web-link to the pull request is attached to the JIRA ticket.
\- The JIRA ticket has the Patch Available state.
\- The description of the JIRA ticket explains WHAT was made, WHY and HOW.
\- The pull request title is treated as the final commit message. The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
- [ ] **Design:** new code conforms with the design principles of the components it is added to.
- [ ] **Patch quality:** patch cannot be split into smaller pieces, its size must be reasonable.
- [ ] **Code quality:** code is clean and readable, necessary developer documentation is added if needed.
- [ ] **Tests code quality:** test set covers positive/negative scenarios, happy/edge cases. Tests are effective in terms of execution time and resources.
### Notes
- [Apache Ignite Coding Guidelines](https://cwiki.apache.org/confluence/display/IGNITE/Java+Code+Style+Guide)
--
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: notifications-unsubscribe@ignite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] IGNITE-26824 Add migration tools doc [ignite-3]
Posted by "tmgodinho (via GitHub)" <gi...@apache.org>.
tmgodinho commented on code in PR #6849:
URL: https://github.com/apache/ignite-3/pull/6849#discussion_r2465963054
##########
docs/_docs/administrators-guide/config/cli-config.adoc:
##########
@@ -37,21 +37,21 @@ ignite.auth.basic.password=
|======
|Property|Default|Description
-|ignite.jdbc.key-store.path||Path to the JDBC keystore file.
-|ignite.cluster-endpoint-url|http://127.0.1.1:10300|
+|ignite.jdbc.key-store.path||Path to the JDBC keystore file for SSL/TLS client authentication.
+|ignite.cluster-endpoint-url|http://127.0.1.1:10300|URL endpoint for connecting to the Ignite cluster REST API.
|ignite.jdbc.client-auth||If JDBC client authorization is enabled in CLI.
-|ignite.rest.key-store.password||
-|ignite.jdbc.key-store.password||
+|ignite.rest.key-store.password||Password for the REST API keystore file.
+|ignite.jdbc.key-store.password||Password for the JDBC keystore file.
|ignite.cli.sql.multiline|true|Enables multiline input mode for SQL commands in the CLI.
|ignite.cli.syntax-highlighting|true|Enables syntax highlighting in CLI output.
-|ignite.rest.trust-store.path||
-|ignite.jdbc.trust-store.password||
-|ignite.auth.basic.username||
-|ignite.jdbc-url|jdbc:ignite:thin://127.0.0.1:10800|
-|ignite.rest.key-store.path||
-|ignite.rest.trust-store.password||
-|ignite.jdbc.trust-store.path||
-|ignite.auth.basic.password||
+|ignite.rest.trust-store.path||Path to the REST API truststore file for SSL/TLS server verification.
+|ignite.jdbc.trust-store.password||Password for the JDBC truststore file.
+|ignite.auth.basic.username||Username for basic authentication when connecting to the cluster.
+|ignite.jdbc-url|jdbc:ignite:thin://127.0.0.1:10800|JDBC connection URL for connecting to the Ignite cluster.
+|ignite.rest.key-store.path||Path to the REST API keystore file for SSL/TLS client authentication.
+|ignite.rest.trust-store.password||Password for the REST API truststore file.
+|ignite.jdbc.trust-store.path||Path to the JDBC truststore file for SSL/TLS server verification.
+|ignite.auth.basic.password||Password for basic authentication when connecting to the cluster.
Review Comment:
Hi @IgGusev , perhaps we should order the properties by componnent. For instance, first the auth properties then JDBC and rest
--
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: notifications-unsubscribe@ignite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] IGNITE-26824 Add migration tools doc [ignite-3]
Posted by "tmgodinho (via GitHub)" <gi...@apache.org>.
tmgodinho commented on code in PR #6849:
URL: https://github.com/apache/ignite-3/pull/6849#discussion_r2465978233
##########
docs/_docs/installation/migration-from-ai2/persistent-migration.adoc:
##########
@@ -0,0 +1,133 @@
+= Persistent Data Migration
+
+Ignite 3 persistent data storage is not directly compatible with Ignite 2. You need to convert your Ignite 2 caches to Ignite 3 tables. The migration tool can be used to convert most Ignite 2 caches
+
+There are two options for creating the required Ignite 3 configuration:
+
+* <<Replicating Caches to Ignite 3>>
+* <<Migrating Persistent Data>>
+
+Regardless of the approach you choose, you need to download the migration tools first:
+
+. link:https://dlcdn.apache.org/ignite/3.1.0/migration-tools-cli-3.1.0.zip[Download] the migration tools.
+. Unpack the downloaded archive to your machine.
+. To explore the available commands in help, run the following command from the migration tools directory:
++
+[source, shell]
+----
+bin/migration-tools --help
+----
++
+or
++
+[source, shell]
+----
+bin/migration-tools {command} --help
+----
++
+Substitute `{command}` with the actual command name.
+
+== Replicating Caches to Ignite 3
+
+The migration tool can analyze Ignite 2 configuration and create DDL requests required to reproduce the caches. These caches will be empty, and you can then use the link:migration-from-gg-8/dcr-from-gg8[data center replication between Ignite 2 and Ignite 3] to transfer data between your clusters.
Review Comment:
`link:migration-from-gg-8/dcr-from-gg8[data center replication between Ignite 2 and Ignite 3]` is this correct??
--
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: notifications-unsubscribe@ignite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] IGNITE-26824 Add migration tools doc [ignite-3]
Posted by "tmgodinho (via GitHub)" <gi...@apache.org>.
tmgodinho commented on code in PR #6849:
URL: https://github.com/apache/ignite-3/pull/6849#discussion_r2465972654
##########
docs/_docs/installation/migration-from-ai2/config.adoc:
##########
@@ -0,0 +1,217 @@
+= Configuration Migration
+
+This section describes how to configure a Ignite 2 cluster into which you will migrate all the components of your Ignite 2 cluster.
+
+== Configure the Cluster
+
+You need to configure the cluster you have created to match the Ignite 2 cluster you are migrating from.
+
+While cluster configuration files for Ignite 2 are XML beans, Ignite 3 uses the HOCON format. Moreover, many configuration structures in version 9 are different from those used in version 8.
Review Comment:
```suggestion
While cluster configuration files for Ignite 2 are XML beans, Ignite 3 uses the HOCON format. Moreover, many configuration structures in version 2 are different from those used in version 3.
```
--
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: notifications-unsubscribe@ignite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] IGNITE-26824 Add migration tools doc [ignite-3]
Posted by "zstan (via GitHub)" <gi...@apache.org>.
zstan commented on code in PR #6849:
URL: https://github.com/apache/ignite-3/pull/6849#discussion_r2472186498
##########
docs/_docs/administrators-guide/metrics/system-views.adoc:
##########
@@ -94,16 +181,27 @@ A node system view that lists the currently active locks.
[width="100%", cols="15%a, 15%a, 60%a",opts="header"]
|=======
|Column |Data Type| Description
+
| OWNING_NODE_ID | STRING | The ID of the node that owns the lock.
-| TX_ID | STRING | The ID of the transaction that created the lock.
+| TRANSACTION_ID | STRING | The ID of the transaction that created the lock.
| OBJECT_ID | STRING | The ID of the locked object.
-| MODE | STRING a| The link:https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=211885498#IEP91:Transactionprotocol-Lockingmodel[lock mode]. Possible values are:
+| LOCK_MODE | STRING a| The link:https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=211885498#IEP91:Transactionprotocol-Lockingmodel[lock mode]. Possible values are:
+
+* IS - intention shared lock
+* S- shared lock
+* IX - intention exclusive lock
+* SIX - shared intention exclusive lock
+* X - exclusive lock
+
+| TX_ID | STRING | *Deprecated*. The ID of the transaction that created the lock.
+| MODE | STRING a| *Deprecated*. The link:https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=211885498#IEP91:Transactionprotocol-Lockingmodel[lock mode]. Possible values are:
* IS - intention shared lock
-* S - shared lock
+* S- shared lock
Review Comment:
need space ?
--
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: notifications-unsubscribe@ignite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] IGNITE-26824 Add migration tools doc [ignite-3]
Posted by "zstan (via GitHub)" <gi...@apache.org>.
zstan merged PR #6849:
URL: https://github.com/apache/ignite-3/pull/6849
--
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: notifications-unsubscribe@ignite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] IGNITE-26824 Add migration tools doc [ignite-3]
Posted by "IgGusev (via GitHub)" <gi...@apache.org>.
IgGusev commented on code in PR #6849:
URL: https://github.com/apache/ignite-3/pull/6849#discussion_r2472274195
##########
docs/_docs/administrators-guide/metrics/system-views.adoc:
##########
@@ -94,16 +181,27 @@ A node system view that lists the currently active locks.
[width="100%", cols="15%a, 15%a, 60%a",opts="header"]
|=======
|Column |Data Type| Description
+
| OWNING_NODE_ID | STRING | The ID of the node that owns the lock.
-| TX_ID | STRING | The ID of the transaction that created the lock.
+| TRANSACTION_ID | STRING | The ID of the transaction that created the lock.
| OBJECT_ID | STRING | The ID of the locked object.
-| MODE | STRING a| The link:https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=211885498#IEP91:Transactionprotocol-Lockingmodel[lock mode]. Possible values are:
+| LOCK_MODE | STRING a| The link:https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=211885498#IEP91:Transactionprotocol-Lockingmodel[lock mode]. Possible values are:
+
+* IS - intention shared lock
+* S- shared lock
+* IX - intention exclusive lock
+* SIX - shared intention exclusive lock
+* X - exclusive lock
+
+| TX_ID | STRING | *Deprecated*. The ID of the transaction that created the lock.
+| MODE | STRING a| *Deprecated*. The link:https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=211885498#IEP91:Transactionprotocol-Lockingmodel[lock mode]. Possible values are:
* IS - intention shared lock
-* S - shared lock
+* S- shared lock
Review Comment:
Yes, thank you. Added,
--
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: notifications-unsubscribe@ignite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] IGNITE-26824 Add migration tools doc [ignite-3]
Posted by "IgGusev (via GitHub)" <gi...@apache.org>.
IgGusev commented on code in PR #6849:
URL: https://github.com/apache/ignite-3/pull/6849#discussion_r2465974449
##########
docs/_docs/administrators-guide/config/cli-config.adoc:
##########
@@ -37,21 +37,21 @@ ignite.auth.basic.password=
|======
|Property|Default|Description
-|ignite.jdbc.key-store.path||Path to the JDBC keystore file.
-|ignite.cluster-endpoint-url|http://127.0.1.1:10300|
+|ignite.jdbc.key-store.path||Path to the JDBC keystore file for SSL/TLS client authentication.
+|ignite.cluster-endpoint-url|http://127.0.1.1:10300|URL endpoint for connecting to the Ignite cluster REST API.
|ignite.jdbc.client-auth||If JDBC client authorization is enabled in CLI.
-|ignite.rest.key-store.password||
-|ignite.jdbc.key-store.password||
+|ignite.rest.key-store.password||Password for the REST API keystore file.
+|ignite.jdbc.key-store.password||Password for the JDBC keystore file.
|ignite.cli.sql.multiline|true|Enables multiline input mode for SQL commands in the CLI.
|ignite.cli.syntax-highlighting|true|Enables syntax highlighting in CLI output.
-|ignite.rest.trust-store.path||
-|ignite.jdbc.trust-store.password||
-|ignite.auth.basic.username||
-|ignite.jdbc-url|jdbc:ignite:thin://127.0.0.1:10800|
-|ignite.rest.key-store.path||
-|ignite.rest.trust-store.password||
-|ignite.jdbc.trust-store.path||
-|ignite.auth.basic.password||
+|ignite.rest.trust-store.path||Path to the REST API truststore file for SSL/TLS server verification.
+|ignite.jdbc.trust-store.password||Password for the JDBC truststore file.
+|ignite.auth.basic.username||Username for basic authentication when connecting to the cluster.
+|ignite.jdbc-url|jdbc:ignite:thin://127.0.0.1:10800|JDBC connection URL for connecting to the Ignite cluster.
+|ignite.rest.key-store.path||Path to the REST API keystore file for SSL/TLS client authentication.
+|ignite.rest.trust-store.password||Password for the REST API truststore file.
+|ignite.jdbc.trust-store.path||Path to the JDBC truststore file for SSL/TLS server verification.
+|ignite.auth.basic.password||Password for basic authentication when connecting to the cluster.
Review Comment:
This is the order users will get when running the cli config show command, so we just follow it in doc.
--
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: notifications-unsubscribe@ignite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org