[incubator-pinot] branch update_merge_logic created (now 4896956)

This is an automated email from the ASF dual-hosted git repository. xhsun pushed a change to branch update_merge_logic in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 4896956 [TE] Update anomaly merge logic 1. Don't output existing anomalies that not get

[incubator-pinot] 01/01: [TE] Update anomaly merge logic 1. Don't output existing anomalies that not get merged. 2. Merge new anomaly's properties into existing anomaly. 3. Do not split existing anoma

This is an automated email from the ASF dual-hosted git repository. xhsun pushed a commit to branch update_merge_logic in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 48969567d08c706a66c7bcbea5365bf32300cd74 Author: Xiaohui Sun AuthorDate: Fri May 10 14:25:00 2019

[GitHub] [incubator-pinot] xiaohui-sun opened a new pull request #4201: [TE] Update anomaly merge logic

xiaohui-sun opened a new pull request #4201: [TE] Update anomaly merge logic URL: https://github.com/apache/incubator-pinot/pull/4201 1. Don't output existing anomalies that not get merged. 2. Merge new anomaly's properties into existing anomaly. 3. Do not split existing anomalies.

[GitHub] [incubator-pinot] fx19880617 opened a new pull request #4200: Adding script to build and publish docker image

fx19880617 opened a new pull request #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200 Adding docker support to build image. This is an automated message from the

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4202: [TE] Evaluation metrics store

jihaozh opened a new pull request #4202: [TE] Evaluation metrics store URL: https://github.com/apache/incubator-pinot/pull/4202 The performance metrics need to be stored as a by-product of each detection. Later this will be read by the model evaluators to monitor the detection models.

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4201: [TE] Update anomaly merge logic

jihaozh commented on a change in pull request #4201: [TE] Update anomaly merge logic URL: https://github.com/apache/incubator-pinot/pull/4201#discussion_r283062076 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/algorithm/MergeWrapper.java

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4203: [TE] Threshold filter on current value of an anomaly

jihaozh opened a new pull request #4203: [TE] Threshold filter on current value of an anomaly URL: https://github.com/apache/incubator-pinot/pull/4203 This PR make the threshold filter to filter on the current value of the anomaly.

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image

Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#discussion_r283033682 ## File path: docker/Dockerfile ## @@ -0,0 +1,35 @@ +FROM openjdk:8u181-jdk-stretch +

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image

Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#discussion_r283035190 ## File path: docker/docker-build.sh ## @@ -0,0 +1,28 @@ +#!/bin/bash + +if [[ "$#" -gt

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image

Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#discussion_r283035164 ## File path: docker/docker-build-and-push.sh ## @@ -0,0 +1,11 @@ +#!/bin/bash + +sh

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image

Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#discussion_r283033921 ## File path: docker/Dockerfile ## @@ -0,0 +1,35 @@ +FROM openjdk:8u181-jdk-stretch +

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image

Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#discussion_r283034651 ## File path: docker/Dockerfile ## @@ -0,0 +1,35 @@ +FROM openjdk:8u181-jdk-stretch +

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image

Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#discussion_r283034168 ## File path: docker/Dockerfile ## @@ -0,0 +1,35 @@ +FROM openjdk:8u181-jdk-stretch +

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image

Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#discussion_r283034301 ## File path: docker/Dockerfile ## @@ -0,0 +1,35 @@ +FROM openjdk:8u181-jdk-stretch +

[incubator-pinot] branch update_merge_logic updated (4896956 -> 92ffca8)

This is an automated email from the ASF dual-hosted git repository. xhsun pushed a change to branch update_merge_logic in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 4896956 [TE] Update anomaly merge logic 1. Don't output existing anomalies that not get

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #4047: Create leadControllerResource in helix cluster

codecov-io edited a comment on issue #4047: Create leadControllerResource in helix cluster URL: https://github.com/apache/incubator-pinot/pull/4047#issuecomment-479147030 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4047?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] snleee edited a comment on issue #4200: Adding script to build and publish docker image

snleee edited a comment on issue #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#issuecomment-491474406 Where do we publish the docker image? Apache Incubator had the restriction on releasing non-apache approved releases to

[GitHub] [incubator-pinot] snleee edited a comment on issue #4200: Adding script to build and publish docker image

snleee edited a comment on issue #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#issuecomment-491474406 Where do we publish the docker image? Apache Incubator had the restriction on releasing non-apache approved releases to

[GitHub] [incubator-pinot] codecov-io commented on issue #4198: Enhance the support for BYTES data type

