codecov-commenter commented on PR #11195:
URL: https://github.com/apache/pinot/pull/11195#issuecomment-1653018230
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11195?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
KKcorps opened a new pull request, #11195:
URL: https://github.com/apache/pinot/pull/11195
* This will allow us to simply extend this test for more usecases
* The primary difference in most such test is what data is loaded and how to
check if data is loaded or not. Hence seperating out th
shenyu0127 commented on code in PR #11078:
URL: https://github.com/apache/pinot/pull/11078#discussion_r1275660929
##
pinot-core/src/main/java/org/apache/pinot/core/operator/filter/BaseColumnFilterOperator.java:
##
@@ -0,0 +1,74 @@
+/**
+ * Licensed to the Apache Software Foundat
codecov-commenter commented on PR #11193:
URL: https://github.com/apache/pinot/pull/11193#issuecomment-1652802466
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11193?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
Jackie-Jiang commented on code in PR #11078:
URL: https://github.com/apache/pinot/pull/11078#discussion_r1275644261
##
pinot-core/src/main/java/org/apache/pinot/core/operator/filter/BaseColumnFilterOperator.java:
##
@@ -0,0 +1,74 @@
+/**
+ * Licensed to the Apache Software Found
shenyu0127 commented on PR #11078:
URL: https://github.com/apache/pinot/pull/11078#issuecomment-1652790028
@Jackie-Jiang I merged this branch with updated master. Please take another
look.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 1c7fde0c5a safely multiply integers to prevent overflow (#11186)
add 61dcea6b71 Rbac impl (#11016)
No new revisio
Jackie-Jiang merged PR #11016:
URL: https://github.com/apache/pinot/pull/11016
--
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 to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 1c7fde0c5a safely multiply integers to prevent ov
Jackie-Jiang merged PR #11186:
URL: https://github.com/apache/pinot/pull/11186
--
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 to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
Jackie-Jiang opened a new issue, #11194:
URL: https://github.com/apache/pinot/issues/11194
Currently there is no way to know the untarred segment size when uploading
segment in METADATA mode because controller doesn't download the segment for
performance concern. With #11193 we use tarred s
Jackie-Jiang opened a new pull request, #11193:
URL: https://github.com/apache/pinot/pull/11193
Currently there is no way to get the untarred segment size for METADATA push
type. Use the tarred segment size as an approximation so that storage quota can
still be applied.
--
This is an aut
codecov-commenter commented on PR #11192:
URL: https://github.com/apache/pinot/pull/11192#issuecomment-1652764427
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11192?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
codecov-commenter commented on PR #11190:
URL: https://github.com/apache/pinot/pull/11190#issuecomment-1652747719
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11190?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
codecov-commenter commented on PR #11191:
URL: https://github.com/apache/pinot/pull/11191#issuecomment-1652747586
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11191?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
Jackie-Jiang opened a new pull request, #11192:
URL: https://github.com/apache/pinot/pull/11192
- When multiple `loadLookupTable()` happens at the same time, only keep one
and abort the others because only one will be kept in the end
- For `MemOptimisedDimensionTable`, only read the prima
xiangfu0 opened a new pull request, #11191:
URL: https://github.com/apache/pinot/pull/11191
- support st-* functions in multi-stage engine
--
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 to the spec
walterddr opened a new pull request, #11190:
URL: https://github.com/apache/pinot/pull/11190
this fixes: #11189
--
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 to the specific comment.
To unsubscr
walterddr opened a new issue, #11189:
URL: https://github.com/apache/pinot/issues/11189
this is most likely due to `FunctionRegistry` registering multiple
`@ScalarFunction` names.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new d83d1e8713 Non-leaf filter operators NULL support
Jackie-Jiang merged PR #11185:
URL: https://github.com/apache/pinot/pull/11185
--
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 to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
codecov-commenter commented on PR #11188:
URL: https://github.com/apache/pinot/pull/11188#issuecomment-1652661966
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11188?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
shenyu0127 opened a new pull request, #11188:
URL: https://github.com/apache/pinot/pull/11188
(no comment)
--
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 to the specific comment.
To unsubscribe, e
xiangfu0 opened a new issue, #11187:
URL: https://github.com/apache/pinot/issues/11187
Assume there is a table A on serve-0 with only 1 segment, 1 replica.
During mailbox assignment, the routing table gives routing as no
serverInstance mapping, and one unavailable segment.
It turns out
xiangfu0 closed issue #10864: [multistage] server restart causes broker unable
to find server
URL: https://github.com/apache/pinot/issues/10864
--
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 to the sp
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/pinot.git
from a86ba9c42f Add brokerId and brokerReduceTimeMs to the broker response
stats (#11142)
add a0ff2e8420 [multistage]
codecov-commenter commented on PR #11186:
URL: https://github.com/apache/pinot/pull/11186#issuecomment-1652492457
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11186?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
xiangfu0 merged PR #7:
URL: https://github.com/apache/pinot/pull/7
--
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 to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot.apa
priyen commented on code in PR #10926:
URL: https://github.com/apache/pinot/pull/10926#discussion_r1275464930
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/indexsegment/mutable/MutableSegmentImpl.java:
##
@@ -1211,17 +1230,29 @@ private static Map> fromAggr
priyen commented on code in PR #10926:
URL: https://github.com/apache/pinot/pull/10926#discussion_r1275450645
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/aggregator/DistinctCountHLLValueAggregator.java:
##
@@ -28,11 +33,31 @@
public class DistinctCount
itschrispeck opened a new pull request, #11186:
URL: https://github.com/apache/pinot/pull/11186
We've observed this exception, blocking ingestion:
```
j.l.IllegalArgumentException: Illegal memory allocation -2116790016 for
segment __0__50__20230724T1340Z column
__0__50__20230724T1340Z
codecov-commenter commented on PR #11185:
URL: https://github.com/apache/pinot/pull/11185#issuecomment-1652344196
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11185?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
soumitra-st commented on code in PR #11016:
URL: https://github.com/apache/pinot/pull/11016#discussion_r1275368362
##
pinot-core/src/main/java/org/apache/pinot/core/auth/Actions.java:
##
@@ -0,0 +1,144 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new a86ba9c42f Add brokerId and brokerReduceTimeMs to
Jackie-Jiang merged PR #11142:
URL: https://github.com/apache/pinot/pull/11142
--
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 to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
shenyu0127 opened a new pull request, #11185:
URL: https://github.com/apache/pinot/pull/11185
This PR includes the following changes:
- Adds `getTrues()`, `getNulls()`, and `getFalses()` to `BaseFilterOperator`.
- Adds NULL support for `AndFilterOperator`, `OrFilterOperator`, and
`NotF
Jackie-Jiang commented on code in PR #11016:
URL: https://github.com/apache/pinot/pull/11016#discussion_r1275335845
##
pinot-core/src/main/java/org/apache/pinot/core/auth/Actions.java:
##
@@ -0,0 +1,144 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * o
Jackie-Jiang commented on issue #11182:
URL: https://github.com/apache/pinot/issues/11182#issuecomment-1652275677
Thanks for the research and detailed explanation. @snleee @swaminathanmanish
Can you help take a look at this?
--
This is an automated message from the Apache Git Service.
To
fritzb commented on issue #6768:
URL: https://github.com/apache/pinot/issues/6768#issuecomment-1652264822
I have a simplified DBT use case, which I believe is the first step to
involve Pinot in DBT. I'd like to start the discussion about whether this idea
is feasible as a short project. The
Jackie-Jiang merged PR #10463:
URL: https://github.com/apache/pinot/pull/10463
--
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 to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
Jackie-Jiang closed issue #6912: Segment compaction for upsert real-time tables
URL: https://github.com/apache/pinot/issues/6912
--
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 to the specific comment.
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 4d72eb5a58 Segment compaction for upsert real-tim
Jackie-Jiang closed issue #11178: Delete format support in Explain physical
plan sql queries
URL: https://github.com/apache/pinot/issues/11178
--
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 to the spe
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 09bc525598 removing the format support in explain
Jackie-Jiang merged PR #11179:
URL: https://github.com/apache/pinot/pull/11179
--
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 to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot
Jackie-Jiang closed issue #10793: Ensure min/max value are always generated in
the segment metadata
URL: https://github.com/apache/pinot/issues/10793
--
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 to
codecov-commenter commented on PR #11184:
URL: https://github.com/apache/pinot/pull/11184#issuecomment-1652176998
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11184?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new e87a05045d Fix compat test by adding -am flag to
xiangfu0 merged PR #11181:
URL: https://github.com/apache/pinot/pull/11181
--
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 to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot.apa
jadami10 opened a new pull request, #11184:
URL: https://github.com/apache/pinot/pull/11184
This is authored by @priyen and is a small `feature` addition.
This updates the `tenants/{tenantName}/tables` API to support also
specifying type to return tagged tables by the broker tenant. T
robertzych commented on PR #10463:
URL: https://github.com/apache/pinot/pull/10463#issuecomment-1652004263
@Jackie-Jiang I've applied [Xiang's
fix](https://github.com/apache/pinot/pull/11181) for the compatibility test
issue. Now the only failure appears to be with an unrelated flaky unit t
robertzych commented on code in PR #11181:
URL: https://github.com/apache/pinot/pull/11181#discussion_r1274929986
##
compatibility-verifier/checkoutAndBuild.sh:
##
@@ -99,14 +99,14 @@ function build() {
if [ $? -eq 0 ]; then break; fi
if [ $i -eq $maxRetry ]; then exit
saurabhd336 commented on code in PR #11128:
URL: https://github.com/apache/pinot/pull/11128#discussion_r1274842035
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTenantRestletResource.java:
##
@@ -103,6 +112,9 @@ public class PinotTenantRestletR
saurabhd336 commented on code in PR #11128:
URL: https://github.com/apache/pinot/pull/11128#discussion_r1274829847
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/tenant/DefaultTenantRebalancer.java:
##
@@ -0,0 +1,193 @@
+/**
+ * Licensed to th
saurabhd336 commented on code in PR #11128:
URL: https://github.com/apache/pinot/pull/11128#discussion_r1274817803
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/tenant/DefaultTenantRebalancer.java:
##
@@ -0,0 +1,193 @@
+/**
+ * Licensed to th
saurabhd336 commented on code in PR #11128:
URL: https://github.com/apache/pinot/pull/11128#discussion_r1274817803
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/tenant/DefaultTenantRebalancer.java:
##
@@ -0,0 +1,193 @@
+/**
+ * Licensed to th
Aravind-Suresh opened a new pull request, #11183:
URL: https://github.com/apache/pinot/pull/11183
Fixes #10916.
**Approach**: This PR adds a new Reload Table API to reload table
configurations. More details can be found in #10916.
**Test plan**: TODO (will update the draft PR i
This is an automated email from the ASF dual-hosted git repository.
kharekartik pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new d9c64f347a Add Integration test for Upsert P
KKcorps merged PR #11160:
URL: https://github.com/apache/pinot/pull/11160
--
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 to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot.apac
shounakmk219 commented on code in PR #11128:
URL: https://github.com/apache/pinot/pull/11128#discussion_r1274574539
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/tenant/DefaultTenantRebalancer.java:
##
@@ -0,0 +1,193 @@
+/**
+ * Licensed to t
VisarBuza opened a new issue, #11182:
URL: https://github.com/apache/pinot/issues/11182
We are using Apache Pinot with S3 as our deep store, specifically, Ceph as
an S3 compatible storage. We are encountering an issue during the segment
finalization process when a copy request is issued to
61 matches
Mail list logo
