This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch txt-hotfix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 9c86da467d607f8f05e5e27889bdda3c635bc51d
Author: Jack Li(Analytics Engineering)
AuthorDate: Mon Oct 26 22:34:
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch txt-hotfix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 184655b Modify .travis_install.sh
new 9c86da4 Modify .travis_install.sh
This update added new revi
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch txt-hotfix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard fdc4bce Modify .travis_install.sh
new 184655b Modify .travis_install.sh
This update added new revi
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch txt-hotfix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 184655b654d01683d8986161e3fcaaa70703ca53
Author: Jack Li(Analytics Engineering)
AuthorDate: Mon Oct 26 22:34:
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch txt-hotfix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/txt-hotfix by this push:
new fdc4bce Modify .travis_install.s
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch refactor-integration-tests
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard be71612 Reduce number of sql queries from 10k to 500 in
pinot-integration-tests module
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/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 0f4ee68 Compute absolute docId in
siddharthteotia merged pull request #6194:
URL: https://github.com/apache/incubator-pinot/pull/6194
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
This is an automated email from the ASF dual-hosted git repository.
snlee 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 ec1c859 Relaxing timeColumnName and ind
snleee merged pull request #6185:
URL: https://github.com/apache/incubator-pinot/pull/6185
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-io edited a comment on pull request #6185:
URL: https://github.com/apache/incubator-pinot/pull/6185#issuecomment-715657660
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6185?src=pr&el=h1)
Report
> Merging
[#6185](https://codecov.io/gh/apache/incubator-pinot/pull
codecov-io edited a comment on pull request #6185:
URL: https://github.com/apache/incubator-pinot/pull/6185#issuecomment-715657660
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6185?src=pr&el=h1)
Report
> Merging
[#6185](https://codecov.io/gh/apache/incubator-pinot/pull
codecov-io edited a comment on pull request #6185:
URL: https://github.com/apache/incubator-pinot/pull/6185#issuecomment-715657660
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6185?src=pr&el=h1)
Report
> Merging
[#6185](https://codecov.io/gh/apache/incubator-pinot/pull
codecov-io edited a comment on pull request #6185:
URL: https://github.com/apache/incubator-pinot/pull/6185#issuecomment-715657660
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6185?src=pr&el=h1)
Report
> Merging
[#6185](https://codecov.io/gh/apache/incubator-pinot/pull
siddharthteotia opened a new pull request #6194:
URL: https://github.com/apache/incubator-pinot/pull/6194
The lookup table of lucene docId to pinot docId is built on absolute docIDs.
We merge multiple sub-indexes into a single index (to also restrict the number
of files) but the internal m
jackjlli commented on pull request #6192:
URL: https://github.com/apache/incubator-pinot/pull/6192#issuecomment-716904164
I've filed an issue to track the issue of comparison code and label it as
`help-wanted`: https://github.com/apache/incubator-pinot/issues/6193
jackjlli opened a new issue #6193:
URL: https://github.com/apache/incubator-pinot/issues/6193
Currently the comparison logic of SQL queries is so limited that:
* any queries with multi-value column will skip the comparison.
* the order by columns has to be in the selection clause.
*
jackjlli commented on pull request #6192:
URL: https://github.com/apache/incubator-pinot/pull/6192#issuecomment-716895165
@siddharthteotia I agree on that. But the restriction is not on SQL but on
the code of validating the results of SQL in pinot-integration-tests module.
E.g. the compari
mcvsubbu commented on pull request #6192:
URL: https://github.com/apache/incubator-pinot/pull/6192#issuecomment-716892305
@jackjlli for this iteration, if there is comparison missing you can choose
to leave it as is with a TODO and an issue (unless you are able to add a
comparator trivial
This is an automated email from the ASF dual-hosted git repository.
siddteotia pushed a change to branch txt-hotfix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 7dd0523 Use docBase from leaf context for multi-segment search
new 3e1326f Use docBase from l
This is an automated email from the ASF dual-hosted git repository.
siddteotia pushed a commit to branch txt-hotfix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 3e1326fbc009e597d8dc7b17aca86bc4370bcf0d
Author: Siddharth Teotia
AuthorDate: Mon Oct 26 09:39:37 2020
siddharthteotia commented on pull request #6192:
URL: https://github.com/apache/incubator-pinot/pull/6192#issuecomment-716887472
> > Can we make the sql queries one-to-one mapping with the pql queries?
>
> It's a good idea but the thing is that there are some restrictions in SQL
comp
codecov-io commented on pull request #6192:
URL: https://github.com/apache/incubator-pinot/pull/6192#issuecomment-716877506
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6192?src=pr&el=h1)
Report
> Merging
[#6192](https://codecov.io/gh/apache/incubator-pinot/pull/6192?s
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch fix-asc-signiture-in-pinot-spark-connector-module
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 966f36d9f61920be8412becf2afd4480eb296d6b
Author: Jack Li(Analytics Engi
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch fix-asc-signiture-in-pinot-spark-connector-module
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 966f36d Fix missing asc signiture file for
target/pinot-spark-con
npawar merged pull request #6190:
URL: https://github.com/apache/incubator-pinot/pull/6190
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
jackjlli commented on issue #5346:
URL:
https://github.com/apache/incubator-pinot/issues/5346#issuecomment-716870409
@Jackie-Jiang Yes, we should keep it open. The issue auto closes itself when
some PR is merged.
This is an
This is an automated email from the ASF dual-hosted git repository.
nehapawar 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 0b8cc88 Updating Pinot UI GIF (#619
mcvsubbu opened a new issue #5346:
URL: https://github.com/apache/incubator-pinot/issues/5346
Recently, a bug was introduced in PR #5132 that led to bad results when
executing queries that involved indexed and non-indexed columns in a certain
combination. We had a query in our integration
jackjlli commented on pull request #6192:
URL: https://github.com/apache/incubator-pinot/pull/6192#issuecomment-716870123
> Can we make the sql queries one-to-one mapping with the pql queries?
It's a good idea but the thing is that there are some restrictions in SQL
comparisons, e.g.
Jackie-Jiang commented on issue #5346:
URL:
https://github.com/apache/incubator-pinot/issues/5346#issuecomment-716862682
@jackjlli Should we keep this issue open to track all the PRs improving the
test coverage?
This is an
jackjlli opened a new pull request #6192:
URL: https://github.com/apache/incubator-pinot/pull/6192
This PR:
* removes queries with empty response.
* reduces the number of sql queries from 10k to 500 in
pinot-integration-tests module.
-
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch refactor-integration-tests
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 5675d4d Reduce number of sql queries from 10k to 500 in
pinot-integration-tests module
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch refactor-integration-tests
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 5675d4d Reduce number of sql queries from 10k to 500 in
pinot-integration-tests module
siddharthteotia commented on issue #6191:
URL:
https://github.com/apache/incubator-pinot/issues/6191#issuecomment-716845267
Can we use the existing issue?
https://github.com/apache/incubator-pinot/issues/5664
This is an a
dharakk opened a new issue #6191:
URL: https://github.com/apache/incubator-pinot/issues/6191
This issue represents the work to support join like functionality via Lookup
UDFs in Pinot.
Design document for proposed feature:
https://docs.google.com/document/d/1_Ms0qBFBWN69UWrjkJiDC1VB
codecov-io edited a comment on pull request #6190:
URL: https://github.com/apache/incubator-pinot/pull/6190#issuecomment-716806976
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6190?src=pr&el=h1)
Report
> Merging
[#6190](https://codecov.io/gh/apache/incubator-pinot/pull
codecov-io commented on pull request #6190:
URL: https://github.com/apache/incubator-pinot/pull/6190#issuecomment-716806976
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6190?src=pr&el=h1)
Report
> Merging
[#6190](https://codecov.io/gh/apache/incubator-pinot/pull/6190?s
kbastani commented on pull request #6190:
URL: https://github.com/apache/incubator-pinot/pull/6190#issuecomment-716784750
@snleee Sorry about that. Merge conflicts are resolved.
This is an automated message from the Apache Gi
snleee commented on pull request #6190:
URL: https://github.com/apache/incubator-pinot/pull/6190#issuecomment-716780582
@kbastani Please resolve the merge conflict before you merge this.
This is an automated message from the
kbastani opened a new pull request #6190:
URL: https://github.com/apache/incubator-pinot/pull/6190
## Description
Updated the Pinot UI GIF to fix dead link issue with GitHub camo requiring
GIFs to be under 5mb for proxy.
-
snleee closed issue #6111:
URL: https://github.com/apache/incubator-pinot/issues/6111
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 t
This is an automated email from the ASF dual-hosted git repository.
nehapawar 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 b97460c Updated README with Pinot U
npawar merged pull request #6164:
URL: https://github.com/apache/incubator-pinot/pull/6164
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
npawar commented on pull request #6164:
URL: https://github.com/apache/incubator-pinot/pull/6164#issuecomment-716764995
Looks good!
This is an automated message from the Apache Git Service.
To respond to the message, please l
jackjlli closed issue #5346:
URL: https://github.com/apache/incubator-pinot/issues/5346
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
jackjlli merged pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181
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
jtao15 commented on issue #6111:
URL:
https://github.com/apache/incubator-pinot/issues/6111#issuecomment-716738758
The pr is merged, please close the issue, thanks!
This is an automated message from the Apache Git Service.
T
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/incubator-pinot.git.
from 05734bf Rewrite possible array aggregation functions to one level
(#6127)
add ef970e6 Minion command
snleee merged pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184
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
jackjlli commented on pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181#issuecomment-716709788
@Jackie-Jiang Updated the pql query file.
This is an automated message from the Apache Git Service.
To
jackjlli commented on a change in pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181#discussion_r512145494
##
File path:
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/BaseClusterIntegrationTestSet.java
##
@@ -60,7 +60,7 @@
jackjlli commented on a change in pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181#discussion_r512145400
##
File path:
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/BaseClusterIntegrationTestSet.java
##
@@ -320,19 +320,18
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch refactor-integration-tests
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard aca3f22 Reduce pql queries from 10k to 500
add 2c48fb8 Reduce pql queries from 10k
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch refactor-integration-tests
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 6aa17fd Reduce pql queries from 10k to 500
add aca3f22 Reduce pql queries from 10k
noahprince22 commented on issue #6187:
URL:
https://github.com/apache/incubator-pinot/issues/6187#issuecomment-716703570
https://github.com/apache/incubator-pinot/issues/6189
Created an issue for the segment pruning.
noahprince22 opened a new issue #6189:
URL: https://github.com/apache/incubator-pinot/issues/6189
For time based tables, segments can easily pile up. High qps would lead to a
lot of unnecessary work being done server side to read and eliminate segments.
--
mayankshriv commented on pull request #6185:
URL: https://github.com/apache/incubator-pinot/pull/6185#issuecomment-716687643
Doesn't the same apply to other configs e.g.: `invertedIndexColumns: [""]`?
This is an automated mes
This is an automated email from the ASF dual-hosted git repository.
siddteotia pushed a commit to branch txt-hotfix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/txt-hotfix by this push:
new 7dd0523 Use docBase from l
This is an automated email from the ASF dual-hosted git repository.
siddteotia pushed a change to branch txt-hotfix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at a122728 [TE] Added support for BigQuery as data source (#5868)
No new revisions were added by this
noahprince22 opened a new issue #6188:
URL: https://github.com/apache/incubator-pinot/issues/6188
https://github.com/apache/incubator-pinot/pull/6053
This pull request introduced some support for BigDecimal. Add a formal
BigDecimal data type.
--
kishoreg commented on issue #6187:
URL:
https://github.com/apache/incubator-pinot/issues/6187#issuecomment-716600900
Thanks @noahprince22 for doing the analysis and creating this issue. This is
an amazing feature to have.
Adding broker pruning by time should be easy. Yes, routing ta
noahprince22 opened a new issue #6187:
URL: https://github.com/apache/incubator-pinot/issues/6187
https://apache-pinot.slack.com/archives/CDRCA57FC/p1603720037246100
Some discussion already here.
This would involve modifying the pinot server to include a lazy mode that
would s
63 matches
Mail list logo
