pabrahamusa commented on issue #6449:
URL:
https://github.com/apache/incubator-pinot/issues/6449#issuecomment-765873995
@subourbonite At the moment the helm chart in the repo uses Statefulset for
controller. which always creates separate storage. The folder in the chart for
controller is
codecov-io commented on pull request #6483:
URL: https://github.com/apache/incubator-pinot/pull/6483#issuecomment-765862515
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6483?src=pr&el=h1)
Report
> Merging
[#6483](https://codecov.io/gh/apache/incubator-pinot/pull/6483?s
daniellavoie commented on pull request #6440:
URL: https://github.com/apache/incubator-pinot/pull/6440#issuecomment-765862374
> > > So, we are still making the assumption that the client credentials are
sent via http headers. I suppose that is OK, but maybe we should also add an
interface
mcvsubbu commented on pull request #6440:
URL: https://github.com/apache/incubator-pinot/pull/6440#issuecomment-765860188
> > So, we are still making the assumption that the client credentials are
sent via http headers. I suppose that is OK, but maybe we should also add an
interface that p
mcvsubbu commented on a change in pull request #6440:
URL: https://github.com/apache/incubator-pinot/pull/6440#discussion_r563012906
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentUploadDownloadRestletResource.java
##
@@ -492
daniellavoie commented on pull request #6440:
URL: https://github.com/apache/incubator-pinot/pull/6440#issuecomment-765859442
> So, we are still making the assumption that the client credentials are
sent via http headers. I suppose that is OK, but maybe we should also add an
interface that
mcvsubbu commented on pull request #6466:
URL: https://github.com/apache/incubator-pinot/pull/6466#issuecomment-765857360
How would retention manager work on these segments?
This is an automated message from the Apache Git Se
mcvsubbu opened a new pull request #6483:
URL: https://github.com/apache/incubator-pinot/pull/6483
Fixes being considered for issues #5559 and #5263 may end up
resurfacing some race conditions that we have strived to avoid
via automated mechanisms. The race conditions may lead to multi
This is an automated email from the ASF dual-hosted git repository.
jihao 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 830eb94 [TE]frontend - Activate the com
jihaozh merged pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472
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
codecov-io commented on pull request #6466:
URL: https://github.com/apache/incubator-pinot/pull/6466#issuecomment-765818572
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6466?src=pr&el=h1)
Report
> Merging
[#6466](https://codecov.io/gh/apache/incubator-pinot/pull/6466?s
kbastani opened a new pull request #6482:
URL: https://github.com/apache/incubator-pinot/pull/6482
Adding a code of conduct per the recommended community checklist found in
the project's insight page. See:
https://github.com/apache/incubator-pinot/community
## Description
Add a d
kbastani opened a new pull request #6481:
URL: https://github.com/apache/incubator-pinot/pull/6481
Add a contribution guideline README, per the recommended GitHub community
checklist found in project insights. See:
https://github.com/apache/incubator-pinot/community
-
apucher merged pull request #6418:
URL: https://github.com/apache/incubator-pinot/pull/6418
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
fx19880617 opened a new issue #6480:
URL: https://github.com/apache/incubator-pinot/issues/6480
Seeing more requirements on this.
Kafka consumer supports exact-once semantics but requires code changes.
Code snippet:
```
KafkaConsumer consumer = createKafkaConsumer(
"boot
codecov-io edited a comment on pull request #6403:
URL: https://github.com/apache/incubator-pinot/pull/6403#issuecomment-759105793
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6403?src=pr&el=h1)
Report
> Merging
[#6403](https://codecov.io/gh/apache/incubator-pinot/pull
JoeFuentes-WebDev edited a comment on pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#issuecomment-765728382
LGTM 👍🏽
This is an automated message from the Apache Git Service.
To respond to the me
JoeFuentes-WebDev commented on pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#issuecomment-765728382
LGTM
This is an automated message from the Apache Git Service.
To respond to the message, plea
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch segment-creation-in-one-pass
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 51aaa35 Support data ingestion for offline segment in one pass
add 3e4c325 [INFRA
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch segment-creation-in-one-pass
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 7bcaa01 Support data ingestion for offline segment in one pass
add 51aaa35 Suppor
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 04e2bcc Update superset docker image repo (#6477)
add dde3c18 TLS-support for client-pinot and pinot
codecov-io edited a comment on pull request #6403:
URL: https://github.com/apache/incubator-pinot/pull/6403#issuecomment-759105793
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6403?src=pr&el=h1)
Report
> Merging
[#6403](https://codecov.io/gh/apache/incubator-pinot/pull
Jackie-Jiang commented on issue #6478:
URL:
https://github.com/apache/incubator-pinot/issues/6478#issuecomment-765688325
The problem is that we don't have a scalar function implementation of
`cast()`, thus it won't be resolved at compilation
--
jackjlli opened a new pull request #6479:
URL: https://github.com/apache/incubator-pinot/pull/6479
## Description
Currently when generating an offline segment, raw data has to be traversed
two times; one time for gathering stats in
`RecordReaderSegmentCreationDataSource`, another time f
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch segment-creation-in-one-pass
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 7bcaa01d55070b36a1f3219e0cf33de8724e7084
Author: Jack Li(Analytics Engineering)
AuthorDate:
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch segment-creation-in-one-pass
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 7bcaa01 Support data ingestion for offline segment in one pass
This branch includes th
mayankshriv commented on issue #6478:
URL:
https://github.com/apache/incubator-pinot/issues/6478#issuecomment-765673018
I did a simple test to compile the query to BrokerRequest, and do see
`cast(now() - 17280 as long)` as LHS. My test did not actually go through
any query execution.
subourbonite closed issue #6478:
URL: https://github.com/apache/incubator-pinot/issues/6478
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
tejasajmera commented on a change in pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#discussion_r562895316
##
File path: thirdeye/thirdeye-frontend/app/pods/manage/explore/route.js
##
@@ -12,9 +12,14 @@ import { formatYamlFilter, redundantParse } fr
Jackie-Jiang commented on issue #6478:
URL:
https://github.com/apache/incubator-pinot/issues/6478#issuecomment-765662098
This is unexpected. `cast(now() - 17280 as long)` should be replaced
with the actual value on the broker side, and these 2 queries should be
identical on the server
JoeFuentes-WebDev commented on a change in pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#discussion_r562879413
##
File path: thirdeye/thirdeye-frontend/app/pods/manage/explore/route.js
##
@@ -12,9 +12,14 @@ import { formatYamlFilter, redundantPars
apucher commented on pull request #6418:
URL: https://github.com/apache/incubator-pinot/pull/6418#issuecomment-765648243
@mcvsubbu fixed the message. thanks for bearing with me.
This is an automated message from the Apache Gi
tejasajmera commented on a change in pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#discussion_r562865959
##
File path: thirdeye/thirdeye-frontend/app/pods/manage/explore/route.js
##
@@ -106,16 +116,16 @@ export default Route.extend(AuthenticatedRo
jackjlli commented on pull request #6361:
URL: https://github.com/apache/incubator-pinot/pull/6361#issuecomment-765640917
@mayankshriv Regarding performance impact, I've benchmarked this PR with two
type use cases. One is with high qps, one is with high number of columns in the
schema. Nei
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 7649f9f Remove the usage of deprecated range delimiter (#6475)
add 04e2bcc Update superset docker im
fx19880617 merged pull request #6477:
URL: https://github.com/apache/incubator-pinot/pull/6477
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
JoeFuentes-WebDev commented on a change in pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#discussion_r562845797
##
File path: thirdeye/thirdeye-frontend/app/pods/manage/explore/route.js
##
@@ -106,16 +116,16 @@ export default Route.extend(Authentic
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/incubator-pinot.git.
from f7af798 Handle scheduler calls with proper response when it's
disabled. (#6474)
add 7649f9f Remove t
Jackie-Jiang merged pull request #6475:
URL: https://github.com/apache/incubator-pinot/pull/6475
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
tejasajmera commented on a change in pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#discussion_r562844719
##
File path: thirdeye/thirdeye-frontend/app/pods/manage/explore/route.js
##
@@ -12,9 +12,14 @@ import { formatYamlFilter, redundantParse } fr
tejasajmera commented on a change in pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#discussion_r562844274
##
File path: thirdeye/thirdeye-frontend/app/pods/manage/explore/route.js
##
@@ -12,9 +12,14 @@ import { formatYamlFilter, redundantParse } fr
tejasajmera commented on a change in pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#discussion_r562841868
##
File path: thirdeye/thirdeye-frontend/app/pods/manage/explore/route.js
##
@@ -106,16 +116,16 @@ export default Route.extend(AuthenticatedRo
fx19880617 commented on issue #6471:
URL:
https://github.com/apache/incubator-pinot/issues/6471#issuecomment-765611219
> > I think it's not a good idea. Pinot will start using SQL instead of PQL
completely. Also, connector uses SQL compiler of Pinot, and I don't know SQL
compiler supports
subourbonite opened a new issue #6478:
URL: https://github.com/apache/incubator-pinot/issues/6478
In some cases, time-column predicates are disqualifying queries from using
star-trees, when they would otherwise be used. Specifically, this appears to be
the case when using a dynamic timesta
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a change to branch pinot-internode-tls
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 80f62e9 use global tls config only
add f66919e expand inferPort() error message
No new
pabrahamusa edited a comment on issue #6449:
URL:
https://github.com/apache/incubator-pinot/issues/6449#issuecomment-765591119
@subourbonite I am running two controllers and two brokers not aware that
the controller data should be shared . I can map same PVC for both
controllers. Thanks
pabrahamusa commented on issue #6449:
URL:
https://github.com/apache/incubator-pinot/issues/6449#issuecomment-765591119
@subourbonite I am running two controllers and two brokers not aware that
the controller data should be shared . I can map a same PVC for both
controllers. Thanks for p
JoeFuentes-WebDev commented on a change in pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#discussion_r562788771
##
File path: thirdeye/thirdeye-frontend/app/pods/manage/explore/route.js
##
@@ -106,16 +116,16 @@ export default Route.extend(Authentic
mcvsubbu commented on a change in pull request #6418:
URL: https://github.com/apache/incubator-pinot/pull/6418#discussion_r562797116
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/HelixBrokerStarter.java
##
@@ -117,13 +124,18 @@ public HelixBroke
JoeFuentes-WebDev commented on a change in pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#discussion_r562792975
##
File path: thirdeye/thirdeye-frontend/app/pods/manage/explore/route.js
##
@@ -12,9 +12,14 @@ import { formatYamlFilter, redundantPars
JoeFuentes-WebDev commented on a change in pull request #6472:
URL: https://github.com/apache/incubator-pinot/pull/6472#discussion_r562792975
##
File path: thirdeye/thirdeye-frontend/app/pods/manage/explore/route.js
##
@@ -12,9 +12,14 @@ import { formatYamlFilter, redundantPars
khanhpv commented on issue #6471:
URL:
https://github.com/apache/incubator-pinot/issues/6471#issuecomment-765462272
> I think it's not a good idea. Pinot will start using SQL instead of PQL
completely. Also, connector uses SQL compiler of Pinot, and I don't know SQL
compiler supports rege
khanhpv commented on issue #6471:
URL:
https://github.com/apache/incubator-pinot/issues/6471#issuecomment-765461480
> I feel in this case, we can still write the query in SparkSQL `LIKE`
syntax. Then in the filter Push Down code
(https://github.com/apache/incubator-pinot/blob/master/pinot
fx19880617 merged pull request #6469:
URL: https://github.com/apache/incubator-pinot/pull/6469
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
fx19880617 edited a comment on issue #6471:
URL:
https://github.com/apache/incubator-pinot/issues/6471#issuecomment-765324592
I feel in this case, we can still write the query in SparkSQL `LIKE` syntax.
Then in the filter Push Down code
(https://github.com/apache/incubator-pinot/blob/mast
fx19880617 commented on issue #6471:
URL:
https://github.com/apache/incubator-pinot/issues/6471#issuecomment-765324592
I feel in this case, we need to write the query in SparkSQL syntax and in
the filter PushDown code
(https://github.com/apache/incubator-pinot/blob/master/pinot-connectors
fx19880617 opened a new pull request #6477:
URL: https://github.com/apache/incubator-pinot/pull/6477
## Description
update superset docker image repo from `apache/incubator-superset` to
`apache/superset`
This is an automa
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch update_superset_image_repo
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 95e7227 Update superset docker image repo
This branch includes the following new comm
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch update_superset_image_repo
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 95e72272e6d9e99acc3dce1bb72c3ee5f7697e17
Author: Xiang Fu
AuthorDate: Fri Jan 22 02:27:46
fx19880617 commented on issue #6449:
URL:
https://github.com/apache/incubator-pinot/issues/6449#issuecomment-765281216
> @fx19880617 do you mean the controller data folder? I have mapped
/var/pinot/controller/data to a SSD and its max size is 10GB however it was
only 50% full. I am using
mangrrua commented on issue #6471:
URL:
https://github.com/apache/incubator-pinot/issues/6471#issuecomment-765272112
I think it's not a good idea. Pinot will start using SQL instead of PQL
completely. Also, connector uses SQL compiler of Pinot, and I don't know SQL
compiler supports regex
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 f7af798 Handle scheduler calls with p
fx19880617 merged pull request #6474:
URL: https://github.com/apache/incubator-pinot/pull/6474
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
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 8c8897e Simplify SegmentGenerationAnd
jihaozh merged pull request #6448:
URL: https://github.com/apache/incubator-pinot/pull/6448
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
65 matches
Mail list logo
