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
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
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
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
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
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
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 @@
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
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
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 @@
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
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
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
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
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
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 @@
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 @@
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 @@
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
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
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
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
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
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 @@
+/**
+ *
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 @@
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}
+
+
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 @@
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
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
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
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 @@
+/**
+
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 @@
+/**
+ *
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 @@
+/**
+ *
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 @@
+/**
+ *
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}
+
+
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
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
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}
+
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
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 @@
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 @@
+/**
+
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 @@
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 @@
+/**
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
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
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 @@
+/**
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
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
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
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
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
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]
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
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
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:
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
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)
57 matches
Mail list logo