[GitHub] [incubator-pinot] fx19880617 merged pull request #5997: Fixing website doc hyperlinks

2020-09-10 Thread GitBox
fx19880617 merged pull request #5997: URL: https://github.com/apache/incubator-pinot/pull/5997 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] vincentchenjl opened a new pull request #5999: [TE] adding back metricUrn in AnomalyFilterWrapper.run()

2020-09-10 Thread GitBox
vincentchenjl opened a new pull request #5999: URL: https://github.com/apache/incubator-pinot/pull/5999 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-pinot] icefury71 commented on a change in pull request #5966: Validate timeColumnName when adding/updating schema/tableConfig

2020-09-10 Thread GitBox
icefury71 commented on a change in pull request #5966: URL: https://github.com/apache/incubator-pinot/pull/5966#discussion_r486588035 ## File path: pinot-core/src/test/java/org/apache/pinot/core/util/SchemaUtilsTest.java ## @@ -35,6 +43,115 @@ */ public class

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5966: Validate timeColumnName when adding/updating schema/tableConfig

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5966: URL: https://github.com/apache/incubator-pinot/pull/5966#discussion_r486480697 ## File path: pinot-core/src/main/java/org/apache/pinot/core/util/TableConfigUtils.java ## @@ -74,19 +80,37 @@ public static void

