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/02/26 17:48:55 UTC

[incubator-pinot] branch improve-segment-validator updated (6a83d22 -> 644ad0e)

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

jlli pushed a change to branch improve-segment-validator
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.


 discard 6a83d22  Address PR comments
 discard d09e7f5  Add Precondition on assignedInstances before updating zk
 discard 5576f32  Address PR comments
 discard 69e43ac  Address PR comments
 discard 8f4fd4b  Improve segment validator to check assigned instances for uploading new segment
     new 61bfdc6  [TE] Endpoint for migrating partially migrated alerts (#3855)
     new 0d981ab  [TE] detection - time series provider loading cache (#3842)
     new c943843  Minor fix for integration test (#3858)
     new 50d3eae  [TE] Update Yaml templates (#3859)
     new bfce886  Fix compile failure issue when docs/_build is present (#3857)
     new a745087  Add header definition for rst files (#3860)
     new 795f494  [TE] frontend - harleyjj/rootcause-anomaly - never show inconsistent data warning for 1 and 5 minute granularity (#3861)
     new b2c3528  Add scheduler wait time for server side log (#3862)
     new 4f96cba  [TE] frontend - harleyjj/alert-details - multiple alert overview improvements for MVP (#3863)
     new 23cda0b  only show inconsistent alert when it is daily metrics (#3865)
     new 1f9d096  [TE] detection - task scheduler backoff fix (#3866)
     new b3f7fee  Fix reporting/logging of numEntriesScannedPostFilter (#3870)
     new f70935e  Disable realtime aggregate metrics if multi value columns present (#3868)
     new e3a9184  Fix listFiles method in HadoopPinotFS (#3838)
     new de950ff  [TE] Update label and detection description (#3878)
     new 126b022  [TE] Include alert description in the email (#3879)
     new e6cdea0  [TE] frontend - harleyjj/manage-alerts - fix display bugs in alerts list (#3882)
     new 209c966  [TE] detection - threshold-based filter (#3881)
     new 4e7ffb3  [TE] Migration - fill in metric urn for old anomalies (#3880)
     new 2da1157  Doc coding guidelines (#3884)
     new 47be0f0  [TE] Enable description after migration (#3883)
     new 785f7e8  Improve segment validator to check assigned instances for uploading new segment
     new 54c1d88  Address PR comments
     new e7f5144  Address PR comments
     new 2923703  Add Precondition on assignedInstances before updating zk
     new 644ad0e  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   (6a83d22)
            \
             N -- N -- N   refs/heads/improve-segment-validator (644ad0e)

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.

The 5775 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 docs/{index.rst => coding_guidelines.rst}          |  48 ++--
 docs/index.rst                                     |   8 +
 .../resources/LLCSegmentCompletionHandlers.java    |   3 +
 .../realtime/PinotLLCRealtimeSegmentManager.java   |   6 +-
 .../realtime/LLRealtimeSegmentDataManager.java     |   2 +-
 .../indexsegment/mutable/MutableSegmentImpl.java   |  19 ++
 .../pinot/core/query/scheduler/QueryScheduler.java |   5 +-
 pinot-distribution/pinot-source-assembly.xml       |   3 +
 .../org/apache/pinot/filesystem/HadoopPinotFS.java |  22 +-
 pom.xml                                            |  11 +
 config/suppressions.xml => rstheaderdefinition.xml |  21 +-
 .../app/pods/components/alert-details/component.js | 264 +++++++++++++++------
 .../app/pods/components/alert-details/template.hbs | 132 +++++++----
 .../pods/components/rootcause-anomaly/component.js |  43 ++--
 .../self-serve-alert-details/template.hbs          |   2 +-
 .../self-serve-alert-yaml-details/template.hbs     |   2 +-
 .../app/pods/components/yaml-editor/template.hbs   |   6 +-
 .../custom/anomalies-table/dimensions/template.hbs |   2 +-
 .../app/pods/manage/alert/explore/controller.js    |  12 +-
 .../app/pods/manage/alerts/index/controller.js     |  14 +-
 .../app/pods/manage/alerts/index/route.js          |  10 +-
 .../app/pods/manage/yaml/template.hbs              |   2 +-
 thirdeye/thirdeye-frontend/app/utils/anomaly.js    |  33 +++
 thirdeye/thirdeye-frontend/app/utils/constants.js  | 101 ++++----
 .../app/utils/manage-alert-utils.js                |  20 +-
 .../components/rootcause-anomaly/component-test.js |   6 +-
 .../alert/content/BaseEmailContentFormatter.java   |  13 +-
 ...HierarchicalAnomaliesEmailContentFormatter.java |   2 +
 .../MultipleAnomaliesEmailContentFormatter.java    |   4 +-
 .../pinot/thirdeye/detection/DataProvider.java     |   8 -
 .../thirdeye/detection/DefaultDataProvider.java    |  62 +++--
 .../detection/DetectionMigrationResource.java      |  80 ++++++-
 .../thirdeye/detection/DetectionPipelineJob.java   |  25 +-
 .../detection/LegacyAnomalyFunctionTranslator.java |   2 +-
 .../detection/alert/DetectionAlertJob.java         |  19 +-
 .../components/PercentageChangeRuleDetector.java   |   4 +-
 .../components/ThresholdRuleAnomalyFilter.java     |  32 ++-
 .../detection/spec/ThresholdRuleFilterSpec.java    |  40 +++-
 .../detection/wrapper/AnomalyDetectorWrapper.java  |   8 +-
 .../thirdeye/detection/yaml/YamlResource.java      |   3 +-
 .../thirdeye/detector/holiday-anomaly-report.ftl   |   6 +-
 .../pinot/thirdeye/detection/DataProviderTest.java |  82 +------
 .../detection/DetectionMigrationResourceTest.java  |   4 +
 .../components/ThresholdRuleAnomalyFilterTest.java |  66 ++++--
 .../components/ThresholdRuleDetectorTest.java      |  34 +--
 .../finetune/GridSearchTuningAlgorithmTest.java    | 134 -----------
 .../wrapper/AnomalyDetectorWrapperTest.java        |  11 +-
 .../detection/migrated-detection-config-1.json     |   4 +-
 .../detection/migrated-detection-config-2.json     |   4 +-
 ...multiple-anomalies-email-content-formatter.html |   3 +-
 50 files changed, 866 insertions(+), 581 deletions(-)
 copy docs/{index.rst => coding_guidelines.rst} (56%)
 copy config/suppressions.xml => rstheaderdefinition.xml (67%)
 delete mode 100644 thirdeye/thirdeye-pinot/src/test/java/org/apache/pinot/thirdeye/detection/finetune/GridSearchTuningAlgorithmTest.java


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