This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch manual-authorization-annotation
in repository https://gitbox.apache.org/repos/asf/pinot.git
discard f67ac4522d controller auth interface cleanup
discard d38c84cd02 annotation fix
discard 1373
codecov-commenter commented on PR #9257:
URL: https://github.com/apache/pinot/pull/9257#issuecomment-1221186810
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9257?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
sajjad-moradi commented on PR #9249:
URL: https://github.com/apache/pinot/pull/9249#issuecomment-1221184780
> We had discussed about a metadata cache. is that something that is coming
in another PR (and this one is to get us through the immediate problem)?
>
> thanks
Yes that w
klsince commented on code in PR #9242:
URL: https://github.com/apache/pinot/pull/9242#discussion_r950618560
##
pinot-core/src/main/java/org/apache/pinot/core/operator/combine/BaseCombineOperator.java:
##
@@ -118,6 +119,9 @@ public void runJob() {
IntermediateResultsBlock me
klsince commented on code in PR #9242:
URL: https://github.com/apache/pinot/pull/9242#discussion_r950618452
##
pinot-common/src/main/java/org/apache/pinot/common/exception/QueryException.java:
##
@@ -53,6 +53,7 @@ public static void setMaxLinesOfStackTrace(int
maxLinesOfStackTr
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch manual-authorization-annotation
in repository https://gitbox.apache.org/repos/asf/pinot.git
from d38c84cd02 annotation fix
add f67ac4522d controller auth interface cleanup
No new revis
Jackie-Jiang commented on code in PR #9242:
URL: https://github.com/apache/pinot/pull/9242#discussion_r950611815
##
pinot-common/src/main/java/org/apache/pinot/common/exception/QueryException.java:
##
@@ -53,6 +53,7 @@ public static void setMaxLinesOfStackTrace(int
maxLinesOfSt
klsince opened a new pull request, #9257:
URL: https://github.com/apache/pinot/pull/9257
Fix a UT that only shows up on host with more cores. The instance running GH
Action couldn't catch it. I believe it has less than 6 cores.
--
This is an automated message from the Apache Git Servi
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 a5a83aa81b Add more failure info in
RealtimeNonR
Jackie-Jiang merged PR #9254:
URL: https://github.com/apache/pinot/pull/9254
--
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.a
navina commented on code in PR #9244:
URL: https://github.com/apache/pinot/pull/9244#discussion_r950603573
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/LLRealtimeSegmentDataManager.java:
##
@@ -1440,6 +1440,17 @@ private void fetchLatestStreamOffset()
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 7af3edbd52 Fix flakiness and cleanup SegmentLine
xiangfu0 merged PR #9256:
URL: https://github.com/apache/pinot/pull/9256
--
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.apach
codecov-commenter commented on PR #9252:
URL: https://github.com/apache/pinot/pull/9252#issuecomment-1221164092
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9252?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
codecov-commenter commented on PR #9256:
URL: https://github.com/apache/pinot/pull/9256#issuecomment-1221153569
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9256?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch manual-authorization-annotation
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 13736c79f9 linter fix
add d38c84cd02 annotation fix
No new revisions were added by this
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 0a272b2956 Remove currentDataTableVersion from h
xiangfu0 merged PR #9255:
URL: https://github.com/apache/pinot/pull/9255
--
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.apach
codecov-commenter commented on PR #9254:
URL: https://github.com/apache/pinot/pull/9254#issuecomment-1221129016
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9254?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
xiangfu0 opened a new pull request, #9255:
URL: https://github.com/apache/pinot/pull/9255
This is no longer needed as presto/trino side has upgraded the DataTable
version
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch manual-authorization-annotation
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 5ff57efb8c linter fix
add 13736c79f9 linter fix
No new revisions were added by this upd
vvivekiyer commented on PR #9247:
URL: https://github.com/apache/pinot/pull/9247#issuecomment-1221118277
LGTM.
--
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 unsubscri
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch manual-authorization-annotation
in repository https://gitbox.apache.org/repos/asf/pinot.git
from ee391e918b add @ManualAuthorization annotation for non-standard
endpoints
add 5ff57efb8
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 e6c2f5bd3b Refactor upsert quickstarts (#9246)
e
xiangfu0 merged PR #9246:
URL: https://github.com/apache/pinot/pull/9246
--
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.apach
xiangfu0 commented on code in PR #9246:
URL: https://github.com/apache/pinot/pull/9246#discussion_r950573035
##
pinot-tools/src/main/java/org/apache/pinot/tools/PartialUpsertQuickStart.java:
##
@@ -123,7 +67,10 @@ public void execute()
printStatus(Color.CYAN, "Query : " + q
Jackie-Jiang opened a new pull request, #9254:
URL: https://github.com/apache/pinot/pull/9254
For #9253
--
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-m
Jackie-Jiang opened a new issue, #9253:
URL: https://github.com/apache/pinot/issues/9253
```
Error:
RealtimeNonReplicaGroupSegmentAssignmentTest.testRelocateCompletedSegments:231
expected [true] but found [false]
```
It tests the behavior of Helix auto rebalance strategy
(`
codecov-commenter commented on PR #9251:
URL: https://github.com/apache/pinot/pull/9251#issuecomment-1221089837
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9251?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
PrachiKhobragade commented on code in PR #9249:
URL: https://github.com/apache/pinot/pull/9249#discussion_r950550273
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeConsumptionRateManager.java:
##
@@ -131,4 +144,22 @@ public void throttle(int numM
Jackie-Jiang commented on code in PR #9247:
URL: https://github.com/apache/pinot/pull/9247#discussion_r950548988
##
pinot-broker/src/main/java/org/apache/pinot/broker/api/HttpRequesterIdentity.java:
##
@@ -43,4 +45,31 @@ public String getEndpointUrl() {
public void setEndpoin
apucher opened a new pull request, #9252:
URL: https://github.com/apache/pinot/pull/9252
Several endpoints in pinot aren't served well by the AuthFilter's heuristics
for extracting table names. In particular, this includes `/tableConfigs` and
`/schema` endpoints that accept opaque payloads
apucher opened a new pull request, #9251:
URL: https://github.com/apache/pinot/pull/9251
Minor refactor of `PinotAccessControlUserRestletResource` to rely on the
AuthFilter's default behavior rather than performing authorization manually.
CC: original author @INNOCENT-BOY, any subtle
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a commit to branch manual-authorization-annotation
in repository https://gitbox.apache.org/repos/asf/pinot.git
commit ee391e918b188e6e7e6dade5f11157b4c04cf1d4
Author: Alexander Pucher
AuthorDate: Fri Aug 19 13:21:
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch manual-authorization-annotation
in repository https://gitbox.apache.org/repos/asf/pinot.git
at ee391e918b add @ManualAuthorization annotation for non-standard
endpoints
This branch inclu
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a commit to branch use-default-authorization-for-zk-user-config
in repository https://gitbox.apache.org/repos/asf/pinot.git
commit 76184bbfe29b845de0975451af423c0319d07126
Author: Alexander Pucher
AuthorDate: Fri
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch use-default-authorization-for-zk-user-config
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 76184bbfe2 use default authorization behavior for zk user config
This branch in
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 6e6e0a4e86 Fix NPE in gap-fill when order-by
siddharthteotia merged PR #9250:
URL: https://github.com/apache/pinot/pull/9250
--
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...@pino
snleee commented on code in PR #9249:
URL: https://github.com/apache/pinot/pull/9249#discussion_r950515885
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeConsumptionRateManager.java:
##
@@ -85,17 +88,27 @@ public ConsumptionRateLimiter
createRat
sajjad-moradi commented on code in PR #9249:
URL: https://github.com/apache/pinot/pull/9249#discussion_r950500727
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeConsumptionRateManager.java:
##
@@ -131,4 +144,22 @@ public void throttle(int numMsgs
codecov-commenter commented on PR #9250:
URL: https://github.com/apache/pinot/pull/9250#issuecomment-1221018092
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9250?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
sajjad-moradi commented on code in PR #9249:
URL: https://github.com/apache/pinot/pull/9249#discussion_r950491202
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeConsumptionRateManager.java:
##
@@ -85,17 +88,27 @@ public ConsumptionRateLimiter
cr
tanmesh commented on code in PR #9184:
URL: https://github.com/apache/pinot/pull/9184#discussion_r950488823
##
pinot-core/src/test/java/org/apache/pinot/core/operator/transform/function/ExtractTransformFunctionTest.java:
##
@@ -0,0 +1,60 @@
+package org.apache.pinot.core.operato
snleee commented on code in PR #9249:
URL: https://github.com/apache/pinot/pull/9249#discussion_r950487153
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeConsumptionRateManager.java:
##
@@ -131,4 +144,22 @@ public void throttle(int numMsgs) {
tanmesh commented on PR #9184:
URL: https://github.com/apache/pinot/pull/9184#issuecomment-1221006954
PTAL @Jackie-Jiang when you get a chance. Thanks!
--
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 g
navina commented on PR #9244:
URL: https://github.com/apache/pinot/pull/9244#issuecomment-1220985810
> I agree that this would likely be better as a table level check and maybe
even happen continuously through the lifecycle of the server so the broker is
aware what servers are fresh enough
Jackie-Jiang closed issue #8819: Updating schema with new MV column causes
index failures and stops consumption
URL: https://github.com/apache/pinot/issues/8819
--
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
tanmesh commented on code in PR #9184:
URL: https://github.com/apache/pinot/pull/9184#discussion_r950434935
##
pinot-common/src/main/java/org/apache/pinot/common/function/TransformFunctionType.java:
##
@@ -96,6 +96,8 @@ public enum TransformFunctionType {
LOOKUP("lookUp"),
tanmesh commented on code in PR #9184:
URL: https://github.com/apache/pinot/pull/9184#discussion_r950434236
##
pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java:
##
@@ -752,6 +754,8 @@ private static Expression
compileFunctionExpression(SqlBasicCall
tanmesh commented on code in PR #9184:
URL: https://github.com/apache/pinot/pull/9184#discussion_r950433406
##
pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/PinotClientTransport.java:
##
@@ -42,4 +42,4 @@ Future executeQueryAsync(String
brokerAddress, Re
Jackie-Jiang commented on code in PR #9244:
URL: https://github.com/apache/pinot/pull/9244#discussion_r950412090
##
pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java:
##
@@ -385,6 +385,13 @@ public static class Server {
"pinot.server.starter.enabl
codecov-commenter commented on PR #9249:
URL: https://github.com/apache/pinot/pull/9249#issuecomment-1220908246
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/9249?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar
sajjad-moradi opened a new pull request, #9249:
URL: https://github.com/apache/pinot/pull/9249
Setting up consumption rate limit requires knowing the stream partition
count so that each partition rate limit can be calculated based on the topic
consumption rate limit config. Connecting to st
yupeng9 commented on code in PR #9246:
URL: https://github.com/apache/pinot/pull/9246#discussion_r950345665
##
pinot-tools/src/main/java/org/apache/pinot/tools/PartialUpsertQuickStart.java:
##
@@ -123,7 +67,10 @@ public void execute()
printStatus(Color.CYAN, "Query : " + q2
jadami10 commented on PR #9244:
URL: https://github.com/apache/pinot/pull/9244#issuecomment-1220771563
Hey @navina, thank you for looking!
I agree that this would likely be better as a table level check and maybe
even happen continuously through the lifecycle of the server so the brok
walterddr commented on code in PR #9245:
URL: https://github.com/apache/pinot/pull/9245#discussion_r950261062
##
pinot-plugins/pinot-batch-ingestion/pinot-batch-ingestion-standalone/src/main/java/org/apache/pinot/plugin/ingestion/batch/standalone/SegmentGenerationJobRunner.java:
###
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/org.apache.maven.plugins-maven-enforcer-plugin-3.1.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
at a7a52f69ac Bump maven-enforcer-plugin from 3.0.0-M3
dependabot[bot] opened a new pull request, #9248:
URL: https://github.com/apache/pinot/pull/9248
Bumps [maven-enforcer-plugin](https://github.com/apache/maven-enforcer) from
3.0.0-M3 to 3.1.0.
Release notes
Sourced from https://github.com/apache/maven-enforcer/releases";>maven-enfo
gortiz commented on code in PR #8979:
URL: https://github.com/apache/pinot/pull/8979#discussion_r950061067
##
pinot-core/src/main/java/org/apache/pinot/core/operator/blocks/IntermediateResultsBlock.java:
##
@@ -80,9 +81,8 @@ public IntermediateResultsBlock() {
/**
* Const
xiangfu0 commented on issue #9147:
URL: https://github.com/apache/pinot/issues/9147#issuecomment-1220478638
I've added a nightly build image for arm64:
https://github.com/apachepinot/pinot-fork/actions/workflows/nightly-build-pinot-arm64-platform-docker-image.yml
It takes about 5 h
61 matches
Mail list logo