-
Notifications
You must be signed in to change notification settings - Fork 90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
IGNITE-22108 Rename options in CLI #3750
Conversation
37047c4
to
7982dd9
Compare
…-22108 # Conflicts: # modules/cli/src/integrationTest/java/org/apache/ignite/internal/cli/commands/ItClusterCommandTest.java # modules/cli/src/integrationTest/java/org/apache/ignite/internal/cli/commands/cluster/init/ItClusterInitTest.java # modules/cli/src/integrationTest/java/org/apache/ignite/internal/cli/commands/configuration/ItConfigCommandTest.java # modules/cli/src/integrationTest/java/org/apache/ignite/internal/cli/commands/unit/ItDeploymentUnitTest.java # modules/cli/src/test/java/org/apache/ignite/internal/cli/IgniteCliInterfaceTest.java
Some options were reviewed. For example, cluster-name -> name. Also, short options that clash with each other are dropped.
|
||
/** Meta storage node name option short name. */ | ||
public static final String META_STORAGE_NODE_NAME_OPTION_SHORT = "-m"; | ||
public static final String META_STORAGE_NODE_NAME_OPTION = "--ms-node"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this abbreviation is something user is expecting, maybe we could keep the full option name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do you think if we keep both --ms-node
and --meta-storage-node
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do you think if we keep both
--ms-node
and--meta-storage-node
?
I think it could be confusing for users
...in/java/org/apache/ignite/internal/cli/commands/cluster/unit/ClusterUnitUndeployCommand.java
Outdated
Show resolved
Hide resolved
modules/cli/src/main/java/org/apache/ignite/internal/cli/commands/sql/SqlReplCommand.java
Outdated
Show resolved
Hide resolved
modules/cli/src/main/java/org/apache/ignite/internal/cli/commands/sql/SqlReplCommand.java
Outdated
Show resolved
Hide resolved
...pache/ignite/internal/cli/core/repl/completer/cluster/ClusterUrlDynamicCompleterFactory.java
Outdated
Show resolved
Hide resolved
modules/cli/src/test/java/org/apache/ignite/internal/cli/commands/UrlOptionsNegativeTest.java
Outdated
Show resolved
Hide resolved
…nds/cluster/unit/ClusterUnitUndeployCommand.java Co-authored-by: Vadim Pakhnushev <8614891+valepakh@users.noreply.github.com>
…nds/sql/SqlReplCommand.java Co-authored-by: Vadim Pakhnushev <8614891+valepakh@users.noreply.github.com>
…repl/completer/cluster/ClusterUrlDynamicCompleterFactory.java Co-authored-by: Vadim Pakhnushev <8614891+valepakh@users.noreply.github.com>
Some options were reviewed. For example, cluster-name
-> name. Also, short options that clash with each other
are dropped.
https://issues.apache.org/jira/browse/IGNITE-22108