xiangfu0 opened a new issue, #11901:
URL: https://github.com/apache/pinot/issues/11901
This pops up as a requirement when quickstart join query might generates
many rows. It may blow up broker memory as it tries to keep all the rows in
memory then return it back.
The goal of this PR
ksnijjer opened a new issue, #11900:
URL: https://github.com/apache/pinot/issues/11900
Currently in Pinot Query Console if there is any error you need to manually
click on button to view the details.
This seems counter intuitive as in such a scenario you do want to see the
full details
xiangfu0 merged PR #11868:
URL: https://github.com/apache/pinot/pull/11868
--
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
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 22f41716fd Bump net.minidev:json-smart from 2.4.10 to 2.5.0 (#11875)
add 8e8bd328a6 11085: Pre PinotConfig common
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 9022eaec2b Bump org.yaml:snakeyaml from 2.0 to 2.2 (#11876)
add 22f41716fd Bump net.minidev:json-smart from 2.4.1
xiangfu0 merged PR #11875:
URL: https://github.com/apache/pinot/pull/11875
--
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
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch dependabot/maven/net.minidev-json-smart-2.5.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 1922f65993 Bump net.minidev:json-smart from 2.4.10 to 2.5.0
The revisions that
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch dependabot/maven/org.yaml-snakeyaml-2.2
in repository https://gitbox.apache.org/repos/asf/pinot.git
was b14514eb8f Bump org.yaml:snakeyaml from 2.0 to 2.2
The revisions that were on this b
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 9022eaec2b Bump org.yaml:snakeyaml from 2.0 to 2
xiangfu0 merged PR #11876:
URL: https://github.com/apache/pinot/pull/11876
--
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
xiangfu0 commented on PR #11886:
URL: https://github.com/apache/pinot/pull/11886#issuecomment-1783690777
> @xiangfu0 yep! The goal of this PR would be to support exactly what’s
described in #11899. Happy to collaborate, just let me know!
Great!
--
This is an automated message from
egalpin commented on PR #11886:
URL: https://github.com/apache/pinot/pull/11886#issuecomment-1783684115
@xiangfu0 yep! The goal of this PR would be to support exactly what’s
described in https://github.com/apache/pinot/issues/11899. Happy to
collaborate, just let me know!
--
This is an
xiangfu0 commented on PR #11886:
URL: https://github.com/apache/pinot/pull/11886#issuecomment-1783676206
not sure if this is related: https://github.com/apache/pinot/issues/11899
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
xiangfu0 opened a new issue, #11899:
URL: https://github.com/apache/pinot/issues/11899
E.g.
```
SELECT
sum(m1) FILTER d1=1 ,
count(*)
WHERE d2=2
GROUP BY d3
```
For the above query, though
```
SELECT
count(*)
WHERE d2=2
GROUP BY d3
soumitra-st commented on code in PR #11859:
URL: https://github.com/apache/pinot/pull/11859#discussion_r1375134415
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##
@@ -268,20 +268,28 @@ public String recommendConfig(S
somandal commented on code in PR #11578:
URL: https://github.com/apache/pinot/pull/11578#discussion_r1375132678
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/instance/InstanceAssignmentDriver.java:
##
@@ -60,19 +60,31 @@ public InstanceParti
soumitra-st commented on code in PR #11859:
URL: https://github.com/apache/pinot/pull/11859#discussion_r1375134415
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##
@@ -268,20 +268,28 @@ public String recommendConfig(S
swaminathanmanish commented on code in PR #11887:
URL: https://github.com/apache/pinot/pull/11887#discussion_r1375134170
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/CompositeTransformer.java:
##
@@ -69,12 +70,22 @@ public class Composite
soumitra-st commented on code in PR #11859:
URL: https://github.com/apache/pinot/pull/11859#discussion_r1375133871
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##
@@ -268,20 +268,28 @@ public String recommendConfig(S
deemoliu commented on code in PR #11791:
URL: https://github.com/apache/pinot/pull/11791#discussion_r1375104912
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java:
##
@@ -121,6 +121,32 @@ public List getPrimaryKeyCo
snleee commented on PR #11890:
URL: https://github.com/apache/pinot/pull/11890#issuecomment-1783587730
@xiangfu0 added the release document. I will update the documentation as
well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
This is an automated email from the ASF dual-hosted git repository.
snlee pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 8c870b7655 Bump browserify-sign in
/pinot-controller/src/main/resources (#11896)
add 60edf49db6 Add 'jsonKey' opti
snleee merged PR #11890:
URL: https://github.com/apache/pinot/pull/11890
--
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 #11897:
URL: https://github.com/apache/pinot/pull/11897#issuecomment-1783585363
We're currently processing your upload. This comment will be updated when
the results are available.
--
This is an automated message from the Apache Git Service.
To respond t
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch
dependabot/npm_and_yarn/pinot-controller/src/main/resources/browserify-sign-4.2.2
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 9a9c914804 Bump browserify-sign in /pinot-
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 ce25936323 Radomize server port to avoid port already bind issue
(#11861)
add 8c870b7655 Bump browserify-sign in
xiangfu0 merged PR #11896:
URL: https://github.com/apache/pinot/pull/11896
--
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
jasperjiaguo commented on code in PR #11578:
URL: https://github.com/apache/pinot/pull/11578#discussion_r1375083118
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java:
##
@@ -1735,20 +1735,29 @@ private void assignInstances(Ta
xiangfu0 commented on code in PR #11868:
URL: https://github.com/apache/pinot/pull/11868#discussion_r1375082035
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -101,10 +101,16 @@ private static Iterable
getIterable(Iterator keys) {
xiangfu0 commented on PR #11890:
URL: https://github.com/apache/pinot/pull/11890#issuecomment-1783542258
Better to document and add a release notes
--
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
xiangfu0 opened a new pull request, #11897:
URL: https://github.com/apache/pinot/pull/11897
- Support auth for QueryRunner
- Some refactor
--
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 s
codecov-commenter commented on PR #11896:
URL: https://github.com/apache/pinot/pull/11896#issuecomment-1783516530
We're currently processing your upload. This comment will be updated when
the results are available.
--
This is an automated message from the Apache Git Service.
To respond t
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/npm_and_yarn/pinot-controller/src/main/resources/browserify-sign-4.2.2
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 9a9c914804 Bump browserify-sign in /pin
dependabot[bot] opened a new pull request, #11896:
URL: https://github.com/apache/pinot/pull/11896
Bumps
[browserify-sign](https://github.com/crypto-browserify/browserify-sign) from
4.2.1 to 4.2.2.
Changelog
Sourced from https://github.com/browserify/browserify-sign/blob/main/CHAN
xiangfu0 closed pull request #11888: Fix leaf query generation bug when there
are multiple LogicalFilters
URL: https://github.com/apache/pinot/pull/11888
--
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
somandal commented on code in PR #11578:
URL: https://github.com/apache/pinot/pull/11578#discussion_r1374921766
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java:
##
@@ -1735,20 +1735,29 @@ private void assignInstances(TableC
suddendust commented on code in PR #11892:
URL: https://github.com/apache/pinot/pull/11892#discussion_r1374909306
##
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/MultiStageBrokerRequestHandler.java:
##
@@ -191,6 +192,11 @@ protected BrokerResponse handleRequ
xiangfu0 merged PR #11861:
URL: https://github.com/apache/pinot/pull/11861
--
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
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 6853991193 Fix multiple consecutive Exchange returning empty response
(#11885)
add ce25936323 Radomize server po
walterddr commented on issue #11879:
URL: https://github.com/apache/pinot/issues/11879#issuecomment-1783270316
@kim-jaewoong not really, if you change the order by to val it still works.
b/c pinot considers literal as unparsed type until execution --> so it is
forgiving enough to auto-cast
walterddr commented on code in PR #11859:
URL: https://github.com/apache/pinot/pull/11859#discussion_r1374859670
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##
@@ -268,20 +268,28 @@ public String recommendConfig(Str
walterddr commented on code in PR #11885:
URL: https://github.com/apache/pinot/pull/11885#discussion_r1374834351
##
pinot-query-planner/src/main/java/org/apache/pinot/query/planner/logical/PlanFragmenter.java:
##
@@ -128,6 +128,8 @@ public PlanNode visitExchange(ExchangeNode nod
walterddr commented on code in PR #11885:
URL: https://github.com/apache/pinot/pull/11885#discussion_r1374834351
##
pinot-query-planner/src/main/java/org/apache/pinot/query/planner/logical/PlanFragmenter.java:
##
@@ -128,6 +128,8 @@ public PlanNode visitExchange(ExchangeNode nod
Jackie-Jiang commented on code in PR #11859:
URL: https://github.com/apache/pinot/pull/11859#discussion_r1374828964
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##
@@ -268,20 +268,28 @@ public String recommendConfig(
walterddr commented on code in PR #11885:
URL: https://github.com/apache/pinot/pull/11885#discussion_r1374827733
##
pinot-query-planner/src/main/java/org/apache/pinot/query/planner/logical/PlanFragmenter.java:
##
@@ -128,6 +128,8 @@ public PlanNode visitExchange(ExchangeNode nod
walterddr commented on code in PR #11885:
URL: https://github.com/apache/pinot/pull/11885#discussion_r1374827733
##
pinot-query-planner/src/main/java/org/apache/pinot/query/planner/logical/PlanFragmenter.java:
##
@@ -128,6 +128,8 @@ public PlanNode visitExchange(ExchangeNode nod
mcvsubbu commented on PR #11702:
URL: https://github.com/apache/pinot/pull/11702#issuecomment-1783192482
@gortiz is bumping to log4j-slf4j2 absolutely necessary? Can we revert that?
The reason is that this pulls in later version of log4j library (like you
have changed), and they seem
Jackie-Jiang commented on code in PR #11892:
URL: https://github.com/apache/pinot/pull/11892#discussion_r1374783099
##
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/SingleConnectionBrokerRequestHandler.java:
##
@@ -116,6 +117,9 @@ protected BrokerResponseNati
walterddr commented on code in PR #11882:
URL: https://github.com/apache/pinot/pull/11882#discussion_r1374786701
##
pinot-query-planner/src/test/resources/queries/JoinPlans.json:
##
@@ -421,6 +421,28 @@
"\n LogicalTableScan(table=[[b]])",
"\n"
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 6853991193 Fix multiple consecutive Exchange retu
Jackie-Jiang closed issue #11880: [multistage] mailbox receiving stage not
correctly created
URL: https://github.com/apache/pinot/issues/11880
--
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
Jackie-Jiang merged PR #11885:
URL: https://github.com/apache/pinot/pull/11885
--
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
jadami10 opened a new issue, #11895:
URL: https://github.com/apache/pinot/issues/11895
Table rebalance today, except for dry-run and downtime, happens as a series
of steps that update the ideal state and wait for external view to converge.
These ideal state updates will ask servers to both
jadami10 opened a new issue, #11894:
URL: https://github.com/apache/pinot/issues/11894
We tried scaling up an upsert table and rebalancing. With
MinimizeDataMovement, we still saw effectively every single segment moving to a
different server.
Upsert tables are more complicated becau
jadami10 opened a new issue, #11893:
URL: https://github.com/apache/pinot/issues/11893
We got to this error by
- initially creating a table and ingestion GB/TBs of data
- months later, we added a Json column and reloaded all segments
- months after that, we added the Json column to `
codecov-commenter commented on PR #11892:
URL: https://github.com/apache/pinot/pull/11892#issuecomment-1782892671
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11892?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
suddendust closed pull request #11892: Broker Query Timeout Metric
URL: https://github.com/apache/pinot/pull/11892
--
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 unsubscrib
suddendust opened a new pull request, #11892:
URL: https://github.com/apache/pinot/pull/11892
Exposing a new metric to tracing broker query timeouts.
--
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
codecov-commenter commented on PR #11877:
URL: https://github.com/apache/pinot/pull/11877#issuecomment-1782725284
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11877?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
araghavendra16 opened a new issue, #11891:
URL: https://github.com/apache/pinot/issues/11891
Hi, I'm on docker setup.
I'm trying to implement JMX to Prometheus
https://docs.pinot.apache.org/operators/operating-pinot/monitoring
This is my docker-compose
version: '3.7'
s
abhioncbr commented on code in PR #11868:
URL: https://github.com/apache/pinot/pull/11868#discussion_r1374284752
##
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/TlsIntegrationTest.java:
##
@@ -211,7 +211,7 @@ protected PinotConfiguration
getDefaultSe
alexch2000 commented on code in PR #11773:
URL: https://github.com/apache/pinot/pull/11773#discussion_r1372712667
##
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/transformer/datetimehop/EpochToEpochWindowHopTransformer.java:
##
@@ -0,0 +1,48 @@
+/**
+ * Lice
codecov-commenter commented on PR #11890:
URL: https://github.com/apache/pinot/pull/11890#issuecomment-1782499595
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/11890?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
snleee commented on code in PR #11869:
URL: https://github.com/apache/pinot/pull/11869#discussion_r1374204729
##
pinot-spi/src/main/java/org/apache/pinot/spi/utils/builder/TableConfigBuilder.java:
##
@@ -73,8 +72,9 @@ public class TableConfigBuilder {
@Deprecated
private S
snleee opened a new pull request, #11890:
URL: https://github.com/apache/pinot/pull/11890
- Currently, we support 'gcpKey' option which requires to have the gcp
credential file available locally.
- We add 'jsonKey' option where the raw string or base64 encoded gcp
credential can be passe
65 matches
Mail list logo
