Skip to content

Commit 092f58a

Browse files
authored
[Minor] improvement to Partition Gravitino CLI code (#5617)
### What changes were proposed in this pull request? Minor improvements to Partition Gravitino CLI code. ### Why are the changes needed? Remove duplicate short command code and make the same style as other options. Fix: # N/A ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Tested locally
1 parent d8a6ff8 commit 092f58a

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -356,7 +356,7 @@ private void handleTableCommand() {
356356
new ListIndexes(url, ignore, metalake, catalog, schema, table).handle();
357357
} else if (line.hasOption(GravitinoOptions.DISTRIBUTION)) {
358358
new TableDistribution(url, ignore, metalake, catalog, schema, table).handle();
359-
} else if (line.hasOption(GravitinoOptions.Partition)) {
359+
} else if (line.hasOption(GravitinoOptions.PARTITION)) {
360360
new TablePartition(url, ignore, metalake, catalog, schema, table).handle();
361361
} else {
362362
new TableDetails(url, ignore, metalake, catalog, schema, table).handle();

clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoOptions.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ public class GravitinoOptions {
4545
public static final String INDEX = "index";
4646
public static final String FORCE = "force";
4747
public static final String DISTRIBUTION = "distribution";
48-
public static final String Partition = "partition";
48+
public static final String PARTITION = "partition";
4949

5050
/**
5151
* Builds and returns the CLI options for Gravitino.
@@ -66,7 +66,7 @@ public Options options() {
6666
options.addOption(createSimpleOption("a", AUDIT, "display audit information"));
6767
options.addOption(createSimpleOption("x", INDEX, "Display index infromation"));
6868
options.addOption(createSimpleOption("d", DISTRIBUTION, "Display distribution information"));
69-
options.addOption(createSimpleOption("p", Partition, "Display partition information"));
69+
options.addOption(createSimpleOption(null, PARTITION, "Display partition information"));
7070

7171
// Create/update options
7272
options.addOption(createArgOption(null, RENAME, "new entity name"));

0 commit comments

Comments
 (0)