From 1d1a7d34709b6a89985a610f46dd1c97d6c9271a Mon Sep 17 00:00:00 2001 From: Mayank Shrivastava Date: Fri, 14 Jan 2022 12:39:57 -0800 Subject: [PATCH] Cleanup: Remove remaining occurrences of `incubator`. (#8023) Removed occurrences of `incubator` from the code base. - Was mostly used in GH links to Pinot landing page, or issues, etc. - This links were still working today as `incubator-pinot` gets redirected to `pinot`. --- .../apache/pinot/broker/broker/BrokerAdminApiApplication.java | 2 +- .../common/segment/generation/SegmentGenerationUtilsTest.java | 2 +- .../pinot/controller/api/ControllerAdminApiApplication.java | 2 +- .../controller/helix/core/PinotHelixResourceManager.java | 2 +- .../org/apache/pinot/minion/MinionAdminApiApplication.java | 2 +- .../local/indexsegment/mutable/MutableSegmentImpl.java | 4 ++-- .../pinot/server/starter/helix/AdminApiApplication.java | 2 +- .../main/java/org/apache/pinot/spi/data/TimeFieldSpec.java | 2 +- .../tools/service/PinotServiceManagerAdminApiApplication.java | 2 +- 9 files changed, 10 insertions(+), 10 deletions(-) diff --git a/pinot-broker/src/main/java/org/apache/pinot/broker/broker/BrokerAdminApiApplication.java b/pinot-broker/src/main/java/org/apache/pinot/broker/broker/BrokerAdminApiApplication.java index 7b74592efd5f..9480acd41488 100644 --- a/pinot-broker/src/main/java/org/apache/pinot/broker/broker/BrokerAdminApiApplication.java +++ b/pinot-broker/src/main/java/org/apache/pinot/broker/broker/BrokerAdminApiApplication.java @@ -82,7 +82,7 @@ private void setupSwagger() { BeanConfig beanConfig = new BeanConfig(); beanConfig.setTitle("Pinot Broker API"); beanConfig.setDescription("APIs for accessing Pinot broker information"); - beanConfig.setContact("https://github.com/apache/incubator-pinot"); + beanConfig.setContact("https://github.com/apache/pinot"); beanConfig.setVersion("1.0"); beanConfig.setSchemes(new String[]{CommonConstants.HTTP_PROTOCOL, CommonConstants.HTTPS_PROTOCOL}); beanConfig.setBasePath("/"); diff --git a/pinot-common/src/test/java/org/apache/pinot/common/segment/generation/SegmentGenerationUtilsTest.java b/pinot-common/src/test/java/org/apache/pinot/common/segment/generation/SegmentGenerationUtilsTest.java index 9229021c6366..525c136ca2c5 100644 --- a/pinot-common/src/test/java/org/apache/pinot/common/segment/generation/SegmentGenerationUtilsTest.java +++ b/pinot-common/src/test/java/org/apache/pinot/common/segment/generation/SegmentGenerationUtilsTest.java @@ -78,7 +78,7 @@ public void testInvalidRelativeURIs() } // Don't lose authority portion of inputDirURI when creating output files - // https://github.com/apache/incubator-pinot/issues/6355 + // https://github.com/apache/pinot/issues/6355 @Test public void testGetFileURI() diff --git a/pinot-controller/src/main/java/org/apache/pinot/controller/api/ControllerAdminApiApplication.java b/pinot-controller/src/main/java/org/apache/pinot/controller/api/ControllerAdminApiApplication.java index def0b2a717f5..8b2e69e49d4b 100644 --- a/pinot-controller/src/main/java/org/apache/pinot/controller/api/ControllerAdminApiApplication.java +++ b/pinot-controller/src/main/java/org/apache/pinot/controller/api/ControllerAdminApiApplication.java @@ -102,7 +102,7 @@ private void setupSwagger(HttpServer httpServer) { BeanConfig beanConfig = new BeanConfig(); beanConfig.setTitle("Pinot Controller API"); beanConfig.setDescription("APIs for accessing Pinot Controller information"); - beanConfig.setContact("https://github.com/apache/incubator-pinot"); + beanConfig.setContact("https://github.com/apache/pinot"); beanConfig.setVersion("1.0"); beanConfig.setSchemes(new String[]{CommonConstants.HTTP_PROTOCOL, CommonConstants.HTTPS_PROTOCOL}); beanConfig.setBasePath("/"); diff --git a/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java b/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java index 13a1fc36a5a1..ad2919296da9 100644 --- a/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java +++ b/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java @@ -2594,7 +2594,7 @@ private PinotResourceManagerResponse enableInstance(String instanceName, boolean // TODO: Handle the case when realtime segments are in OFFLINE state because there're some problem with // realtime segment consumption, // and realtime segment will mark itself as OFFLINE in ideal state. - // Issue: https://github.com/apache/incubator-pinot/issues/4653 + // Issue: https://github.com/apache/pinot/issues/4653 if ((enableInstance && !offlineState.equals(state)) || (!enableInstance && offlineState.equals(state))) { toggleSucceeded = false; break; diff --git a/pinot-minion/src/main/java/org/apache/pinot/minion/MinionAdminApiApplication.java b/pinot-minion/src/main/java/org/apache/pinot/minion/MinionAdminApiApplication.java index 7d439f6a71e5..779011adaa31 100644 --- a/pinot-minion/src/main/java/org/apache/pinot/minion/MinionAdminApiApplication.java +++ b/pinot-minion/src/main/java/org/apache/pinot/minion/MinionAdminApiApplication.java @@ -80,7 +80,7 @@ private void setupSwagger() { BeanConfig beanConfig = new BeanConfig(); beanConfig.setTitle("Pinot Minion API"); beanConfig.setDescription("APIs for accessing Pinot Minion information"); - beanConfig.setContact("https://github.com/apache/incubator-pinot"); + beanConfig.setContact("https://github.com/apache/pinot"); beanConfig.setVersion("1.0"); beanConfig.setSchemes(new String[]{CommonConstants.HTTP_PROTOCOL, CommonConstants.HTTPS_PROTOCOL}); beanConfig.setBasePath("/"); diff --git a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/indexsegment/mutable/MutableSegmentImpl.java b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/indexsegment/mutable/MutableSegmentImpl.java index 52474628bbdc..47479a8b1c1a 100644 --- a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/indexsegment/mutable/MutableSegmentImpl.java +++ b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/indexsegment/mutable/MutableSegmentImpl.java @@ -956,7 +956,7 @@ private int getOrCreateDocId() { int i = 0; int[] dictIds = new int[_numKeyColumns]; // dimensions + date time columns + time column. - // FIXME: this for loop breaks for multi value dimensions. https://github.com/apache/incubator-pinot/issues/3867 + // FIXME: this for loop breaks for multi value dimensions. https://github.com/apache/pinot/issues/3867 for (FieldSpec fieldSpec : _physicalDimensionFieldSpecs) { dictIds[i++] = _indexContainerMap.get(fieldSpec.getName())._dictId; } @@ -973,7 +973,7 @@ private int getOrCreateDocId() { *
  • All dimensions and time are dictionary encoded. This is because an integer array containing dictionary id's * is used as key for dimensions to record Id map.
    • *
  • None of the metrics are dictionary encoded.
    • - *
  • All columns should be single-valued (see https://github.com/apache/incubator-pinot/issues/3867)
    • + *
  • All columns should be single-valued (see https://github.com/apache/pinot/issues/3867)
    • * * * TODO: Eliminate the requirement on dictionary encoding for dimension and metric columns. diff --git a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/AdminApiApplication.java b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/AdminApiApplication.java index dbe06e0e3ece..a45ba47940d7 100644 --- a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/AdminApiApplication.java +++ b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/AdminApiApplication.java @@ -115,7 +115,7 @@ private void setupSwagger(HttpServer httpServer) { BeanConfig beanConfig = new BeanConfig(); beanConfig.setTitle("Pinot Server API"); beanConfig.setDescription("APIs for accessing Pinot server information"); - beanConfig.setContact("https://github.com/apache/incubator-pinot"); + beanConfig.setContact("https://github.com/apache/pinot"); beanConfig.setVersion("1.0"); beanConfig.setSchemes(new String[]{CommonConstants.HTTP_PROTOCOL, CommonConstants.HTTPS_PROTOCOL}); beanConfig.setBasePath("/"); diff --git a/pinot-spi/src/main/java/org/apache/pinot/spi/data/TimeFieldSpec.java b/pinot-spi/src/main/java/org/apache/pinot/spi/data/TimeFieldSpec.java index 7880c5af7ba6..c7ec035784c3 100644 --- a/pinot-spi/src/main/java/org/apache/pinot/spi/data/TimeFieldSpec.java +++ b/pinot-spi/src/main/java/org/apache/pinot/spi/data/TimeFieldSpec.java @@ -30,7 +30,7 @@ * @deprecated Use {@link DateTimeFieldSpec} instead. * This should only be used in 1) tests 2) wherever required for backward compatible handling of schemas with * TimeFieldSpec - * https://github.com/apache/incubator-pinot/issues/2756 + * https://github.com/apache/pinot/issues/2756 */ @JsonIgnoreProperties(ignoreUnknown = true) @SuppressWarnings("unused") diff --git a/pinot-tools/src/main/java/org/apache/pinot/tools/service/PinotServiceManagerAdminApiApplication.java b/pinot-tools/src/main/java/org/apache/pinot/tools/service/PinotServiceManagerAdminApiApplication.java index 75684f33e682..c765187754a9 100644 --- a/pinot-tools/src/main/java/org/apache/pinot/tools/service/PinotServiceManagerAdminApiApplication.java +++ b/pinot-tools/src/main/java/org/apache/pinot/tools/service/PinotServiceManagerAdminApiApplication.java @@ -67,7 +67,7 @@ private void setupSwagger() { BeanConfig beanConfig = new BeanConfig(); beanConfig.setTitle("Pinot Starter API"); beanConfig.setDescription("APIs for accessing Pinot Starter information"); - beanConfig.setContact("https://github.com/apache/incubator-pinot"); + beanConfig.setContact("https://github.com/apache/pinot"); beanConfig.setVersion("1.0"); beanConfig.setSchemes(new String[]{CommonConstants.HTTP_PROTOCOL, CommonConstants.HTTPS_PROTOCOL}); beanConfig.setBasePath(_baseUri.getPath());