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

[incubator-pinot] branch master updated: Change ControllerTest default prot to 18998 (#4259)

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 d540d95  Change ControllerTest default prot to 18998 (#4259)
d540d95 is described below

commit d540d95d24833216254ba2d1980955015e667c6f
Author: Xiaotian (Jackie) Jiang <17...@users.noreply.github.com>
AuthorDate: Thu May 30 17:22:05 2019 -0700

    Change ControllerTest default prot to 18998 (#4259)
    
    Port 8998 is commonly used, which might cause address already in use problem.
    Change it to 18998 to avoid this.
    Also remove redundant entries in gitignore.
---
 .gitignore                                                          | 6 ------
 .../test/java/org/apache/pinot/controller/helix/ControllerTest.java | 2 +-
 2 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/.gitignore b/.gitignore
index be407e9..78661c2e 100644
--- a/.gitignore
+++ b/.gitignore
@@ -14,12 +14,6 @@ test-output
 pinot-transport/test-output/
 pinot-broker/test-output/
 pinot-core/test-output/
-pinot-dashboard/.env
-pinot-dashboard/activate
-pinot-dashboard/config.yml
-pinot-dashboard/logs/*
-pinot-dashboard/dist
-pinot-dashboard/pinotui.egg-info
 *.log
 .doppelganger
 docs/_build
diff --git a/pinot-controller/src/test/java/org/apache/pinot/controller/helix/ControllerTest.java b/pinot-controller/src/test/java/org/apache/pinot/controller/helix/ControllerTest.java
index 20bdcfb..9893377 100644
--- a/pinot-controller/src/test/java/org/apache/pinot/controller/helix/ControllerTest.java
+++ b/pinot-controller/src/test/java/org/apache/pinot/controller/helix/ControllerTest.java
@@ -57,7 +57,7 @@ import org.testng.Assert;
 public abstract class ControllerTest {
   public static final String LOCAL_HOST = "localhost";
 
-  private static final int DEFAULT_CONTROLLER_PORT = 8998;
+  private static final int DEFAULT_CONTROLLER_PORT = 18998;
   private static final String DEFAULT_DATA_DIR =
       new File(FileUtils.getTempDirectoryPath(), "test-controller-" + System.currentTimeMillis()).getAbsolutePath();
 


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