[incubator-pinot] branch master updated: Zookeeper override fix for regression from config refactoring (#5662)

[ja...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new a40b1de  Zookeeper override fix for regression from config refactoring (#5662)
a40b1de is described below

commit a40b1de353e4926dd98bfa02bfa5285f1b894bdd
Author: Daniel Lavoie <dl...@live.ca>
AuthorDate: Tue Jul 7 15:12:23 2020 -0400

    Zookeeper override fix for regression from config refactoring (#5662)
    
    Replaced leftover UnsupportedOperationException with setProperty call.
---
 .../src/main/java/org/apache/pinot/controller/ControllerConf.java      | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java b/pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java
index 3f1cd79..edfec43 100644
--- a/pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java
+++ b/pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java
@@ -252,8 +252,7 @@ public class ControllerConf extends PinotConfiguration {
   }
 
   public void setZkStr(String zkStr) {
-    throw new UnsupportedOperationException("Fix me!");
-    //    setProperty(ZK_STR, zkStr);
+    setProperty(ZK_STR, zkStr);
   }
 
   // A boolean to decide whether Jersey API should be the primary one. For now, we set this to be false,


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