[incubator-pinot] branch master updated: Add ingestion descriptor in the header (#5995)

2020-09-10 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli 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 e0f8c11 Add ingestion descriptor in the

[GitHub] [incubator-pinot] jackjlli merged pull request #5995: Add ingestion descriptor in the header

2020-09-10 Thread GitBox
jackjlli merged pull request #5995: URL: https://github.com/apache/incubator-pinot/pull/5995 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

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5966: Validate timeColumnName when adding/updating schema/tableConfig

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5966: URL: https://github.com/apache/incubator-pinot/pull/5966#discussion_r486601450 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSchemaRestletResource.java ## @@ -318,4 +324,19 @@

[GitHub] [incubator-pinot] fx19880617 opened a new issue #5998: Reporting per column level stats for table

2020-09-10 Thread GitBox
fx19880617 opened a new issue #5998: URL: https://github.com/apache/incubator-pinot/issues/5998 We already have table level and per segment level stats right now. The next question here is that what's the per column level information/stats we can expose. One typical question

[incubator-pinot] branch master updated (462cf57 -> 41f3a94)

2020-09-10 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 462cf57 [TE] adding back metricUrn in AnomalyFilterWrapper.run() (#5999) add 41f3a94 upgrade to

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5966: Validate timeColumnName when adding/updating schema/tableConfig

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5966: URL: https://github.com/apache/incubator-pinot/pull/5966#discussion_r486488972 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSchemaRestletResource.java ## @@ -171,7 +173,8 @@

[GitHub] [incubator-pinot] npawar commented on issue #5928: Add connector for Pravega

2020-09-10 Thread GitBox
npawar commented on issue #5928: URL: https://github.com/apache/incubator-pinot/issues/5928#issuecomment-690594852 `A Pravega stream comprises a set of parallel segments, and that set can change over time according to a scaling policy` - is "set of segments" like partitions within the

[GitHub] [incubator-pinot] fx19880617 opened a new issue #6000: Fix broker apis

2020-09-10 Thread GitBox
fx19880617 opened a new issue #6000: URL: https://github.com/apache/incubator-pinot/issues/6000 Current controller `/brokers` APIs only returns instance id and expect instance is in the format of `ROLE_HOSTNAME_PORT`, however this is not the right thing any more. So we propose the new

[incubator-pinot] branch master updated (b2fa2ca -> 473353d)

2020-09-10 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from b2fa2ca Add 0.5 to website. (#5996) add 473353d Fixing website doc hyper links (#5997) No new

[incubator-pinot] branch master updated (e0f8c11 -> 462cf57)

2020-09-10 Thread jihao
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 e0f8c11 Add ingestion descriptor in the header (#5995) add 462cf57 [TE] adding back metricUrn in

[GitHub] [incubator-pinot] jihaozh merged pull request #5999: [TE] adding back metricUrn in AnomalyFilterWrapper.run()

2020-09-10 Thread GitBox
jihaozh merged pull request #5999: URL: https://github.com/apache/incubator-pinot/pull/5999 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

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5966: Validate timeColumnName when adding/updating schema/tableConfig

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5966: URL: https://github.com/apache/incubator-pinot/pull/5966#discussion_r486497471 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSchemaRestletResource.java ## @@ -171,7 +173,8 @@

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #5972: [TE] add threshold-based anomaly labeler

2020-09-10 Thread GitBox
jihaozh commented on a change in pull request #5972: URL: https://github.com/apache/incubator-pinot/pull/5972#discussion_r486511634 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/wrapper/AnomalyFilterWrapper.java ## @@ -81,21 +80,7 @@

[GitHub] [incubator-pinot] icefury71 commented on a change in pull request #5966: Validate timeColumnName when adding/updating schema/tableConfig

2020-09-10 Thread GitBox
icefury71 commented on a change in pull request #5966: URL: https://github.com/apache/incubator-pinot/pull/5966#discussion_r486512420 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSchemaRestletResource.java ## @@ -171,7 +173,8 @@

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5966: Validate timeColumnName when adding/updating schema/tableConfig

2020-09-10 Thread GitBox
Jackie-Jiang commented on a change in pull request #5966: URL: https://github.com/apache/incubator-pinot/pull/5966#discussion_r486562121 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSchemaRestletResource.java ## @@ -318,4 +324,19

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5966: Validate timeColumnName when adding/updating schema/tableConfig

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5966: URL: https://github.com/apache/incubator-pinot/pull/5966#discussion_r486484365 ## File path: pinot-core/src/test/java/org/apache/pinot/core/util/SchemaUtilsTest.java ## @@ -35,6 +43,115 @@ */ public class SchemaUtilsTest

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
akshayrai commented on a change in pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#discussion_r486578318 ## File path: thirdeye/thirdeye-spi/src/main/java/org/apache/pinot/thirdeye/datalayer/pojo/AlertConfigBean.java ## @@ -211,7 +211,7 @@ public

[GitHub] [incubator-pinot] npawar merged pull request #5966: Validate timeColumnName when adding/updating schema/tableConfig

2020-09-10 Thread GitBox
npawar merged pull request #5966: URL: https://github.com/apache/incubator-pinot/pull/5966 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

[incubator-pinot] branch master updated: Validate timeColumnName when adding/updating schema/tableConfig (#5966)

2020-09-10 Thread nehapawar
This is an automated email from the ASF dual-hosted git repository. nehapawar 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 e0ed179 Validate timeColumnName

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486672513 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/collector/CollectorConfig.java ## @@ -0,0 +1,103 @@ +/** + *

[GitHub] [incubator-pinot] suvodeep-pyne commented on a change in pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on a change in pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#discussion_r486692492 ## File path: thirdeye/thirdeye-spi/src/main/java/org/apache/pinot/thirdeye/datalayer/pojo/AlertConfigBean.java ## @@ -211,7 +211,7 @@

[GitHub] [incubator-pinot] suvodeep-pyne commented on a change in pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on a change in pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#discussion_r486692447 ## File path: thirdeye/pom.xml ## @@ -543,6 +550,17 @@ log4j-slf4j-impl ${log4j2.version} + +

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486700161 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/framework/SegmentProcessorFramework.java ## @@ -0,0 +1,194 @@

[GitHub] [incubator-pinot] npawar closed issue #5915: Ensure time-column compatibility between schema and tableconfig

2020-09-10 Thread GitBox
npawar closed issue #5915: URL: https://github.com/apache/incubator-pinot/issues/5915 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 to go

[GitHub] [incubator-pinot] suvodeep-pyne commented on a change in pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on a change in pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#discussion_r486691580 ## File path: thirdeye/pom.xml ## @@ -22,17 +22,18 @@ 4.0.0 org.apache.pinot.thirdeye thirdeye - ${revision}${sha1} Review

[GitHub] [incubator-pinot] suvodeep-pyne commented on a change in pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on a change in pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#discussion_r486691580 ## File path: thirdeye/pom.xml ## @@ -22,17 +22,18 @@ 4.0.0 org.apache.pinot.thirdeye thirdeye - ${revision}${sha1} Review

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486673217 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/framework/SegmentProcessorConfig.java ## @@ -0,0 +1,181 @@ +/** +

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486683841 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/partitioner/NoOpPartitioner.java ## @@ -0,0 +1,32 @@ +/** + *

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486683505 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/collector/RollupCollector.java ## @@ -0,0 +1,159 @@ +/** + *

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486676878 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/utils/SegmentProcessorUtils.java ## @@ -0,0 +1,113 @@ +/** + *

[GitHub] [incubator-pinot] suvodeep-pyne commented on a change in pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on a change in pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#discussion_r486692447 ## File path: thirdeye/pom.xml ## @@ -543,6 +550,17 @@ log4j-slf4j-impl ${log4j2.version} + +

[GitHub] [incubator-pinot] suvodeep-pyne commented on pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#issuecomment-690852472 Thanks a lot @akshayrai ! This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486657189 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/collector/ValueAggregator.java ## @@ -16,7 +16,7 @@ * specific

[GitHub] [incubator-pinot] suvodeep-pyne commented on a change in pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on a change in pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#discussion_r486692006 ## File path: thirdeye/pom.xml ## @@ -22,17 +22,18 @@ 4.0.0 org.apache.pinot.thirdeye thirdeye - ${revision}${sha1} +

[GitHub] [incubator-pinot] suvodeep-pyne commented on a change in pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on a change in pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#discussion_r486691580 ## File path: thirdeye/pom.xml ## @@ -22,17 +22,18 @@ 4.0.0 org.apache.pinot.thirdeye thirdeye - ${revision}${sha1} Review

[GitHub] [incubator-pinot] suvodeep-pyne commented on a change in pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on a change in pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#discussion_r486691075 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/datalayer/dto/OnlineDetectionDataDTO.java ## @@ -1,6 +0,0 @@

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486699922 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/framework/SegmentProcessorConfig.java ## @@ -0,0 +1,181 @@ +/** +

[GitHub] [incubator-pinot] suvodeep-pyne commented on a change in pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on a change in pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#discussion_r486690825 ## File path: thirdeye/thirdeye-spi/src/main/java/org/apache/pinot/thirdeye/datalayer/dto/AlertSnapshotDTO.java ## @@ -80,48 +75,6 @@

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486690778 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/partitioner/ColumnValuePartitioner.java ## @@ -0,0 +1,39 @@ +/**

[GitHub] [incubator-pinot] akshayrai opened a new pull request #6001: [TE] entity anomaly logging for ad-hoc debugging

2020-09-10 Thread GitBox
akshayrai opened a new pull request #6001: URL: https://github.com/apache/incubator-pinot/pull/6001 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [incubator-pinot] fx19880617 merged pull request #5987: upgrade to java 11

2020-09-10 Thread GitBox
fx19880617 merged pull request #5987: URL: https://github.com/apache/incubator-pinot/pull/5987 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] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486692906 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/partitioner/TableConfigPartitioner.java ## @@ -0,0 +1,45 @@ +/**

[GitHub] [incubator-pinot] suvodeep-pyne commented on pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
suvodeep-pyne commented on pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991#issuecomment-690791054 Thanks a lot @akshayrai Let me know if you have any other concerns. This is an automated

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

2020-09-10 Thread GitBox
npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r486677843 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/collector/Collector.java ## @@ -0,0 +1,55 @@ +/** + * Licensed to

[GitHub] [incubator-pinot] adriancole commented on pull request #5917: Starts Broker and Server in parallel when using ServiceManager

2020-09-10 Thread GitBox
adriancole commented on pull request #5917: URL: https://github.com/apache/incubator-pinot/pull/5917#issuecomment-690859024 PS this seems it didn't make it to 0.5.0. Can it be added to the next release after that? It will help to not be perpetually on snapshots

[GitHub] [incubator-pinot] fx19880617 commented on pull request #5917: Starts Broker and Server in parallel when using ServiceManager

2020-09-10 Thread GitBox
fx19880617 commented on pull request #5917: URL: https://github.com/apache/incubator-pinot/pull/5917#issuecomment-690867454 Right, this PR will be included in next release(0.6.0) This is an automated message from the Apache

[GitHub] [incubator-pinot] kishoreg merged pull request #5991: [TE] Moved interfaces and datalayer to a new thirdeye-spi module.

2020-09-10 Thread GitBox
kishoreg merged pull request #5991: URL: https://github.com/apache/incubator-pinot/pull/5991 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

[incubator-pinot] branch master updated (e0ed179 -> 0c1d604)

2020-09-10 Thread kishoreg
This is an automated email from the ASF dual-hosted git repository. kishoreg pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from e0ed179 Validate timeColumnName when adding/updating schema/tableConfig (#5966) add 0c1d604 [TE]

[GitHub] [incubator-pinot] fx19880617 opened a new pull request #5997: Fixing website doc hyper links

2020-09-10 Thread GitBox
fx19880617 opened a new pull request #5997: URL: https://github.com/apache/incubator-pinot/pull/5997 ## Description Fixing the hyperlinks in `docs/misc/build-docker.md` This is an automated message from the Apache Git

[incubator-pinot] 01/01: Fixing website doc hyper links

2020-09-10 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch fixing_doc_link in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 5804663f1d853b95cff437ce53d1c8ced7191469 Author: Xiang Fu AuthorDate: Thu Sep 10 00:38:10 2020 -0700

[incubator-pinot] branch fixing_doc_link created (now 5804663)

2020-09-10 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch fixing_doc_link in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 5804663 Fixing website doc hyper links This branch includes the following new commits:

[GitHub] [incubator-pinot] fx19880617 merged pull request #5996: Update links for release 0.5.0 in the website

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

[incubator-pinot] branch master updated: Add 0.5 to website. (#5996)

2020-09-10 Thread xiangfu
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 b2fa2ca Add 0.5 to website. (#5996)