This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 3bba45fc0c643dbabf05d9f022a63d32dbc66e9f
Author: Jack Li(Analytics Engineering)
AuthorDate: Fri Oct 23
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 3bba45f [maven-release-plugin] prepare release release-0.6.0-rc0
This branch includes the
codecov-io edited a comment on pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184#issuecomment-715567148
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6184?src=pr=h1)
Report
> Merging
codecov-io edited a comment on pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184#issuecomment-715567148
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6184?src=pr=h1)
Report
> Merging
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=h1)
Report
> Merging
codecov-io commented 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=h1)
Report
> Merging
snleee opened a new pull request #6185:
URL: https://github.com/apache/incubator-pinot/pull/6185
Current validation code will fail when the time column name is
set to empty string. We can relax the validation by treating
the empty string the same as the null value. Added the unit
jtao15 commented on a change in pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511235934
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartMinionCommand.java
##
@@ -0,0 +1,110 @@
+/**
+ * Licensed to
jtao15 commented on a change in pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511234621
##
File path:
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/ClusterTest.java
##
@@ -184,7 +184,7 @@ protected void
snleee commented on a change in pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511218564
##
File path:
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/ClusterTest.java
##
@@ -184,7 +184,7 @@ protected void
snleee commented on a change in pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511218200
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotMinion.java
##
@@ -0,0 +1,37 @@
+/**
+ * Licensed to the Apache
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at cad3619 [maven-release-plugin] prepare release release-0.6.0-rc1
This branch includes the
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit cad3619b98a524d26ae549a2c09566d32650f584
Author: Jack Li(Analytics Engineering)
AuthorDate: Fri Oct 23
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit b3f1ff9b2a23a2a8fa44a76047b18eeaf5190367
Author: Jack Li(Analytics Engineering)
AuthorDate: Fri Oct 23
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at b3f1ff9 [maven-release-plugin] prepare release release-0.6.0-rc0
This branch includes the
jtao15 commented on a change in pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511187102
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotMinion.java
##
@@ -0,0 +1,37 @@
+/**
+ * Licensed to the Apache
codecov-io edited a comment on pull request #6127:
URL: https://github.com/apache/incubator-pinot/pull/6127#issuecomment-706417804
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6127?src=pr=h1)
Report
> Merging
codecov-io edited a comment on pull request #6127:
URL: https://github.com/apache/incubator-pinot/pull/6127#issuecomment-706417804
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6127?src=pr=h1)
Report
> Merging
codecov-io edited a comment on pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184#issuecomment-715567148
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6184?src=pr=h1)
Report
> Merging
codecov-io commented on pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184#issuecomment-715567148
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6184?src=pr=h1)
Report
> Merging
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to tag release-0.6.0-rc0
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at bb6a3f4 (commit)
No new revisions were added by this update.
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/release-0.6.0-rc by this push:
new bb6a3f4
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch rewrite_array_aggregation_functions
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 2f93b8b format
discard 463d634 Rewrite possible array aggregation
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to annotated tag release-0.7.0-rc0
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
*** WARNING: tag release-0.7.0-rc0 was modified! ***
from 691b23c (commit)
to 4fa1f8c
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 691b23c [maven-release-plugin] prepare release release-0.7.0-rc0
No new revisions were added by
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/release-0.6.0-rc by this push:
new bb6a3f4
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to annotated tag release-0.7.0-rc0
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
*** WARNING: tag release-0.7.0-rc0 was modified! ***
from 691b23c (commit)
to 4fa1f8c
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 691b23c [maven-release-plugin] prepare release release-0.7.0-rc0
No new revisions were added by
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch rewrite_array_aggregation_functions
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 2f93b8b format
discard 463d634 Rewrite possible array aggregation
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch rewrite_array_aggregation_functions
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 2f93b8b format
discard 463d634 Rewrite possible array aggregation
fx19880617 edited a comment on pull request #6127:
URL: https://github.com/apache/incubator-pinot/pull/6127#issuecomment-715083700
> High level question, should we overload `SUM`, `MIN`, `MAX` to directly
work on MV columns? I feel it is much easier to use if these functions directly
work
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to annotated tag release-0.7.0-rc0
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
*** WARNING: tag release-0.7.0-rc0 was modified! ***
from 691b23c (commit)
to 4fa1f8c
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/release-0.6.0-rc by this push:
new bb6a3f4
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 691b23c [maven-release-plugin] prepare release release-0.7.0-rc0
No new revisions were added by
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/release-0.6.0-rc by this push:
new bb6a3f4
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to annotated tag release-0.7.0-rc0
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
*** WARNING: tag release-0.7.0-rc0 was modified! ***
from 691b23c (commit)
to 4fa1f8c
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 691b23c [maven-release-plugin] prepare release release-0.7.0-rc0
No new revisions were added by
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to annotated tag release-0.7.0-rc0
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
*** WARNING: tag release-0.7.0-rc0 was modified! ***
from 691b23c (commit)
to 4fa1f8c
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/release-0.6.0-rc by this push:
new bb6a3f4
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 691b23c [maven-release-plugin] prepare release release-0.7.0-rc0
No new revisions were added by
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 73d2839 Merge common APIs for Dictionary (#6176)
add 5577e87 Add table level lock for segment
fx19880617 commented on a change in pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165#discussion_r511102497
##
File path:
pinot-spi/src/main/java/org/apache/pinot/spi/utils/retry/RandomDelayRetryPolicy.java
##
@@ -0,0 +1,44 @@
+/**
+ * Licensed to
fx19880617 merged pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
codecov-io edited a comment on pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165#issuecomment-713177186
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6165?src=pr=h1)
Report
> Merging
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a change to branch te-remove-shade-plugin
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from b975140 [TE] Remove maven-shade-plugin in thirdeye-dashboard (#6151)
add 69c29c1 [TE]
jihaozh merged pull request #6180:
URL: https://github.com/apache/incubator-pinot/pull/6180
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
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch add_table_lock_for_segment_upload
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 6d910f9 Using random delay policy
add 27f84c4 update random policy
No
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch release-0.6.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/release-0.6.0-rc by this push:
new 7d91803
mcvsubbu commented on issue #5346:
URL:
https://github.com/apache/incubator-pinot/issues/5346#issuecomment-715500233
> Vote for this solution: `One way to get this is to comb the 10k queries to
select a few hundred that can be enough to detect all issues. And then we add
queries to this
Jackie-Jiang commented on issue #5346:
URL:
https://github.com/apache/incubator-pinot/issues/5346#issuecomment-715497623
Vote for this solution: `One way to get this is to comb the 10k queries to
select a few hundred that can be enough to detect all issues. And then we add
queries to
codecov-io commented on pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181#issuecomment-715496693
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6181?src=pr=h1)
Report
> Merging
snleee commented on a change in pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184#discussion_r511052394
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotMinion.java
##
@@ -0,0 +1,37 @@
+/**
+ * Licensed to the Apache
Jackie-Jiang commented on a change in pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181#discussion_r511048205
##
File path:
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/BaseClusterIntegrationTestSet.java
##
@@ -60,7 +60,7
jtao15 opened a new pull request #6184:
URL: https://github.com/apache/incubator-pinot/pull/6184
Minion command line starter for issue #6111
This is an automated message from the Apache Git Service.
To respond to the
mayankshriv commented on pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181#issuecomment-715487591
@jackjlli Slightly tangential question, most users are either migrating to
or already using SQL. Is there a SQL version these queries that we are
validating
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 d6d2c64 Reduce pql queries from 10k to 500
add 6aa17fd Reduce pql queries from 10k
icefury71 commented on pull request #6172:
URL: https://github.com/apache/incubator-pinot/pull/6172#issuecomment-715459444
> Should we also add api's to get off-heap memory consumption for:
>
> * All tables hosted on the server (multi-tenant case).
> * Usage for a table across all
jackjlli commented on pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181#issuecomment-715457398
> You may want to add a couple of queries that return empty results also.
Not necessarily, the queries with empty response will skip the comparison of
the
mayankshriv commented on a change in pull request #6172:
URL: https://github.com/apache/incubator-pinot/pull/6172#discussion_r511009756
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/realtime/impl/RealtimeSegmentStatsHistory.java
##
@@ -385,7 +395,7 @@ public
mayankshriv commented on pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181#issuecomment-715452220
Do we have a coverage report for this one? Granted that queries with empty
results are not effective, but if the code path is still exercised and we see a
drop in
mcvsubbu commented on a change in pull request #6165:
URL: https://github.com/apache/incubator-pinot/pull/6165#discussion_r510996450
##
File path:
pinot-spi/src/main/java/org/apache/pinot/spi/utils/retry/RandomDelayRetryPolicy.java
##
@@ -0,0 +1,44 @@
+/**
+ * Licensed to the
sajjad-moradi commented on a change in pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181#discussion_r510979784
##
File path:
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/BaseClusterIntegrationTestSet.java
##
@@ -320,19
cyrilou242 opened a new pull request #6183:
URL: https://github.com/apache/incubator-pinot/pull/6183
## Description
For the Jira Alert in ThirdEye:
If a jira reporter user name contains a `@` character, eg
`n...@mycompany.com`, an error is raised when trying to run the JQL
venkatvghub opened a new issue #6182:
URL: https://github.com/apache/incubator-pinot/issues/6182
Hello, am seeing an image loading issue on the "Anamolies" page. At times,
the images show up and at times it doesnt. Have to hit page load many a times
to see a missing image. Same image
jackjlli opened a new pull request #6181:
URL: https://github.com/apache/incubator-pinot/pull/6181
## Description
This PR:
* removes all the queries that return empty result.
* reduces pql 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 d6d2c64 Reduce pql queries from 10k to 500
This branch includes the following new
fx19880617 commented on pull request #6127:
URL: https://github.com/apache/incubator-pinot/pull/6127#issuecomment-715083700
> High level question, should we overload `SUM`, `MIN`, `MAX` to directly
work on MV columns? I feel it is much easier to use if these functions directly
work on
67 matches
Mail list logo