[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4140: [TE] Update TE-Raptor API to support multiple metrics

akshayrai commented on a change in pull request #4140: [TE] Update TE-Raptor API to support multiple metrics URL: https://github.com/apache/incubator-pinot/pull/4140#discussion_r277018559 ## File path:

[GitHub] [incubator-pinot] akshayrai opened a new pull request #4145: [TE] Early termination if there are no 'to' recipients

akshayrai opened a new pull request #4145: [TE] Early termination if there are no 'to' recipients URL: https://github.com/apache/incubator-pinot/pull/4145 This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-pinot] tangdian opened a new pull request #4146: [TE] Migrating documentation to readthedocs.io

tangdian opened a new pull request #4146: [TE] Migrating documentation to readthedocs.io URL: https://github.com/apache/incubator-pinot/pull/4146 Updated some outdated documentation. Documentation can be found at https://thirdeye.readthedocs.io

[GitHub] [incubator-pinot] xiaohui-sun opened a new pull request #4147: Add upper lower bounds

xiaohui-sun opened a new pull request #4147: Add upper lower bounds URL: https://github.com/apache/incubator-pinot/pull/4147 Add upper/lower bounds to TimeSeries. This is an automated message from the Apache Git Service. To

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4146: [TE] Migrating documentation to readthedocs.io

xiaohui-sun commented on a change in pull request #4146: [TE] Migrating documentation to readthedocs.io URL: https://github.com/apache/incubator-pinot/pull/4146#discussion_r277070362 ## File path: thirdeye/docs/basic_config.rst ## @@ -0,0 +1,374 @@ +.. +.. Licensed to the

[incubator-pinot] branch decouple_broker updated (19d6583 -> 7db503f)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch decouple_broker in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 19d6583 Decouple BrokerServerBuilder from Helix new 7db503f Decouple BrokerServerBuilder

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4147: Add upper lower bounds

xiaohui-sun commented on a change in pull request #4147: Add upper lower bounds URL: https://github.com/apache/incubator-pinot/pull/4147#discussion_r277075206 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/spi/model/TimeSeries.java

[incubator-pinot] branch decouple_broker updated (f607f9d -> 19d6583)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch decouple_broker in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard f607f9d Decouple BrokerServerBuilder from Helix new 19d6583 Decouple BrokerServerBuilder

[GitHub] [incubator-pinot] akshayrai commented on issue #4146: [TE] Migrating documentation to readthedocs.io

akshayrai commented on issue #4146: [TE] Migrating documentation to readthedocs.io URL: https://github.com/apache/incubator-pinot/pull/4146#issuecomment-484980537 Thanks @tangdian for putting this together. But I have some concerns with the maintenance of this doc. Editing a doc in

[GitHub] [incubator-pinot] jihaozh commented on issue #4146: [TE] Migrating documentation to readthedocs.io

jihaozh commented on issue #4146: [TE] Migrating documentation to readthedocs.io URL: https://github.com/apache/incubator-pinot/pull/4146#issuecomment-484990161 Thank you for putting together the docs! @akshayrai Looks like Pinot moved away from Github wiki to readthedocs. see

[incubator-pinot] branch master updated: In ServiceStatus, do not log anything if all resources are up (#4141)

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 083b288 In ServiceStatus, do not log

[incubator-pinot] branch service_status_log deleted (was 65c5047)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch service_status_log in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 65c5047 In ServiceStatus, do not log anything if all resources are up The revisions that were

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #4141: In ServiceStatus, do not log anything if all resources are up

Jackie-Jiang merged pull request #4141: In ServiceStatus, do not log anything if all resources are up URL: https://github.com/apache/incubator-pinot/pull/4141 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #4144: Decouple BrokerServerBuilder from Helix

codecov-io edited a comment on issue #4144: Decouple BrokerServerBuilder from Helix URL: https://github.com/apache/incubator-pinot/pull/4144#issuecomment-484773366 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4144?src=pr=h1) Report > Merging

[incubator-pinot] branch master updated: [TE] Early termination if there are no 'to' recipients (#4145)

This is an automated email from the ASF dual-hosted git repository. akshayrai09 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 96752f7 [TE] Early termination

[GitHub] [incubator-pinot] akshayrai merged pull request #4145: [TE] Early termination if there are no 'to' recipients

akshayrai merged pull request #4145: [TE] Early termination if there are no 'to' recipients URL: https://github.com/apache/incubator-pinot/pull/4145 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4147: Add upper lower bounds

jihaozh commented on a change in pull request #4147: Add upper lower bounds URL: https://github.com/apache/incubator-pinot/pull/4147#discussion_r277074035 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/spi/model/TimeSeries.java ##

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4147: Add upper lower bounds

jihaozh commented on a change in pull request #4147: Add upper lower bounds URL: https://github.com/apache/incubator-pinot/pull/4147#discussion_r277073767 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/spi/model/TimeSeries.java ##

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4147: Add upper lower bounds

jihaozh commented on a change in pull request #4147: Add upper lower bounds URL: https://github.com/apache/incubator-pinot/pull/4147#discussion_r277074035 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/spi/model/TimeSeries.java ##

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4147: Add upper lower bounds

jihaozh commented on a change in pull request #4147: Add upper lower bounds URL: https://github.com/apache/incubator-pinot/pull/4147#discussion_r277074035 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/spi/model/TimeSeries.java ##

[incubator-pinot] branch master updated: [TE] Update TE-Raptor API to support multiple metrics (#4140)

This is an automated email from the ASF dual-hosted git repository. akshayrai09 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 983c5d8 [TE] Update TE-Raptor

[GitHub] [incubator-pinot] akshayrai merged pull request #4140: [TE] Update TE-Raptor API to support multiple metrics

akshayrai merged pull request #4140: [TE] Update TE-Raptor API to support multiple metrics URL: https://github.com/apache/incubator-pinot/pull/4140 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4148: [TE] detection - two side threshold filter

xiaohui-sun commented on a change in pull request #4148: [TE] detection - two side threshold filter URL: https://github.com/apache/incubator-pinot/pull/4148#discussion_r277105587 ## File path:

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4149: [TE] support monitoring granularity for Holt Winters

xiaohui-sun commented on a change in pull request #4149: [TE] support monitoring granularity for Holt Winters URL: https://github.com/apache/incubator-pinot/pull/4149#discussion_r277107024 ## File path:

[incubator-pinot] branch master updated: [TE] frontend - harleyjj/anomalies - make anomalies filterable by subscription group (#4131)

This is an automated email from the ASF dual-hosted git repository. jihao 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 d403a28 [TE] frontend -

[GitHub] [incubator-pinot] jihaozh merged pull request #4131: [TE] frontend - harleyjj/anomalies - make anomalies filterable by sub…

jihaozh merged pull request #4131: [TE] frontend - harleyjj/anomalies - make anomalies filterable by sub… URL: https://github.com/apache/incubator-pinot/pull/4131 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-pinot] xiaohui-sun merged pull request #4147: Add upper lower bounds

xiaohui-sun merged pull request #4147: Add upper lower bounds URL: https://github.com/apache/incubator-pinot/pull/4147 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[incubator-pinot] branch master updated: Add upper lower bounds (#4147)

This is an automated email from the ASF dual-hosted git repository. xhsun 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 937ecb4 Add upper lower bounds (#4147)

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4148: [TE] detection - two side threshold filter

jihaozh opened a new pull request #4148: [TE] detection - two side threshold filter URL: https://github.com/apache/incubator-pinot/pull/4148 - This PR makes it possible to set different threshold in different anomaly pattern in percentage change anomaly filter. - Fix baseline values

[GitHub] [incubator-pinot] xiaohui-sun opened a new pull request #4151: [TE] Add upper/lower/current to HoltWinters baseline

xiaohui-sun opened a new pull request #4151: [TE] Add upper/lower/current to HoltWinters baseline URL: https://github.com/apache/incubator-pinot/pull/4151 Add upper/lower/current to HoltWinters baseline. This is an automated

[GitHub] [incubator-pinot] katewang0710 commented on issue #4133: add more types parsing for string coerceValueIntoField

katewang0710 commented on issue #4133: add more types parsing for string coerceValueIntoField URL: https://github.com/apache/incubator-pinot/pull/4133#issuecomment-485050143 Hi @Jackie-Jiang, Do you mean support boolean , byte , char , short , int , long , float and double for

[GitHub] [incubator-pinot] tangdian opened a new pull request #4149: [TE] support monitoring granularity for Holt Winters

tangdian opened a new pull request #4149: [TE] support monitoring granularity for Holt Winters URL: https://github.com/apache/incubator-pinot/pull/4149 This will enable 7_DAYS monitoring granularity and beyond. This is an

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4148: [TE] detection - two side threshold filter

jihaozh commented on a change in pull request #4148: [TE] detection - two side threshold filter URL: https://github.com/apache/incubator-pinot/pull/4148#discussion_r277107279 ## File path:

[incubator-pinot] branch master updated: [TE] Fix for TimeSeries constructor (#4150)

This is an automated email from the ASF dual-hosted git repository. xhsun 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 955748b [TE] Fix for TimeSeries

[GitHub] [incubator-pinot] xiaohui-sun merged pull request #4150: [TE] Fix for TimeSeries constructor

xiaohui-sun merged pull request #4150: [TE] Fix for TimeSeries constructor URL: https://github.com/apache/incubator-pinot/pull/4150 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-pinot] tangdian commented on a change in pull request #4149: [TE] support monitoring granularity for Holt Winters

tangdian commented on a change in pull request #4149: [TE] support monitoring granularity for Holt Winters URL: https://github.com/apache/incubator-pinot/pull/4149#discussion_r277110926 ## File path:

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4149: [TE] support monitoring granularity for Holt Winters

jihaozh commented on a change in pull request #4149: [TE] support monitoring granularity for Holt Winters URL: https://github.com/apache/incubator-pinot/pull/4149#discussion_r277110402 ## File path:

[GitHub] [incubator-pinot] tangdian commented on a change in pull request #4149: [TE] support monitoring granularity for Holt Winters

tangdian commented on a change in pull request #4149: [TE] support monitoring granularity for Holt Winters URL: https://github.com/apache/incubator-pinot/pull/4149#discussion_r277110837 ## File path:

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4149: [TE] support monitoring granularity for Holt Winters

jihaozh commented on a change in pull request #4149: [TE] support monitoring granularity for Holt Winters URL: https://github.com/apache/incubator-pinot/pull/4149#discussion_r277109385 ## File path:

[GitHub] [incubator-pinot] xiaohui-sun opened a new pull request #4150: [TE] Fix for TimeSeries constructor

xiaohui-sun opened a new pull request #4150: [TE] Fix for TimeSeries constructor URL: https://github.com/apache/incubator-pinot/pull/4150 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-pinot] tangdian commented on issue #4149: [TE] support monitoring granularity for Holt Winters

tangdian commented on issue #4149: [TE] support monitoring granularity for Holt Winters URL: https://github.com/apache/incubator-pinot/pull/4149#issuecomment-485036268 I have updated the code to support 1_MONTHS data This is

[incubator-pinot] 01/01: Fix the issue where ZkCacheBaseDataAccessor.getChildren() can return list with null znRecords

This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch null_zn_records in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit a2b0667bd9951d6a6c2cf462ba888923582a8bcb Author: Jackie (Xiaotian) Jiang AuthorDate: Fri Apr 19

[incubator-pinot] branch null_zn_records created (now a2b0667)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch null_zn_records in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at a2b0667 Fix the issue where ZkCacheBaseDataAccessor.getChildren() can return list with null

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #4152: Fix the issue where ZkCacheBaseDataAccessor.getChildren() can return list with null znRecords

Jackie-Jiang opened a new pull request #4152: Fix the issue where ZkCacheBaseDataAccessor.getChildren() can return list with null znRecords URL: https://github.com/apache/incubator-pinot/pull/4152 This can happen if the record gets removed while calling this method. Need to handle it to

[GitHub] [incubator-pinot] codecov-io commented on issue #4152: Fix the issue where ZkCacheBaseDataAccessor.getChildren() can return list with null znRecords

codecov-io commented on issue #4152: Fix the issue where ZkCacheBaseDataAccessor.getChildren() can return list with null znRecords URL: https://github.com/apache/incubator-pinot/pull/4152#issuecomment-485058207 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4152?src=pr=h1)