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

mayanks 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 535a2cb  Set `pinot.server.netty.host` from PerfBenchmarkDriver to 
avoid using NetUtils.getHostAddress(). (#5384)
535a2cb is described below

commit 535a2cb4afda0f27f630459f9beb5846ebb2aa1b
Author: Mayank Shrivastava <[email protected]>
AuthorDate: Wed May 13 20:31:02 2020 -0700

    Set `pinot.server.netty.host` from PerfBenchmarkDriver to avoid using 
NetUtils.getHostAddress(). (#5384)
    
    If `pinot.server.netty.host` is not explictly set, the HELIX_HOST in 
instance config gets
    overwritten by a value returned by `NetUtils.getHostAddress()`, which 
depending on internet/firewall
    may return an addresses that cannot be connected to. Explicitly setting 
this to `localhost` solves
    the issue.
---
 .../src/main/java/org/apache/pinot/tools/perf/PerfBenchmarkDriver.java   | 1 +
 1 file changed, 1 insertion(+)

diff --git 
a/pinot-tools/src/main/java/org/apache/pinot/tools/perf/PerfBenchmarkDriver.java
 
b/pinot-tools/src/main/java/org/apache/pinot/tools/perf/PerfBenchmarkDriver.java
index d953e86..13d6277 100644
--- 
a/pinot-tools/src/main/java/org/apache/pinot/tools/perf/PerfBenchmarkDriver.java
+++ 
b/pinot-tools/src/main/java/org/apache/pinot/tools/perf/PerfBenchmarkDriver.java
@@ -241,6 +241,7 @@ public class PerfBenchmarkDriver {
     
serverConfiguration.addProperty(CommonConstants.Server.CONFIG_OF_INSTANCE_DATA_DIR,
 _serverInstanceDataDir);
     serverConfiguration
         
.addProperty(CommonConstants.Server.CONFIG_OF_INSTANCE_SEGMENT_TAR_DIR, 
_serverInstanceSegmentTarDir);
+    
serverConfiguration.addProperty(CommonConstants.Helix.KEY_OF_SERVER_NETTY_HOST, 
"localhost");
     if (_segmentFormatVersion != null) {
       
serverConfiguration.setProperty(CommonConstants.Server.CONFIG_OF_SEGMENT_FORMAT_VERSION,
 _segmentFormatVersion);
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to