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:31 UTC

[incubator-pinot] branch remove_config_validation created (now e74ff90)

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

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


      at e74ff90  Remove validation for config controller.host

This branch includes the following new commits:

     new e74ff90  Remove validation for config controller.host

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



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


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

Posted by xi...@apache.org.
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