dmagda commented on a change in pull request #8630:
URL: https://github.com/apache/ignite/pull/8630#discussion_r553572548
##########
File path: docs/_docs/starting-nodes.adoc
##########
@@ -213,6 +213,28 @@ You can also pass JVM options by using the `-J` prefix:
----
./ignite.sh -J-Xmx6G -J-DIGNITE_TO_STRING_INCLUDE_SENSITIVE=false
----
+
+== Ignite Internal Settings
+Ignite provide more then hundred internal flags to tweak som apects of the
node behavior.
Review comment:
Please use this desctipion:
_In addition to public configuration settings, you can adjust specific,
usually low-level, Ignite behavior with internal system properties. You can
find all the properties with their descriptions and default values by using the
command below:_
##########
File path: docs/_docs/starting-nodes.adoc
##########
@@ -213,6 +213,28 @@ You can also pass JVM options by using the `-J` prefix:
----
./ignite.sh -J-Xmx6G -J-DIGNITE_TO_STRING_INCLUDE_SENSITIVE=false
----
+
+== Ignite Internal Settings
Review comment:
Could you please do the following?
- Rename this section to "== Setting Ignite System Properties" and move to
the following page (https://ignite.apache.org/docs/latest/setup). Put it after
the "Enabling Modules" section.
- Please also move the entire "== Setting JVM Options" section to the same
setup page mentioned above.
##########
File path: docs/_docs/starting-nodes.adoc
##########
@@ -213,6 +213,28 @@ You can also pass JVM options by using the `-J` prefix:
----
./ignite.sh -J-Xmx6G -J-DIGNITE_TO_STRING_INCLUDE_SENSITIVE=false
----
+
+== Ignite Internal Settings
+Ignite provide more then hundred internal flags to tweak som apects of the
node behavior.
+You can see all flags, their descriptions and default values with the
following command:
+
+[source, shell]
+----
+./ignite.sh -systemProps
Review comment:
Please add a tab for Windows.
##########
File path: docs/_docs/tools/control-script.adoc
##########
@@ -646,4 +646,136 @@ control.bat --change-tag _<new-tag>_
----
--
+== Metric command
+Metrics command prints value(s) of the metric or metric registry from the
parameter.
+
+If you need to get metric from the specific node, please, set the `--node-id`
parameter.
+Without `--node-id` random node will be used.
+
+[tabs]
+--
+tab:Linux[]
+[source,shell,subs="verbatim,quotes"]
+----
+control.sh --metric sys
+----
+tab:Windows[]
+[source,shell,subs="verbatim,quotes"]
+----
+control.bat --metric sys
+----
+--
+
+Example of the metric output:
+[source, text]
+$ ./bin/control.sh --metric sysCurrentThreadCpuTime
Review comment:
Remove the command row from the output here and similar places below.
Let's follow the same format you (we) have already used for other control
script commands documented earlier.
##########
File path: docs/_docs/tools/control-script.adoc
##########
@@ -646,4 +646,136 @@ control.bat --change-tag _<new-tag>_
----
--
+== Metric command
+Metrics command prints value(s) of the metric or metric registry from the
parameter.
+
+If you need to get metric from the specific node, please, set the `--node-id`
parameter.
+Without `--node-id` random node will be used.
+
+[tabs]
+--
+tab:Linux[]
+[source,shell,subs="verbatim,quotes"]
+----
+control.sh --metric sys
+----
+tab:Windows[]
+[source,shell,subs="verbatim,quotes"]
+----
+control.bat --metric sys
+----
+--
+
+Example of the metric output:
+[source, text]
+$ ./bin/control.sh --metric sysCurrentThreadCpuTime
+Command [METRIC] started
+Arguments: --metric sys
+--------------------------------------------------------------------------------
+metric value
+sys.CurrentThreadCpuTime 17270000
+Command [METRIC] finished with code: 0
+
+
+Example of the metric registry output:
+[source, text]
+$ ./bin/control.sh --metric io.dataregion.default
+Command [METRIC] started
+Arguments: --metric sys
+--------------------------------------------------------------------------------
+metric value
+io.dataregion.default.TotalAllocatedSize 0
+io.dataregion.default.LargeEntriesPagesCount 0
+io.dataregion.default.PagesReplaced 0
+io.dataregion.default.PhysicalMemorySize 0
+io.dataregion.default.CheckpointBufferSize 0
+io.dataregion.default.PagesReplaceRate 0
+io.dataregion.default.InitialSize 268435456
+io.dataregion.default.PagesRead 0
+io.dataregion.default.AllocationRate 0
+io.dataregion.default.OffHeapSize 0
+io.dataregion.default.UsedCheckpointBufferSize 0
+io.dataregion.default.MaxSize 6871947673
+io.dataregion.default.OffheapUsedSize 0
+io.dataregion.default.EmptyDataPages 0
+io.dataregion.default.PagesFillFactor 0.0
+io.dataregion.default.DirtyPages 0
+io.dataregion.default.TotalThrottlingTime 0
+io.dataregion.default.EvictionRate 0
+io.dataregion.default.PagesWritten 0
+io.dataregion.default.TotalAllocatedPages 0
+io.dataregion.default.PagesReplaceAge 0
+io.dataregion.default.PhysicalMemoryPages 0
+Command [METRIC] finished with code: 0
+
+== System view command
Review comment:
"== System View Command"
All the first letters must be written in the upper-case.
##########
File path: docs/_docs/starting-nodes.adoc
##########
@@ -213,6 +213,28 @@ You can also pass JVM options by using the `-J` prefix:
----
./ignite.sh -J-Xmx6G -J-DIGNITE_TO_STRING_INCLUDE_SENSITIVE=false
----
+
+== Ignite Internal Settings
+Ignite provide more then hundred internal flags to tweak som apects of the
node behavior.
+You can see all flags, their descriptions and default values with the
following command:
+
+[source, shell]
+----
+./ignite.sh -systemProps
+----
+
+Example of the output:
+
+[source, text]
+$ ./bin/ignite.sh -systemProps
+IGNITE_AFFINITY_HISTORY_SIZE - [Integer]
Maximum size for affinity assignment history. Default is 25.
Review comment:
Please put digits, constant names or bool values into `value` brackets
so that it's highlighted with a special style. For instance, in this row you
should change 25 to `25`. Do for all similar places.
##########
File path: docs/_docs/starting-nodes.adoc
##########
@@ -213,6 +213,28 @@ You can also pass JVM options by using the `-J` prefix:
----
./ignite.sh -J-Xmx6G -J-DIGNITE_TO_STRING_INCLUDE_SENSITIVE=false
----
+
+== Ignite Internal Settings
+Ignite provide more then hundred internal flags to tweak som apects of the
node behavior.
+You can see all flags, their descriptions and default values with the
following command:
+
+[source, shell]
+----
+./ignite.sh -systemProps
+----
+
+Example of the output:
+
+[source, text]
+$ ./bin/ignite.sh -systemProps
+IGNITE_AFFINITY_HISTORY_SIZE - [Integer]
Maximum size for affinity assignment history. Default is 25.
+IGNITE_ALLOW_ATOMIC_OPS_IN_TX - [Boolean]
Allows atomic operations inside transactions. Default is true.
+IGNITE_ALLOW_DML_INSIDE_TRANSACTION - [Boolean]
When set to true, Ignite will allow execute DML operation
(MERGE|INSERT|UPDATE|DELETE) within transaction for non MVCC mode. Default is
false.
Review comment:
Please update with
When set to `true`, Ignite will allow executing DML operation
(`MERGE`|`INSERT`|`UPDATE`|`DELETE`) within transactions for the non-MVCC mode.
Default is `false`.
##########
File path: docs/_docs/tools/control-script.adoc
##########
@@ -646,4 +646,136 @@ control.bat --change-tag _<new-tag>_
----
--
+== Metric command
Review comment:
"Metric**s** Command"
##########
File path: docs/_docs/tools/control-script.adoc
##########
@@ -646,4 +646,136 @@ control.bat --change-tag _<new-tag>_
----
--
+== Metric command
+Metrics command prints value(s) of the metric or metric registry from the
parameter.
+
+If you need to get metric from the specific node, please, set the `--node-id`
parameter.
+Without `--node-id` random node will be used.
+
+[tabs]
+--
+tab:Linux[]
Review comment:
Could you do me a favor by replacing "Linux" with "Unix" for all the
tabs on this page? That's my oversight. We need to use "Unix" since the
commands are relevant for MacOS, Linux and other Unix-distributions.
##########
File path: docs/_docs/starting-nodes.adoc
##########
@@ -213,6 +213,28 @@ You can also pass JVM options by using the `-J` prefix:
----
./ignite.sh -J-Xmx6G -J-DIGNITE_TO_STRING_INCLUDE_SENSITIVE=false
----
+
+== Ignite Internal Settings
+Ignite provide more then hundred internal flags to tweak som apects of the
node behavior.
+You can see all flags, their descriptions and default values with the
following command:
+
+[source, shell]
+----
+./ignite.sh -systemProps
+----
+
+Example of the output:
+
+[source, text]
+$ ./bin/ignite.sh -systemProps
+IGNITE_AFFINITY_HISTORY_SIZE - [Integer]
Maximum size for affinity assignment history. Default is 25.
+IGNITE_ALLOW_ATOMIC_OPS_IN_TX - [Boolean]
Allows atomic operations inside transactions. Default is true.
+IGNITE_ALLOW_DML_INSIDE_TRANSACTION - [Boolean]
When set to true, Ignite will allow execute DML operation
(MERGE|INSERT|UPDATE|DELETE) within transaction for non MVCC mode. Default is
false.
+IGNITE_ALLOW_START_CACHES_IN_PARALLEL - [Boolean]
Enables start caches in parallel. Default is true.
Review comment:
Say "Allows to start multiple caches in parallel. Default is `true`."
##########
File path: docs/_docs/tools/control-script.adoc
##########
@@ -646,4 +646,136 @@ control.bat --change-tag _<new-tag>_
----
--
+== Metric command
+Metrics command prints value(s) of the metric or metric registry from the
parameter.
+
+If you need to get metric from the specific node, please, set the `--node-id`
parameter.
+Without `--node-id` random node will be used.
+
+[tabs]
+--
+tab:Linux[]
+[source,shell,subs="verbatim,quotes"]
+----
+control.sh --metric sys
+----
+tab:Windows[]
+[source,shell,subs="verbatim,quotes"]
+----
+control.bat --metric sys
+----
+--
+
+Example of the metric output:
+[source, text]
+$ ./bin/control.sh --metric sysCurrentThreadCpuTime
+Command [METRIC] started
+Arguments: --metric sys
+--------------------------------------------------------------------------------
+metric value
+sys.CurrentThreadCpuTime 17270000
+Command [METRIC] finished with code: 0
+
+
+Example of the metric registry output:
+[source, text]
+$ ./bin/control.sh --metric io.dataregion.default
+Command [METRIC] started
+Arguments: --metric sys
+--------------------------------------------------------------------------------
+metric value
+io.dataregion.default.TotalAllocatedSize 0
+io.dataregion.default.LargeEntriesPagesCount 0
+io.dataregion.default.PagesReplaced 0
+io.dataregion.default.PhysicalMemorySize 0
+io.dataregion.default.CheckpointBufferSize 0
+io.dataregion.default.PagesReplaceRate 0
+io.dataregion.default.InitialSize 268435456
+io.dataregion.default.PagesRead 0
+io.dataregion.default.AllocationRate 0
+io.dataregion.default.OffHeapSize 0
+io.dataregion.default.UsedCheckpointBufferSize 0
+io.dataregion.default.MaxSize 6871947673
+io.dataregion.default.OffheapUsedSize 0
+io.dataregion.default.EmptyDataPages 0
+io.dataregion.default.PagesFillFactor 0.0
+io.dataregion.default.DirtyPages 0
+io.dataregion.default.TotalThrottlingTime 0
+io.dataregion.default.EvictionRate 0
+io.dataregion.default.PagesWritten 0
+io.dataregion.default.TotalAllocatedPages 0
+io.dataregion.default.PagesReplaceAge 0
+io.dataregion.default.PhysicalMemoryPages 0
+Command [METRIC] finished with code: 0
+
+== System view command
+
+System view command prints content of the system view from the parameter.
Review comment:
The system view command prints out the content of a system view provided
in the parameters list. Use the `--node-id` parameter, If you need to get a
metric from a specific node. Ignite selects a random node, if the `-node-id` is
not set.
##########
File path: docs/_docs/tools/control-script.adoc
##########
@@ -646,4 +646,136 @@ control.bat --change-tag _<new-tag>_
----
--
+== Metric command
+Metrics command prints value(s) of the metric or metric registry from the
parameter.
Review comment:
The metrics command prints out the value of a metric or metric registry
provided in the parameters list. Use the `--node-id` parameter, If you need to
get a metric from a specific node. Ignite selects a random node, if the
`--node-id` is not set.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]