You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by xi...@apache.org on 2019/10/06 05:36:32 UTC

[incubator-pinot] 01/01: Remove validation for config controller.host

This is an automated email from the ASF dual-hosted git repository.

xiangfu pushed a commit to branch remove_config_validation
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git

commit e74ff90aff7cf0f110c91a1698ee72905e10f593
Author: Xiang Fu <fx...@gmail.com>
AuthorDate: Sat Oct 5 22:36:14 2019 -0700

    Remove validation for config controller.host
---
 .../org/apache/pinot/tools/admin/command/StartControllerCommand.java | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartControllerCommand.java b/pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartControllerCommand.java
index 45d36a2..3cdd136 100644
--- a/pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartControllerCommand.java
+++ b/pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartControllerCommand.java
@@ -197,11 +197,6 @@ public class StartControllerCommand extends AbstractBaseAdminCommand implements
       return false;
     }
 
-    if (conf.getControllerHost() == null) {
-      LOGGER.error("Error: missing hostname, please specify 'controller.host' property in config file.");
-      return false;
-    }
-
     if (conf.getControllerPort() == null) {
       LOGGER.error("Error: missing controller port, please specify 'controller.port' property in config file.");
       return false;


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org