[GitHub] [incubator-pinot] harleyjj opened a new pull request #6304: [TE] frontend - rca/harleyjj - Enable forecast baseline in RCA UI

harleyjj opened a new pull request #6304: URL: https://github.com/apache/incubator-pinot/pull/6304 ## Description - Adds forecast as baseline option in RCA UI - Enables selection of future dates only when forecast baseline is selected - Fixes small bug where calendar will disappear

[incubator-pinot] branch master updated: [TE] frontend harleyjj/rca - double encode filter value to protect against special characters (#6281)

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 5b0d1df [TE] frontend harleyjj/rca - do

[GitHub] [incubator-pinot] jihaozh merged pull request #6281: [TE] frontend harleyjj/rca - double encode filter value to protect ag…

jihaozh merged pull request #6281: URL: https://github.com/apache/incubator-pinot/pull/6281 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

[GitHub] [incubator-pinot] harleyjj commented on pull request #6281: [TE] frontend harleyjj/rca - double encode filter value to protect ag…

harleyjj commented on pull request #6281: URL: https://github.com/apache/incubator-pinot/pull/6281#issuecomment-736216219 @vincentchenjl @jihaozh The PR is passing tests after the rebase This is an automated message from the

[GitHub] [incubator-pinot] kishoreg commented on issue #6303: [pinot] Throw error when column doesn't exist during aggregations

kishoreg commented on issue #6303: URL: https://github.com/apache/incubator-pinot/issues/6303#issuecomment-736216130 @jackjlli there was some implementation on this right? This is an automated message from the Apache Git Ser

[GitHub] [incubator-pinot] harleyjj commented on pull request #6265: [TE] rest-api - harleyjj/rca - add forecast as baseline option for me…

harleyjj commented on pull request #6265: URL: https://github.com/apache/incubator-pinot/pull/6265#issuecomment-736215966 @jihaozh @bryantachen This one is passing tests after the rebase This is an automated message from the

[GitHub] [incubator-pinot] Aka-shi opened a new issue #6303: [pinot] Throw error when column doesn't exist during aggregations

Aka-shi opened a new issue #6303: URL: https://github.com/apache/incubator-pinot/issues/6303 As of now, when we call an aggregate function on a column which doesn't exist in the pinot table, the query returns the default value for that aggregation type instead of throwing `column doesn't e

[incubator-pinot] branch master updated: Adding custom metadata props into both segment metadata properties file and zk metadata record (#6299)

This is an automated email from the ASF dual-hosted git repository. xiangfu 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 d6862a2 Adding custom metadata props

[GitHub] [incubator-pinot] fx19880617 merged pull request #6299: Adding custom metadata props into both segment metadata properties file and zk metadata record

fx19880617 merged pull request #6299: URL: https://github.com/apache/incubator-pinot/pull/6299 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #6299: Adding custom metadata props into both segment metadata properties file and zk metadata record

fx19880617 commented on a change in pull request #6299: URL: https://github.com/apache/incubator-pinot/pull/6299#discussion_r533057260 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/index/metadata/SegmentMetadataImpl.java ## @@ -531,6 +553,12 @@ public Js

[GitHub] [incubator-pinot] mcvsubbu commented on pull request #6287: Parallelize segment index init and building

mcvsubbu commented on pull request #6287: URL: https://github.com/apache/incubator-pinot/pull/6287#issuecomment-736193107 @JohnTortugo can you elaborate on where you observed the performance problem? Was it during offline ingestion or in realtime pinot? ---

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6287: Parallelize segment index init and building

codecov-io edited a comment on pull request #6287: URL: https://github.com/apache/incubator-pinot/pull/6287#issuecomment-733427767 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6287?src=pr&el=h1) Report > Merging [#6287](https://codecov.io/gh/apache/incubator-pinot/pull

[GitHub] [incubator-pinot] Aka-shi opened a new issue #6302: Support for pausing the realtime consumption without disabling the table.

Aka-shi opened a new issue #6302: URL: https://github.com/apache/incubator-pinot/issues/6302 As of now, to pause the realtime consumption from kafka we have to disable the table. This also leads to the table not being available for querying. It would be helpful if there is support f

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #6299: Adding custom metadata props into both segment metadata properties file and zk metadata record

fx19880617 commented on a change in pull request #6299: URL: https://github.com/apache/incubator-pinot/pull/6299#discussion_r533032361 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/index/metadata/SegmentMetadataImpl.java ## @@ -531,6 +553,12 @@ public Js

[GitHub] [incubator-pinot] zhangloo333 commented on a change in pull request #6290: [TE]frontend - Build the tree parser for composite anomalies

zhangloo333 commented on a change in pull request #6290: URL: https://github.com/apache/incubator-pinot/pull/6290#discussion_r533016435 ## File path: thirdeye/thirdeye-frontend/app/utils/anomalies-tree-parser.js ## @@ -0,0 +1,563 @@ +import { isEmpty } from "@ember/utils"; +imp

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6259: Broker time range pruning(#6189)

Jackie-Jiang commented on a change in pull request #6259: URL: https://github.com/apache/incubator-pinot/pull/6259#discussion_r532999473 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/routing/segmentpruner/TimeSegmentPruner.java ## @@ -0,0 +1,344 @@ +/** + *

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6255: Creating a pluggable interface for Table config tuner

codecov-io edited a comment on pull request #6255: URL: https://github.com/apache/incubator-pinot/pull/6255#issuecomment-725034051 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6255?src=pr&el=h1) Report > Merging [#6255](https://codecov.io/gh/apache/incubator-pinot/pull

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6255: Creating a pluggable interface for Table config tuner

codecov-io edited a comment on pull request #6255: URL: https://github.com/apache/incubator-pinot/pull/6255#issuecomment-725034051 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6255?src=pr&el=h1) Report > Merging [#6255](https://codecov.io/gh/apache/incubator-pinot/pull

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6299: Adding custom metadata props into both segment metadata properties file and zk metadata record

Jackie-Jiang commented on a change in pull request #6299: URL: https://github.com/apache/incubator-pinot/pull/6299#discussion_r532991651 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/index/metadata/SegmentMetadataImpl.java ## @@ -531,6 +553,12 @@ public

[GitHub] [incubator-pinot] tejasajmera commented on a change in pull request #6290: [TE]frontend - Build the tree parser for composite anomalies

tejasajmera commented on a change in pull request #6290: URL: https://github.com/apache/incubator-pinot/pull/6290#discussion_r532978875 ## File path: thirdeye/thirdeye-frontend/app/utils/anomalies-tree-parser.js ## @@ -0,0 +1,563 @@ +import { isEmpty } from "@ember/utils"; +imp

[GitHub] [incubator-pinot] tejasajmera commented on a change in pull request #6290: [TE]frontend - Build the tree parser for composite anomalies

tejasajmera commented on a change in pull request #6290: URL: https://github.com/apache/incubator-pinot/pull/6290#discussion_r532978272 ## File path: thirdeye/thirdeye-frontend/app/utils/anomalies-tree-parser.js ## @@ -0,0 +1,563 @@ +import { isEmpty } from "@ember/utils"; +imp

[GitHub] [incubator-pinot] tejasajmera commented on a change in pull request #6290: [TE]frontend - Build the tree parser for composite anomalies

tejasajmera commented on a change in pull request #6290: URL: https://github.com/apache/incubator-pinot/pull/6290#discussion_r532974984 ## File path: thirdeye/thirdeye-frontend/app/utils/anomalies-tree-parser.js ## @@ -0,0 +1,563 @@ +import { isEmpty } from "@ember/utils"; +imp

[incubator-pinot] branch master updated (3eb0f9c -> b31b82e)

This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 3eb0f9c Use sorted index based filtering only for dictionary encoded column (#6288) add b31b82e [TE]

[GitHub] [incubator-pinot] jihaozh merged pull request #6260: [TE] frontend - harleyjj/alert-details - change alert charting logic …

jihaozh merged pull request #6260: URL: https://github.com/apache/incubator-pinot/pull/6260 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6299: Adding custom metadata props into both segment metadata properties file and zk metadata record

codecov-io edited a comment on pull request #6299: URL: https://github.com/apache/incubator-pinot/pull/6299#issuecomment-735545157 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6299?src=pr&el=h1) Report > Merging [#6299](https://codecov.io/gh/apache/incubator-pinot/pull

[incubator-pinot] branch all_custom_metadata_props_in_segment_creation updated (2b0df65 -> 1e8393f)

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch all_custom_metadata_props_in_segment_creation in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 2b0df65 Adding custom metadata props into both segment metadata pr

[incubator-pinot] branch master updated: Use sorted index based filtering only for dictionary encoded column (#6288)

This is an automated email from the ASF dual-hosted git repository. siddteotia 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 3eb0f9c Use sorted index based fil

[GitHub] [incubator-pinot] siddharthteotia merged pull request #6288: Use sorted index based filtering only for dictionary encoded column

siddharthteotia merged pull request #6288: URL: https://github.com/apache/incubator-pinot/pull/6288 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #6288: Use sorted index based filtering only for dictionary encoded column

siddharthteotia commented on a change in pull request #6288: URL: https://github.com/apache/incubator-pinot/pull/6288#discussion_r532849542 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/filter/FilterOperatorUtils.java ## @@ -47,9 +47,15 @@ public static

[GitHub] [incubator-pinot] pb-stripe opened a new issue #6301: Support for Non-Integer Percentile Calculations

pb-stripe opened a new issue #6301: URL: https://github.com/apache/incubator-pinot/issues/6301 In evaluating Pinot for an [SLI/SLO](https://cloud.google.com/blog/products/gcp/sre-fundamentals-slis-slas-and-slos) analytics use case, I was curious if there's built-in support for percentiles

[incubator-pinot] branch master updated: [TE] fix changing createdTime of anomalies (#6269)

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 0dee4d1 [TE] fix changing createdTime o

[GitHub] [incubator-pinot] jihaozh merged pull request #6269: [TE] fix changing createdTime of anomalies

jihaozh merged pull request #6269: URL: https://github.com/apache/incubator-pinot/pull/6269 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

[GitHub] [incubator-pinot] icefury71 opened a new issue #6300: Json schema validation for table config

icefury71 opened a new issue #6300: URL: https://github.com/apache/incubator-pinot/issues/6300 Currently, we're doing manual checks for each field in the schema regarding data type, range etc... Instead, we can use Json schema (https://json-schema.org/) to use annotation for auto-v

[incubator-pinot] 01/01: Initial commit for H3 based geospatial indexing

This is an automated email from the ASF dual-hosted git repository. kishoreg pushed a commit to branch h3-index in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit fe0e2862d157373f1be98c43f06fa09a4398a7ee Author: kishoreg AuthorDate: Mon Nov 30 09:58:32 2020 -0800 I

[incubator-pinot] branch h3-index created (now fe0e286)

This is an automated email from the ASF dual-hosted git repository. kishoreg pushed a change to branch h3-index in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at fe0e286 Initial commit for H3 based geospatial indexing This branch includes the following new commits

[GitHub] [incubator-pinot] zhangloo333 commented on a change in pull request #6290: [TE]frontend - Build the tree parser for composite anomalies

zhangloo333 commented on a change in pull request #6290: URL: https://github.com/apache/incubator-pinot/pull/6290#discussion_r532779193 ## File path: thirdeye/thirdeye-frontend/app/utils/anomalies-tree-parser.js ## @@ -0,0 +1,563 @@ +import { isEmpty } from "@ember/utils"; +imp

[GitHub] [incubator-pinot] mangrrua commented on pull request #5787: [Feature] - Spark Pinot Connector

mangrrua commented on pull request #5787: URL: https://github.com/apache/incubator-pinot/pull/5787#issuecomment-735817630 > @mangrrua any plans on implementing the write side of this? Seems very useful. We are working on this. ---

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6299: Adding custom metadata props into both segment metadata properties file and zk metadata record

codecov-io edited a comment on pull request #6299: URL: https://github.com/apache/incubator-pinot/pull/6299#issuecomment-735545157 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6299?src=pr&el=h1) Report > Merging [#6299](https://codecov.io/gh/apache/incubator-pinot/pull

[incubator-pinot] branch all_custom_metadata_props_in_segment_creation updated (a05ccdd -> 2b0df65)

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch all_custom_metadata_props_in_segment_creation in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard a05ccdd Adding custom metadata props into both segment metadata pr