codecov-commenter commented on PR #14631:
URL: https://github.com/apache/pinot/pull/14631#issuecomment-2533696984
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/14631?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
yashmayya commented on code in PR #14163:
URL: https://github.com/apache/pinot/pull/14163#discussion_r1792979784
##
pinot-core/src/main/java/org/apache/pinot/core/query/optimizer/filter/NumericalFilterOptimizer.java:
##
@@ -346,6 +365,156 @@ private static Expression
rewriteRan
ankitsultana opened a new pull request, #14631:
URL: https://github.com/apache/pinot/pull/14631
WIP
--
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:
vrajat commented on PR #14110:
URL: https://github.com/apache/pinot/pull/14110#issuecomment-2533647419
Pending discussions:
- shared-nothing vs shared-storage
- cursor response constructor
- Naming - Response Store
- Auth config param
--
This is an automated message from the Ap
vrajat commented on PR #14110:
URL: https://github.com/apache/pinot/pull/14110#issuecomment-2533642293
I have added javadoc in `ResponseStore` to emphasis that there is a response
store per broker and it is assumed that it is a shared-nothing storage. This is
far more simpler and from exper
vrajat commented on code in PR #14110:
URL: https://github.com/apache/pinot/pull/14110#discussion_r1879342446
##
pinot-common/src/main/java/org/apache/pinot/common/cursors/AbstractResponseStore.java:
##
@@ -0,0 +1,264 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF)
jadami10 commented on code in PR #14612:
URL: https://github.com/apache/pinot/pull/14612#discussion_r1879333029
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##
@@ -1256,8 +1256,9 @@ private long validateSegmentStateF
codecov-commenter commented on PR #14630:
URL: https://github.com/apache/pinot/pull/14630#issuecomment-2533595117
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/14630?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
praveenc7 opened a new pull request, #14630:
URL: https://github.com/apache/pinot/pull/14630
Instructions:
1. The PR has to be tagged with at least one of the following labels (*):
1. `feature`
2. `bugfix`
3. `performance`
4. `ui`
5. `backward-incompat`
chrajeshbabu commented on issue #14552:
URL: https://github.com/apache/pinot/issues/14552#issuecomment-2533385724
Able to make the job gets started by excluding the dependency causing
org.apache.hadoop.yarn.webapp.WebApps$Builder.start(WebApps.java:472) loading
from pinot jar which leads
ankitsultana commented on code in PR #14477:
URL: https://github.com/apache/pinot/pull/14477#discussion_r1879085302
##
pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/upsertcompactmerge/UpsertCompactMergeTaskExecutor.java:
#
ankitsultana commented on code in PR #14477:
URL: https://github.com/apache/pinot/pull/14477#discussion_r1879069977
##
pinot-core/src/main/java/org/apache/pinot/core/common/MinionConstants.java:
##
@@ -199,4 +200,59 @@ public static class UpsertCompactionTask {
*/
pub
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 b4066fbd40 Fix flaky quota integration tests (#14
Jackie-Jiang merged PR #14602:
URL: https://github.com/apache/pinot/pull/14602
--
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
Jackie-Jiang commented on code in PR #14612:
URL: https://github.com/apache/pinot/pull/14612#discussion_r1878946618
##
pinot-common/src/main/java/org/apache/pinot/common/restlet/resources/TableSegmentValidationInfo.java:
##
@@ -41,4 +44,8 @@ public boolean isValid() {
public
ilamhs commented on issue #14545:
URL: https://github.com/apache/pinot/issues/14545#issuecomment-2532826617
@ashishjayamohan Please do raise a PR 👍
FYI: @zeronerdzerogeekzerocool
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
Jackie-Jiang commented on code in PR #14628:
URL: https://github.com/apache/pinot/pull/14628#discussion_r1878763687
##
pinot-core/src/main/java/org/apache/pinot/core/operator/filter/H3IndexFilterOperator.java:
##
@@ -78,7 +78,15 @@ public H3IndexFilterOperator(IndexSegment segme
ashishjayamohan commented on issue #14545:
URL: https://github.com/apache/pinot/issues/14545#issuecomment-2532651118
@ilamhs Mind if I take a swing at this? I already have the changes ready for
a PR. No worries if you're already working on it.
--
This is an automated message from the Apac
rohityadav1993 opened a new issue, #14629:
URL: https://github.com/apache/pinot/issues/14629
when doing a `select count(*) from table_name` query on a table with no
segments, the response is empty instead of 0.
This is a correct behavour for non aggregation queries but for general
ag
tibrewalpratik17 merged PR #14611:
URL: https://github.com/apache/pinot/pull/14611
--
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...@p
This is an automated email from the ASF dual-hosted git repository.
pratik 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 1b5858b5b4 [timeseries] Part-3: Add Time Series E
codecov-commenter commented on PR #14628:
URL: https://github.com/apache/pinot/pull/14628#issuecomment-2532111553
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/14628?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
bziobrowski opened a new pull request, #14628:
URL: https://github.com/apache/pinot/pull/14628
PR fixes H3IndexFilterOperator's accuracy when used with ST_Distance
function.
Calculations in master branch are inaccurate because:
- hexagon size in is not constant per level, so using
ankitsultana commented on code in PR #14611:
URL: https://github.com/apache/pinot/pull/14611#discussion_r1878237795
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/timeseries/serde/TimeSeriesBlockSerde.java:
##
@@ -0,0 +1,203 @@
+/**
+ * Licensed to the Apach
swaminathanmanish commented on code in PR #14623:
URL: https://github.com/apache/pinot/pull/14623#discussion_r1878219843
##
pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/realtimetoofflinesegments/RealtimeToOfflineSegmentsT
codecov-commenter commented on PR #14627:
URL: https://github.com/apache/pinot/pull/14627#issuecomment-2531698564
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/14627?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
yashmayya opened a new pull request, #14627:
URL: https://github.com/apache/pinot/pull/14627
- Currently, these log lines don't include any information on the exact
query (apart from the table name which isn't too useful just by itself).
- This minor patch includes the query's request ID
This is an automated email from the ASF dual-hosted git repository.
yashmayya pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 442c0fc713 Configurable sketch accuracy in merge rollup task (#14373)
add 1641a6d4bf Bump software.amazon.awssd
This is an automated email from the ASF dual-hosted git repository.
yashmayya pushed a change to branch
dependabot/maven/software.amazon.awssdk-bom-2.29.30
in repository https://gitbox.apache.org/repos/asf/pinot.git
was d41b1bb1d8 Bump software.amazon.awssdk:bom from 2.29.29 to 2.29.30
Th
yashmayya merged PR #14626:
URL: https://github.com/apache/pinot/pull/14626
--
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.ap
tibrewalpratik17 commented on code in PR #14477:
URL: https://github.com/apache/pinot/pull/14477#discussion_r1877937813
##
pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/upsertcompactmerge/UpsertCompactMergeTaskExecutor.jav
codecov-commenter commented on PR #14625:
URL: https://github.com/apache/pinot/pull/14625#issuecomment-2531334222
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/14625?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/software.amazon.awssdk-bom-2.29.30
in repository https://gitbox.apache.org/repos/asf/pinot.git
at d41b1bb1d8 Bump software.amazon.awssdk:bom from 2.29.29 to 2.29.30
N
dependabot[bot] opened a new pull request, #14626:
URL: https://github.com/apache/pinot/pull/14626
Bumps software.amazon.awssdk:bom from 2.29.29 to 2.29.30.
Most Recent Ignore Conditions Applied to This Pull Request
| Dependency Name | Ignore Conditions |
| --- | --- |
davecromberge opened a new pull request, #14625:
URL: https://github.com/apache/pinot/pull/14625
Feature
Give the user more control over how sketches are merged in different time
buckets for the Merge Rollup Task.
Examples where this could be helpful is if the user
tibrewalpratik17 commented on code in PR #14611:
URL: https://github.com/apache/pinot/pull/14611#discussion_r1877868802
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/timeseries/serde/TimeSeriesBlockSerde.java:
##
@@ -0,0 +1,203 @@
+/**
+ * Licensed to the A
tibrewalpratik17 commented on code in PR #14611:
URL: https://github.com/apache/pinot/pull/14611#discussion_r1877862949
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/timeseries/serde/TimeSeriesBlockSerde.java:
##
@@ -0,0 +1,203 @@
+/**
+ * Licensed to the A
tibrewalpratik17 commented on code in PR #14611:
URL: https://github.com/apache/pinot/pull/14611#discussion_r1877850019
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/timeseries/serde/TimeSeriesBlockSerde.java:
##
@@ -0,0 +1,203 @@
+/**
+ * Licensed to the A
Harnoor7 commented on code in PR #14623:
URL: https://github.com/apache/pinot/pull/14623#discussion_r1877724647
##
pinot-common/src/main/java/org/apache/pinot/common/minion/RealtimeToOfflineSegmentsTaskMetadata.java:
##
@@ -41,19 +47,37 @@
public class RealtimeToOfflineSegments
swaminathanmanish merged PR #14373:
URL: https://github.com/apache/pinot/pull/14373
--
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...@
This is an automated email from the ASF dual-hosted git repository.
manishswaminathan 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 442c0fc713 Configurable sketch accurac
41 matches
Mail list logo
