Mmuzaf commented on a change in pull request #8207:
URL: https://github.com/apache/ignite/pull/8207#discussion_r485579491
##########
File path:
modules/core/src/main/java/org/apache/ignite/startup/cmdline/CommandLineStartup.java
##########
@@ -66,6 +86,48 @@
/** Quite log flag. */
private static final boolean QUITE;
+ /** Command to print Ignite system properties info. */
+ static final String PRINT_PROPS_COMMAND = "-systemProps";
+
+ /** Classes with Ignite system properties. */
+ static final List<Class<?>> PROPS_CLS = new ArrayList<>(Arrays.asList(
+ IgniteSystemProperties.class,
Review comment:
Should we add checks here at compile time to ensure that this list is
fully enough? Some of the developers may forgot adding new classes here. Also,
we should to mention it in javadoc of the `@SystemProperty` annotation.
----------------------------------------------------------------
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]
