kishoreg commented on a change in pull request #5728:
URL: https://github.com/apache/incubator-pinot/pull/5728#discussion_r461661440
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/routing/segmentselector/OfflineSegmentSelector.java
##
@@ -42,7 +49,14 @@ publi
Jackie-Jiang commented on pull request #5746:
URL: https://github.com/apache/incubator-pinot/pull/5746#issuecomment-664682965
Please take a look at #5760 which is a general fix for this issue.
This is an automated message fro
suvodeep-pyne commented on pull request #5740:
URL: https://github.com/apache/incubator-pinot/pull/5740#issuecomment-665395239
Resolving older comments as per conversation: All yaml template changes have
been removed. And the `lastTimestamp` has been exposed as is with a default
value of 3
shahsank3t commented on pull request #5763:
URL: https://github.com/apache/incubator-pinot/pull/5763#issuecomment-665189824
@kishoreg
Addressed the comment and updated the PR.
+https://imgur.com/GNevDZ0.png"; align="
fx19880617 commented on pull request #5763:
URL: https://github.com/apache/incubator-pinot/pull/5763#issuecomment-665215001
This is awesome!
Two features maybe useful:
1. Show decoded message for compressed metadata. (E.g. large idealStates is
compressed)
2. Search support base
Jackie-Jiang commented on a change in pull request #5728:
URL: https://github.com/apache/incubator-pinot/pull/5728#discussion_r461791755
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/routing/segmentselector/OfflineSegmentSelector.java
##
@@ -42,7 +49,14 @@ p
jackjlli commented on a change in pull request #5760:
URL: https://github.com/apache/incubator-pinot/pull/5760#discussion_r461247207
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/data/recordtransformer/DataTypeTransformer.java
##
@@ -73,10 +80,15 @@ public Gener
guruguha commented on a change in pull request #5718:
URL: https://github.com/apache/incubator-pinot/pull/5718#discussion_r461303906
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/restlet/resources/SegmentStatus.java
##
@@ -0,0 +1,64 @@
+/**
+ * Licensed to t
jackjlli commented on a change in pull request #5764:
URL: https://github.com/apache/incubator-pinot/pull/5764#discussion_r461924349
##
File path:
pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SegmentFetcherAndLoader.java
##
@@ -184,39 +185,44 @@ private boo
mayankshriv opened a new issue #5762:
URL: https://github.com/apache/incubator-pinot/issues/5762
Currently, Pinot supports storing encrypted segments in deep-store (backing
the controller). However, segments on server are still not encrypted. For
sensitive data stored in the cloud, we are
sajjad-moradi commented on issue #5751:
URL:
https://github.com/apache/incubator-pinot/issues/5751#issuecomment-665145249
@mayankshriv sure, I can start the initial design doc.
This is an automated message from the Apache Gi
apucher merged pull request #5750:
URL: https://github.com/apache/incubator-pinot/pull/5750
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
kishoreg commented on issue #5490:
URL:
https://github.com/apache/incubator-pinot/issues/5490#issuecomment-665268832
Any volunteers to take this up. cc @shounakmk219 @KKcorps?
This is an automated message from the Apache Git
mcvsubbu commented on issue #5751:
URL:
https://github.com/apache/incubator-pinot/issues/5751#issuecomment-665112328
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
elonazoulay commented on a change in pull request #5758:
URL: https://github.com/apache/incubator-pinot/pull/5758#discussion_r461731334
##
File path:
pinot-plugins/pinot-input-format/pinot-confluent-avro/src/main/java/org/apache/pinot/plugin/inputformat/avro/confluent/KafkaConfluentSc
Jackie-Jiang commented on a change in pull request #5744:
URL: https://github.com/apache/incubator-pinot/pull/5744#discussion_r461224432
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/StUnionAggregationFunction.java
##
@@ -0,0 +1,137 @@
yupeng9 commented on pull request #5763:
URL: https://github.com/apache/incubator-pinot/pull/5763#issuecomment-665380502
This is awesome!
+1 to the search/filter support on the creation/updated time, to show most
recent state changes.
--
singalravi commented on issue #5753:
URL:
https://github.com/apache/incubator-pinot/issues/5753#issuecomment-664844703
use case:
Most of the queries in our system are for recent time frame (last 7 days)
but we want to retain data for much longer time period. For this, we want to
deploy
Jackie-Jiang commented on pull request #5756:
URL: https://github.com/apache/incubator-pinot/pull/5756#issuecomment-664682845
Please take a look at #5760 which is a general fix for this issue.
This is an automated message fro
mangrrua commented on issue #5753:
URL:
https://github.com/apache/incubator-pinot/issues/5753#issuecomment-665255240
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
yupeng9 commented on a change in pull request #5744:
URL: https://github.com/apache/incubator-pinot/pull/5744#discussion_r461983532
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/StUnionAggregationFunction.java
##
@@ -0,0 +1,137 @@
+/**
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch distinct-count-bitmap
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard cbcc6a0 Adding distinct count support based on bitmap
add cb149be [TE] Make few intern
kishoreg commented on pull request #5758:
URL: https://github.com/apache/incubator-pinot/pull/5758#issuecomment-665172447
@elonazoulay @daniellavoie can you please review this?
This is an automated message from the Apache Git
vincentchenjl opened a new pull request #5767:
URL: https://github.com/apache/incubator-pinot/pull/5767
This PR is frontend to support storing dimensional filters per metric level
for RCA v2 POC. It needs to be merged with #5735 .
--
pradeepgv42 commented on a change in pull request #5758:
URL: https://github.com/apache/incubator-pinot/pull/5758#discussion_r461904284
##
File path:
pinot-plugins/pinot-input-format/pinot-confluent-avro/src/main/java/org/apache/pinot/plugin/inputformat/avro/confluent/KafkaConfluentSc
haibow merged pull request #5731:
URL: https://github.com/apache/incubator-pinot/pull/5731
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
mcvsubbu commented on a change in pull request #5764:
URL: https://github.com/apache/incubator-pinot/pull/5764#discussion_r461914758
##
File path:
pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SegmentFetcherAndLoader.java
##
@@ -184,39 +185,44 @@ private boo
siddharthteotia commented on a change in pull request #5760:
URL: https://github.com/apache/incubator-pinot/pull/5760#discussion_r461235521
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/data/recordtransformer/DataTypeTransformerTest.java
##
@@ -0,0 +1,149 @@
+/*
Jackie-Jiang opened a new pull request #5765:
URL: https://github.com/apache/incubator-pinot/pull/5765
## Description
For DistinctCount aggregation function, we can store dictIds within segment
without fetching the values, and read the values from dictionary before
returning the result
mayankshriv edited a comment on issue #5751:
URL:
https://github.com/apache/incubator-pinot/issues/5751#issuecomment-664700503
Summarizing the offline discussion between @mcvsubbu @snleee @sajjad-moradi
and @mayankshriv:
1. Since there are use cases that may push partial data for th
Jackie-Jiang commented on a change in pull request #5765:
URL: https://github.com/apache/incubator-pinot/pull/5765#discussion_r462055513
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/docvalsets/ProjectionBlockValSet.java
##
@@ -32,32 +35,33 @@
public c
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 a8fbdae Optimize DistinctCount to stor
Jackie-Jiang opened a new pull request #5760:
URL: https://github.com/apache/incubator-pinot/pull/5760
## Description
Enhance DataTypeTransformer to handle nested Collection/Map/Object[]
- Empty `Collection/Map/Object[]` will be treated as `null`
- Single-entry `Collection/Map/O
jackjlli opened a new pull request #5764:
URL: https://github.com/apache/incubator-pinot/pull/5764
## Description
This PR adds retry logic to download segment tar file in pinot server.
Even if the tar file has been downloaded successfully, the file itself could
be corrupted during the
vincentchenjl commented on a change in pull request #5735:
URL: https://github.com/apache/incubator-pinot/pull/5735#discussion_r461989948
##
File path:
thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/v2/rootcause/AbstractRCAModuleConfig.java
###
mayankshriv commented on issue #5751:
URL:
https://github.com/apache/incubator-pinot/issues/5751#issuecomment-664700503
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
yupeng9 opened a new pull request #5761:
URL: https://github.com/apache/incubator-pinot/pull/5761
## Description
Address issue https://github.com/apache/incubator-pinot/issues/5022
The `*shaded.pom` files were generated because of the application of shade
plugin on the parent modules.
mayankshriv commented on issue #5753:
URL:
https://github.com/apache/incubator-pinot/issues/5753#issuecomment-664739660
While periodically moving the segments from realtime to offline is a good
idea, in many cases, it would also benefit to have the segment merge/rollup
performed before mo
fx19880617 commented on issue #5753:
URL:
https://github.com/apache/incubator-pinot/issues/5753#issuecomment-665344520
> That will be great! If merge/rollup can be applied(@mayankshriv 's
suggestion), users can have a lot of flexibility. Because generally, realtime
segments represents min
apucher commented on pull request #5750:
URL: https://github.com/apache/incubator-pinot/pull/5750#issuecomment-664708535
Hi folks - ***seems*** to be working, but impossible to unit test. If you
see any issues please revert immediately. @akshayrai @jihaozh
---
siddharthteotia commented on pull request #5766:
URL: https://github.com/apache/incubator-pinot/pull/5766#issuecomment-665457815
This is fantastic. Sorry, by mistake approved it. Will review by tomorrow
This is an automated m
Jackie-Jiang commented on a change in pull request #5760:
URL: https://github.com/apache/incubator-pinot/pull/5760#discussion_r461237235
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/data/recordtransformer/DataTypeTransformerTest.java
##
@@ -0,0 +1,149 @@
+/**
+
Jackie-Jiang merged pull request #5760:
URL: https://github.com/apache/incubator-pinot/pull/5760
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
jihaozh merged pull request #5752:
URL: https://github.com/apache/incubator-pinot/pull/5752
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
Jackie-Jiang commented on pull request #5760:
URL: https://github.com/apache/incubator-pinot/pull/5760#issuecomment-665204779
@jackjlli Addressed all the comments
This is an automated message from the Apache Git Service.
To r
akshayrai commented on a change in pull request #5740:
URL: https://github.com/apache/incubator-pinot/pull/5740#discussion_r461792659
##
File path: thirdeye/thirdeye-frontend/app/utils/yaml-tools.js
##
@@ -29,6 +28,20 @@ rules:
type: DATA_SLA # Alert if data i
shahsank3t opened a new pull request #5763:
URL: https://github.com/apache/incubator-pinot/pull/5763
## Description
This PR is to add the Zookeeper Browser module into existing UI.
but we want to retain data for much longer time period. For this, we want to
depl
jackjlli closed pull request #5756:
URL: https://github.com/apache/incubator-pinot/pull/5756
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
jackjlli closed pull request #5746:
URL: https://github.com/apache/incubator-pinot/pull/5746
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
mcvsubbu commented on a change in pull request #5718:
URL: https://github.com/apache/incubator-pinot/pull/5718#discussion_r457517934
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java
##
@@ -485,4 +493,91 @@
mcvsubbu commented on a change in pull request #5764:
URL: https://github.com/apache/incubator-pinot/pull/5764#discussion_r462422577
##
File path:
pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SegmentFetcherAndLoader.java
##
@@ -184,39 +186,48 @@ private boo
yupeng9 commented on pull request #5763:
URL: https://github.com/apache/incubator-pinot/pull/5763#issuecomment-665765905
> > This is awesome!
> > +1 to the search/filter support on the creation/updated time, to show
most recent state changes.
>
> @yupeng9, can you elaborate on thi
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch add-untar-failure-server-meter
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit ef93477b20f84ffe57e0ee39481d3c759d5f86b7
Author: Jack Li(Analytics Engineering)
AuthorDat
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch add-untar-failure-server-meter
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at ef93477 Add untar failure server meter
This branch includes the following new commit
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 a8fbdae Optimize DistinctCount to store dictIds within segment (#5765)
add 0c4a673 [TE] add enpoints t
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 0c4a673 [TE] add enpoints to save dimension filter for RCA v2 POC
(#5735)
add ba9e14f [TE] add auto-l
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 a8fbdae Optimize DistinctCount to store dictIds within segment (#5765)
add 0c4a673 [TE] add enpoints t
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 0c4a673 [TE] add enpoints to save dimension filter for RCA v2 POC
(#5735)
add ba9e14f [TE] add auto-l
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a commit to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 39fcef438b63f71cfbc4d083972bff2a92b8b22b
Author: Jihao Zhang
AuthorDate: Wed Jul 22 14:22:33 2020
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a commit to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit b4e073314c403674437d6b6df4f0a4be00d84a64
Author: Jihao Zhang
AuthorDate: Wed Jul 29 16:11:52 2020
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a commit to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit d37df19fe9baa1633499a5db30c88a75e851c212
Author: Jihao Zhang
AuthorDate: Wed Jul 29 16:06:53 2020
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a change to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at b4e0733 add java docs
This branch includes the following new commits:
new 39fcef4 new
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a commit to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 2a2be355b869110ebcfcf8e0c8af335a03c39415
Author: Jihao Zhang
AuthorDate: Wed Jul 22 14:22:33 2020
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a change to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard b4e0733 add java docs
discard d37df19 anomaly searcher
discard 39fcef4 new anomalies endpo
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a commit to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 94f3757d1913057bc387ca02a088faaae7a7a32f
Author: Jihao Zhang
AuthorDate: Wed Jul 22 14:22:33 2020
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a change to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 2a2be35 new anomalies endpoint
add c9975c5 [TE] support optional 'name' field in data so
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a change to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 94f3757 new anomalies endpoint
new ffb0c3b new anomalies endpoint
This update added new
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a commit to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit ffb0c3b6d8e35d635f86bf5a4b8641ad2d45a561
Author: Jihao Zhang
AuthorDate: Wed Jul 22 14:22:33 2020
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a commit to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/anomalies-pagination by this
push:
new 48e937f un
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a change to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 48e937f unit tests
new a9d4aea unit tests
This update added new revisions after undoing
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a commit to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit a9d4aea4e27122f9b103b4bdb813d97e5c77f8af
Author: Jihao Zhang
AuthorDate: Wed Jul 29 16:52:39 2020
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 1f69041 [Part 2] Add geo support - ad
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch distinct-count-bitmap
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard a68bfba Add DistinctCountBitmap aggregation function
add 0c4a673 [TE] add enpoints to
This is an automated email from the ASF dual-hosted git repository.
akshayrai09 pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 1f69041 [Part 2] Add geo support - add a geo aggregate function
st_union (#5744)
add b1ab891 [
85 matches
Mail list logo
