[incubator-pinot] branch master updated: Set `pinot.server.netty.host` from PerfBenchmarkDriver to avoid using NetUtils.getHostAddress(). (#5384)

[ma...@apache.org]

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 <ms...@linkedin.com>
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: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org