Add a message on definecolumn that notifies the user when the column has already been defined.
Project: http://git-wip-us.apache.org/repos/asf/incubator-blur/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-blur/commit/13a2f791 Tree: http://git-wip-us.apache.org/repos/asf/incubator-blur/tree/13a2f791 Diff: http://git-wip-us.apache.org/repos/asf/incubator-blur/diff/13a2f791 Branch: refs/heads/blur-console-v2 Commit: 13a2f7914ada187c541d65b1a50890a56e2c913a Parents: 2fb5776 Author: Aaron McCurry <[email protected]> Authored: Thu Oct 31 12:59:13 2013 -0400 Committer: Aaron McCurry <[email protected]> Committed: Thu Oct 31 13:35:11 2013 -0400 ---------------------------------------------------------------------- .../java/org/apache/blur/shell/AddColumnDefinitionCommand.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-blur/blob/13a2f791/blur-shell/src/main/java/org/apache/blur/shell/AddColumnDefinitionCommand.java ---------------------------------------------------------------------- diff --git a/blur-shell/src/main/java/org/apache/blur/shell/AddColumnDefinitionCommand.java b/blur-shell/src/main/java/org/apache/blur/shell/AddColumnDefinitionCommand.java index d2d1c4f..4bac392 100644 --- a/blur-shell/src/main/java/org/apache/blur/shell/AddColumnDefinitionCommand.java +++ b/blur-shell/src/main/java/org/apache/blur/shell/AddColumnDefinitionCommand.java @@ -62,7 +62,9 @@ public class AddColumnDefinitionCommand extends Command implements TableFirstArg } } } - client.addColumnDefinition(args[1], columnDefinition); + if (!client.addColumnDefinition(args[1], columnDefinition)) { + out.println("Column Definition was not added, check to see if the column has already been added to the table."); + } } @Override @@ -79,7 +81,7 @@ public class AddColumnDefinitionCommand extends Command implements TableFirstArg public String name() { return "definecolumn"; } - + @SuppressWarnings("static-access") private static CommandLine parse(String[] otherArgs, Writer out) { Options options = new Options();
