Repository: sqoop Updated Branches: refs/heads/sqoop2 8e08a7bdc -> e24263a2e
SQOOP-2693: Sqoop2: Command line should let user retry Name it's invalid (Dian Fu via Jarek Jarcec Cecho) Project: http://git-wip-us.apache.org/repos/asf/sqoop/repo Commit: http://git-wip-us.apache.org/repos/asf/sqoop/commit/e24263a2 Tree: http://git-wip-us.apache.org/repos/asf/sqoop/tree/e24263a2 Diff: http://git-wip-us.apache.org/repos/asf/sqoop/diff/e24263a2 Branch: refs/heads/sqoop2 Commit: e24263a2e29e562067d3ae64fd56bdc3272b7733 Parents: 8e08a7b Author: Jarek Jarcec Cecho <[email protected]> Authored: Tue Nov 17 08:20:51 2015 -0800 Committer: Jarek Jarcec Cecho <[email protected]> Committed: Tue Nov 17 08:20:51 2015 -0800 ---------------------------------------------------------------------- .../main/java/org/apache/sqoop/shell/utils/ConfigFiller.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/sqoop/blob/e24263a2/shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java ---------------------------------------------------------------------- diff --git a/shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java b/shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java index 3a4a18d..86d8230 100644 --- a/shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java +++ b/shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java @@ -1118,9 +1118,10 @@ public final class ConfigFiller { fillInputStringWithBundle(nameInput, reader, getResourceBundle()); if (StringUtils.isEmpty(nameInput.getValue())) { errorMessage(nameInput, "Job name or link name cannot be null"); - continue; + } else { + break; } - } while (false); + } while (true); return nameInput.getValue(); }
