You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by jl...@apache.org on 2019/07/29 20:42:03 UTC

[incubator-pinot] branch add-logic-for-lead-controller-resource updated (5ca90e9 -> b4f3c17)

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

jlli pushed a change to branch add-logic-for-lead-controller-resource
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.


    omit 5ca90e9  Address PR comments
    omit 91d28e9  Put HelixManager to constructor
    omit 0c66ab7  Address PR comments
    omit 52b7f2e  Use resource config to check whether resource is enabled
    omit 7933ac0  Address PR comments
    omit 9fecd6a  Move murmur2 functions to an util class in pinot-common
    omit 0529ed2  Move murmur2 functions to an util class in pinot-common
    omit 51232d5  Address PR comments; Remove API design out of this PR
    omit bd9a544  Address PR comments
    omit c62b4e7  Add logic for lead controller resource on controller side
    omit 835679d  Add logic for leveraging lead controller resource
     add a1fff55  Ensuring partition config is valid during Hadoop preprocess (#4463)
     add 14ef543  Remove the usage of guava deprecated methods (#4457)
     add 1642f00  [TE] logs and precondition checks for pipeline re-tuning (#4465)
     add eca1d2e  [TE] frontend - set analysis range using alertDetailsDefaultWindowSize from endpoint (#4460)
     add 1d45f87  Enable periodic rebalance as a temporary work-around for the Helix issue (#4459)
     add cac4f24  Allow customized metrics prefix in pinot controller/broker/server (#4392)
     add c01b852  Add table name parameter for segment push (#4454)
     add c81adda  [TE] Include alert description in entity email alert (#4471)
     add 29bc069  [TE] Include admin recipients in every email alert (cc address) (#4470)
     add 9d64d8a  Refactor HelixBrokerStarterTest (#4473)
     add 4f856a7  Add logic for leveraging lead controller resource
     add 6aae6a3  Add logic for lead controller resource on controller side
     add 78ba05f  Address PR comments
     add 52c3b83  Address PR comments; Remove API design out of this PR
     add e88f3b3  Move murmur2 functions to an util class in pinot-common
     add 29fe013  Move murmur2 functions to an util class in pinot-common
     add 5282f31  Address PR comments
     add be12fc6  Use resource config to check whether resource is enabled
     add f2dbadf  Address PR comments
     add 04916a3  Put HelixManager to constructor
     add b4f3c17  Address PR comments

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (5ca90e9)
            \
             N -- N -- N   refs/heads/add-logic-for-lead-controller-resource (b4f3c17)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

No new revisions were added by this update.

Summary of changes:
 .../pinot/broker/broker/BrokerServerBuilder.java   |   3 +-
 .../pinot/broker/routing/CfgBasedRouting.java      |  64 ----
 .../EmptyBrokerOnlineOfflineStateModelFactory.java |  70 ----
 ...EmptySegmentOnlineOfflineStateModelFactory.java |  71 -----
 .../broker/broker/HelixBrokerStarterTest.java      | 354 ++++++++-------------
 .../apache/pinot/common/metrics/BrokerMetrics.java |  12 +-
 .../pinot/common/metrics/ControllerMetrics.java    |   9 +-
 .../apache/pinot/common/metrics/ServerMetrics.java |  24 +-
 .../apache/pinot/common/utils/CommonConstants.java |  15 +-
 .../common/utils/FileUploadDownloadClient.java     |  43 ++-
 .../apache/pinot/controller/ControllerConf.java    |   7 +
 .../apache/pinot/controller/ControllerStarter.java |   2 +-
 .../PinotSegmentUploadRestletResource.java         |  28 +-
 .../api/PinotInstanceRestletResourceTest.java      |   4 +-
 .../pinot/controller/helix/ControllerTest.java     |  11 +
 .../test/java/org/apache/pinot/util/TestUtils.java |  64 +---
 .../pinot/hadoop/job/DefaultControllerRestApi.java |   4 +-
 .../pinot/hadoop/job/SegmentPreprocessingJob.java  |   7 +
 .../apache/pinot/hadoop/job/SegmentTarPushJob.java |   4 +-
 .../apache/pinot/hadoop/job/SegmentUriPushJob.java |   4 +-
 .../hadoop/job/mappers/SegmentCreationMapper.java  |   2 +-
 .../pinot/integration/tests/ClusterTest.java       |   4 +-
 .../ControllerPeriodicTasksIntegrationTests.java   |   2 +-
 .../DeleteAPIHybridClusterIntegrationTest.java     |   2 +-
 .../tests/HybridClusterIntegrationTest.java        |   2 +-
 ...ridClusterIntegrationTestCommandLineRunner.java |   2 +-
 ...onaryAggregationPlanClusterIntegrationTest.java |   2 +-
 .../tests/OfflineClusterIntegrationTest.java       |   4 +-
 .../tests/PinotURIUploadIntegrationTest.java       |   7 +-
 .../tests/StarTreeClusterIntegrationTest.java      |   2 +-
 .../tests/StarTreeV2ClusterIntegrationTest.java    |   2 +-
 .../tests/UploadRefreshDeleteIntegrationTest.java  |   2 +-
 .../org/apache/pinot/minion/MinionStarter.java     |   4 +-
 .../BaseMultipleSegmentsConversionExecutor.java    |  17 +-
 .../BaseSingleSegmentConversionExecutor.java       |  13 +-
 .../apache/pinot/minion/metrics/MinionMetrics.java |   6 +-
 .../apache/pinot/perf/BenchmarkQueryEngine.java    |   1 -
 .../org/apache/pinot/server/conf/ServerConf.java   |   7 +
 .../apache/pinot/server/starter/ServerBuilder.java |   4 +-
 .../request/ScheduledRequestHandlerTest.java       |  11 +-
 .../command/BackfillDateTimeColumnCommand.java     |   2 +-
 .../tools/admin/command/UploadSegmentCommand.java  |   6 +-
 .../pinot/tools/backfill/BackfillSegmentUtils.java |   7 +-
 .../pinot/tools/perf/PerfBenchmarkDriver.java      |  19 --
 .../pinot/tools/perf/PerfBenchmarkDriverConf.java  |  63 ----
 .../apache/pinot/transport/netty/NettyServer.java  |   3 +-
 .../transport/perf/ScatterGatherPerfClient.java    |   5 +-
 .../pool/AsyncPoolResourceManagerAdapterTest.java  |  14 +-
 .../pinot/transport/pool/KeyedPoolImplTest.java    |  25 +-
 .../app/pods/components/alert-details/component.js |  11 +-
 .../app/pods/manage/explore/route.js               |   6 +-
 .../app/pods/manage/explore/template.hbs           |   1 +
 .../app/pods/manage/yaml/route.js                  |   6 +-
 .../app/pods/manage/yaml/template.hbs              |   1 +
 .../content/EntityGroupKeyContentFormatter.java    |   1 +
 .../anomaly/utils/ThirdeyeMetricsUtil.java         |   3 +
 .../detection/DetectionPipelineTaskRunner.java     |  11 +-
 .../pinot/thirdeye/detection/ModelRetuneFlow.java  |  13 +-
 .../alert/scheme/DetectionEmailAlerter.java        |  20 +-
 .../detector/entity-groupkey-anomaly-report.ftl    |   1 +
 .../TestEntityGroupKeyEmailContentFormatter.java   |   1 +
 ...est-entity-groupby-email-content-formatter.html |   1 +
 62 files changed, 423 insertions(+), 693 deletions(-)
 delete mode 100644 pinot-broker/src/main/java/org/apache/pinot/broker/routing/CfgBasedRouting.java
 delete mode 100644 pinot-broker/src/test/java/org/apache/pinot/broker/broker/EmptyBrokerOnlineOfflineStateModelFactory.java
 delete mode 100644 pinot-broker/src/test/java/org/apache/pinot/broker/broker/EmptySegmentOnlineOfflineStateModelFactory.java


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