mayankshriv opened a new issue #5490:
URL: https://github.com/apache/incubator-pinot/issues/5490
With the Pinot community growing, as well as Pinot evolving for the better,
we anticipate a ton of features that would go into performance critical parts
of the code (e.g. query execution, RT s
mayankshriv commented on a change in pull request #5483:
URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r434318564
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/request/v2/utils/BrokerRequestToQueryRequestConverter.java
##
@@ -0,0 +1,154
mayankshriv commented on pull request #5483:
URL: https://github.com/apache/incubator-pinot/pull/5483#issuecomment-637964051
Should we solve the issues in BrokerRequest, or move to PinotQuery? Latter
would be ideal, except in organizations with large Pinot deployments, moving to
SQL may no
mayankshriv commented on a change in pull request #5461:
URL: https://github.com/apache/incubator-pinot/pull/5461#discussion_r434311036
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/BinaryOperatorTransformFunction.java
##
@@ -0,0 +1,3
yupeng9 commented on pull request #5487:
URL: https://github.com/apache/incubator-pinot/pull/5487#issuecomment-637961796
@mayankshriv That's true for better code reuse. Though this PR is for a
minor improvement, and I believe we could the refactoring in another PR.
---
yupeng9 commented on a change in pull request #5487:
URL: https://github.com/apache/incubator-pinot/pull/5487#discussion_r434310701
##
File path: pinot-tools/src/main/java/org/apache/pinot/tools/SegmentDumpTool.java
##
@@ -91,11 +102,27 @@ public void doMain(String[] args)
npawar commented on issue #5390:
URL:
https://github.com/apache/incubator-pinot/issues/5390#issuecomment-637941159
I think one approach could be as simple as:
We introduce an API for reading physical segment's metadata files
`/segments/index-metadata` (need a better name)
1. controll
fx19880617 opened a new pull request #5489:
URL: https://github.com/apache/incubator-pinot/pull/5489
## Description
Upgrade pypi pinotdb(https://pypi.org/project/pinotdb/) version to 0.3.1 to
use new pinot sql api.
## Release Notes
No
-
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch upgrade-superset-pinotdb-version
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 9d9df06 Upgrade pinotdb version to 0.3.1 to use new pinot sql api
No new revisi
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch upgrade-superset-pinotdb-version
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 9d9df06 Upgrade pinotdb version to 0.3.1 to use new pinot sql api
No new revisi
fx19880617 commented on a change in pull request #5487:
URL: https://github.com/apache/incubator-pinot/pull/5487#discussion_r434288816
##
File path: pinot-tools/src/main/java/org/apache/pinot/tools/SegmentDumpTool.java
##
@@ -91,11 +102,27 @@ public void doMain(String[] args)
jackjlli opened a new pull request #5488:
URL: https://github.com/apache/incubator-pinot/pull/5488
This PR moves LinkedIn bintray target as a parameter in `mvn deploy` command.
Reference:
https://stackoverflow.com/questions/12435283/adding-a-maven-distribution-repository-on-the-co
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch bintray-cli
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 870b0b99a3d84d43b0dcc1dff8369c74bbd87fa0
Author: Jack Li(Analytics Engineering)
AuthorDate: Tue Jun 2 20:29:
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch bintray-cli
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 870b0b9 Remove bintray target to mvn deploy command
This branch includes the following new commits:
guruguha commented on issue #5390:
URL:
https://github.com/apache/incubator-pinot/issues/5390#issuecomment-637918220
I have an initial approach to this issue:
We can introduce a segment state monitor that keeps track of all the
segments and their index version updates. Similar to
`_las
yupeng9 opened a new pull request #5487:
URL: https://github.com/apache/incubator-pinot/pull/5487
Also, add the segment dump tool as part of the pinot-tool.sh script
Currently segment dump tool supports single-value only. This change adds the
multi-value support. It also makes segmen
guruguha removed a comment on issue #5390:
URL:
https://github.com/apache/incubator-pinot/issues/5390#issuecomment-637914979
I have an initial approach to this issue:
- provide an end point for users to query for segment status
- the end point would need to query the ZK to get the inp
guruguha commented on issue #5390:
URL:
https://github.com/apache/incubator-pinot/issues/5390#issuecomment-637914979
I have an initial approach to this issue:
- provide an end point for users to query for segment status
- the end point would need to query the ZK to get the input segme
kishoreg commented on pull request #5440:
URL: https://github.com/apache/incubator-pinot/pull/5440#issuecomment-637897871
@sidd only for scalarfunctions.
This is an automated message from the Apache Git Service.
To respond t
siddharthteotia commented on pull request #5440:
URL: https://github.com/apache/incubator-pinot/pull/5440#issuecomment-637875934
Is the plan to use this wrapper solely for invoking scalar functions (like
already done in this PR for StringFunctions) or are we expecting follow-ups to
integra
jackjlli commented on issue #5485:
URL:
https://github.com/apache/incubator-pinot/issues/5485#issuecomment-637868513
@haibow Thanks for the heads up. I can work on this.
This is an automated message from the Apache Git Serv
jamesyfshao commented on pull request #5394:
URL: https://github.com/apache/incubator-pinot/pull/5394#issuecomment-637852570
> @jamesyfshao I looked over your design doc, thanks for adding the section
on interfaces you need. I also looked at the repo you mentioned.
> It is not clear why
Jackie-Jiang commented on a change in pull request #5483:
URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r434193068
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/request/v2/ServerQuery.java
##
@@ -0,0 +1,181 @@
+/**
+ * Licensed to the A
mcvsubbu commented on a change in pull request #5336:
URL: https://github.com/apache/incubator-pinot/pull/5336#discussion_r434191769
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/utils/fetcher/PeerServerSegmentFetcher.java
##
@@ -0,0 +1,155 @@
+/**
+ * Licen
mcvsubbu commented on pull request #5394:
URL: https://github.com/apache/incubator-pinot/pull/5394#issuecomment-637813614
@jamesyfshao I looked over your design doc, thanks for adding the section on
interfaces you need. I also looked at the repo you mentioned.
It is not clear why we need
This is an automated email from the ASF dual-hosted git repository.
mcvsubbu 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 8a3eb43 Add a new table config field
mcvsubbu merged pull request #5478:
URL: https://github.com/apache/incubator-pinot/pull/5478
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 opened a new pull request #5486:
URL: https://github.com/apache/incubator-pinot/pull/5486
…gOffset
Updated the segment commit protocol so that new element
streamPartitionMsgOffset
is populated in requests (as request parameters) and in response (as JSON
string element)
jamesyfshao commented on pull request #5394:
URL: https://github.com/apache/incubator-pinot/pull/5394#issuecomment-637721377
> Hey James, the last we discussed, I thought the only callback needed was
after the realtime row was indexed.
>
> Can you add to your design doc the following
Jackie-Jiang commented on a change in pull request #5444:
URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r434066870
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/dociditerators/AndDocIdIterator.java
##
@@ -18,112 +18,52 @@
*/
pac
Jackie-Jiang commented on a change in pull request #5444:
URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r434064515
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/dociditerators/AndDocIdIterator.java
##
@@ -18,112 +18,52 @@
*/
pac
Jackie-Jiang commented on a change in pull request #5444:
URL: https://github.com/apache/incubator-pinot/pull/5444#discussion_r434063565
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/dociditerators/ArrayBasedDocIdIterator.java
##
@@ -36,37 +35,15 @@ pub
fx19880617 merged pull request #5480:
URL: https://github.com/apache/incubator-pinot/pull/5480
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 change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from ed26e85 [TE] fix document for adding new application (#5473)
add aae985f Adding more information int
mcvsubbu commented on a change in pull request #5478:
URL: https://github.com/apache/incubator-pinot/pull/5478#discussion_r433971348
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/utils/config/TableConfigUtils.java
##
@@ -201,4 +207,16 @@ public static void v
haibow opened a new issue #5485:
URL: https://github.com/apache/incubator-pinot/issues/5485
In #5190, LinkedIn deployment configs were added to the pom file. This
caused Apache release to fail, and I had to remove the following lines:
https://github.com/apache/incubator-pinot/blob/e
36 matches
Mail list logo