codecov-io commented on issue #4198: Enhance the support for BYTES data type URL: https://github.com/apache/incubator-pinot/pull/4198#issuecomment-491464486 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4198?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] snleee edited a comment on issue #4200: Adding script to build and publish docker image

snleee edited a comment on issue #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#issuecomment-491474406 Where do we publish the docker image? Apache Incubator had the restriction on releasing non-apache approved releases to

[GitHub] [incubator-pinot] snleee commented on issue #4200: Adding script to build and publish docker image

snleee commented on issue #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#issuecomment-491474406 Where do we publish the docker image? Apache Incubator had the restriction on releasing non-apache approved releases to public.

[GitHub] [incubator-pinot] chenboat commented on a change in pull request #4143: Use patricia trie to OnHeapTrieBasedStringDictionary

chenboat commented on a change in pull request #4143: Use patricia trie to OnHeapTrieBasedStringDictionary URL: https://github.com/apache/incubator-pinot/pull/4143#discussion_r282756708 ## File path:

[GitHub] [incubator-pinot] chenboat commented on a change in pull request #4143: Use patricia trie to OnHeapTrieBasedStringDictionary

chenboat commented on a change in pull request #4143: Use patricia trie to OnHeapTrieBasedStringDictionary URL: https://github.com/apache/incubator-pinot/pull/4143#discussion_r282756752 ## File path:

[GitHub] [incubator-pinot] chenboat commented on a change in pull request #4143: Use patricia trie to OnHeapTrieBasedStringDictionary

chenboat commented on a change in pull request #4143: Use patricia trie to OnHeapTrieBasedStringDictionary URL: https://github.com/apache/incubator-pinot/pull/4143#discussion_r282757154 ## File path:

[GitHub] [incubator-pinot] sunny209868 opened a new issue #4199: Support for SQL plans

sunny209868 opened a new issue #4199: Support for SQL plans URL: https://github.com/apache/incubator-pinot/issues/4199 Is there any plan to support SQL? It seems that the current PQL is still not convenient for all kinds of complex queries

[GitHub] [incubator-pinot] chenboat commented on a change in pull request #4143: Use patricia trie to OnHeapTrieBasedStringDictionary

chenboat commented on a change in pull request #4143: Use patricia trie to OnHeapTrieBasedStringDictionary URL: https://github.com/apache/incubator-pinot/pull/4143#discussion_r282758974 ## File path:

[incubator-pinot] branch master updated: [TE] Update anomaly merge logic (#4201)

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 1ac071f [TE] Update anomaly merge

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4202: [TE] Evaluation metrics store

xiaohui-sun commented on a change in pull request #4202: [TE] Evaluation metrics store URL: https://github.com/apache/incubator-pinot/pull/4202#discussion_r283077433 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/algorithm/MergeWrapper.java

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4202: [TE] Evaluation metrics store

xiaohui-sun commented on a change in pull request #4202: [TE] Evaluation metrics store URL: https://github.com/apache/incubator-pinot/pull/4202#discussion_r283077065 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/datalayer/dto/EvaluationDTO.java

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4202: [TE] Evaluation metrics store

xiaohui-sun commented on a change in pull request #4202: [TE] Evaluation metrics store URL: https://github.com/apache/incubator-pinot/pull/4202#discussion_r283076801 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/datalayer/bao/EvaluationManager.java

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4202: [TE] Evaluation metrics store

xiaohui-sun commented on a change in pull request #4202: [TE] Evaluation metrics store URL: https://github.com/apache/incubator-pinot/pull/4202#discussion_r283077562 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/Evaluation.java ##

[GitHub] [incubator-pinot] snleee edited a comment on issue #4200: Adding script to build and publish docker image

snleee edited a comment on issue #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#issuecomment-491474406 Where do we publish the docker image? Apache Incubator had the restriction on releasing non-apache approved releases to

[GitHub] [incubator-pinot] sunny209868 commented on issue #4199: Support for SQL plans

sunny209868 commented on issue #4199: Support for SQL plans URL: https://github.com/apache/incubator-pinot/issues/4199#issuecomment-491474556 For example, Having, the addition, subtraction, multiplication, and division of an aggregate indicator(SUM(a)/SUM(b))

[GitHub] [incubator-pinot] sunny209868 edited a comment on issue #4199: Support for SQL plans

sunny209868 edited a comment on issue #4199: Support for SQL plans URL: https://github.com/apache/incubator-pinot/issues/4199#issuecomment-491474556 For example, having, aggregating the addition, subtraction, multiplication, and division of metrics (sum(a)/sum(b)), I plan to support these