xiangfu0 closed issue #12742: Skip node level throttle during server bootstrap
URL: https://github.com/apache/pinot/issues/12742
--
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.
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 1393750622 Allow Server throttling just before e
xiangfu0 merged PR #12930:
URL: https://github.com/apache/pinot/pull/12930
--
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.apa
ankitsultana commented on code in PR #12538:
URL: https://github.com/apache/pinot/pull/12538#discussion_r1566736270
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/HashUtils.java:
##
@@ -44,8 +73,31 @@ public static Object hashPrimaryKey(PrimaryKey prim
codecov-commenter commented on PR #12938:
URL: https://github.com/apache/pinot/pull/12938#issuecomment-2058266890
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12938?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
codecov-commenter commented on PR #12930:
URL: https://github.com/apache/pinot/pull/12930#issuecomment-2058261465
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12930?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
codecov-commenter commented on PR #12937:
URL: https://github.com/apache/pinot/pull/12937#issuecomment-2058246151
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12937?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
xiangfu0 merged PR #12385:
URL: https://github.com/apache/pinot/pull/12385
--
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.apa
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/pinot.git
from ec452a49f3 Refine PeerServerSegmentFinder (#12933)
add 3c45469c39 Update superset docker build script (#12385)
N
klsince opened a new pull request, #12938:
URL: https://github.com/apache/pinot/pull/12938
Try to use refCnt to track threads using partition upsert metadata mgr so
that we can be sure when we can reset the isPreloading flag, so that the mgr
can load segments via preloading again, e.g. when
xiangfu0 closed pull request #12882: Bump com.tdunning:t-digest from 3.2 to 3.3
URL: https://github.com/apache/pinot/pull/12882
--
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.
xiangfu0 opened a new pull request, #12937:
URL: https://github.com/apache/pinot/pull/12937
Bumps
[org.codehaus.mojo:buildnumber-maven-plugin](https://github.com/mojohaus/buildnumber-maven-plugin)
from 1.3 to 3.2.0.
--
This is an automated message from the Apache Git Service.
To respond
deemoliu commented on code in PR #12437:
URL: https://github.com/apache/pinot/pull/12437#discussion_r1566583241
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -597,6 +596,39 @@ public static String splitPart(String input, String
deemoliu commented on code in PR #12437:
URL: https://github.com/apache/pinot/pull/12437#discussion_r1566582209
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -597,6 +596,39 @@ public static String splitPart(String input, String
deemoliu commented on code in PR #12437:
URL: https://github.com/apache/pinot/pull/12437#discussion_r1566580709
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -597,6 +596,39 @@ public static String splitPart(String input, String
codecov-commenter commented on PR #12936:
URL: https://github.com/apache/pinot/pull/12936#issuecomment-2058042624
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12936?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
deemoliu commented on code in PR #12437:
URL: https://github.com/apache/pinot/pull/12437#discussion_r1566580853
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -597,6 +596,39 @@ public static String splitPart(String input, String
Jackie-Jiang commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1566574337
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/plan/MultiStageQueryStats.java:
##
@@ -0,0 +1,668 @@
+/**
+ * Licensed to the Apache Software F
Jackie-Jiang commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1566570540
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/MetadataBlock.java:
##
@@ -36,108 +39,129 @@
*/
public class MetadataBlock extends BaseDataBlock
Jackie-Jiang commented on code in PR #12392:
URL: https://github.com/apache/pinot/pull/12392#discussion_r1566565835
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -581,6 +586,107 @@ public static String[] split(String input, Str
Jackie-Jiang opened a new pull request, #12936:
URL: https://github.com/apache/pinot/pull/12936
Separated from #12886 for easier review
- Make `SegmentFetcherFactory` a util class
- Integrate the new `API` with URI supplier from `SegmentFetcher` into
`SegmentFetcherFactory`
--
This
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 ec452a49f3 Refine PeerServerSegmentFinder (#12933
Jackie-Jiang merged PR #12933:
URL: https://github.com/apache/pinot/pull/12933
--
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
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/pinot.git
from edf9d53597 Specify version for commons-validator (#12935)
add c08ba2ccd5 Bump org.apache.maven.scm:maven-scm-provi
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/org.apache.maven.scm-maven-scm-provider-gitexe-2.1.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 068abcfc4d Bump org.apache.maven.scm:maven-scm-provider
Jackie-Jiang merged PR #12925:
URL: https://github.com/apache/pinot/pull/12925
--
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
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 edf9d53597 Specify version for commons-validator
Jackie-Jiang merged PR #12935:
URL: https://github.com/apache/pinot/pull/12935
--
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 opened a new pull request, #12935:
URL: https://github.com/apache/pinot/pull/12935
Solves the dependency conflict from confluent libraries
--
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 g
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/pinot.git
from 684cece35e Bump org.apache.maven.plugins:maven-jar-plugin from 3.3.0
to 3.4.0 (#12927)
add 013435a6a1 Bump aws.sd
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/aws.sdk.version-2.25.31
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 5863c220e8 Bump aws.sdk.version from 2.25.30 to 2.25.31
The revisions that were on t
Jackie-Jiang commented on issue #8537:
URL: https://github.com/apache/pinot/issues/8537#issuecomment-2057959769
Still flaky..
--
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.
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 684cece35e Bump org.apache.maven.plugins:maven-ja
Jackie-Jiang merged PR #12928:
URL: https://github.com/apache/pinot/pull/12928
--
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 merged PR #12926:
URL: https://github.com/apache/pinot/pull/12926
--
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
walterddr opened a new issue, #8537:
URL: https://github.com/apache/pinot/issues/8537
Error is occurring to Pulsar test case randomly on multiple of my PRs
```
PulsarConsumerTest.testPartitionLevelConsumerBatchMessages:246 expected
[500] but found [491]
```
CI Logs. Cli
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/org.apache.maven.plugins-maven-jar-plugin-3.4.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 3130039947 Bump org.apache.maven.plugins:maven-jar-plugin fr
Jackie-Jiang merged PR #12927:
URL: https://github.com/apache/pinot/pull/12927
--
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
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/slf4j.version-2.0.13
in repository https://gitbox.apache.org/repos/asf/pinot.git
was dfcda2cd41 Bump slf4j.version from 2.0.12 to 2.0.13
The revisions that were on this bra
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 2459cfce75 Bump slf4j.version from 2.0.12 to 2.0.
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch clean-up-task-manager-for-tables
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 7068a246e9 Do task related cleanup for non-controller leader
This change permanently discards
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch new-site-dev
in repository https://gitbox.apache.org/repos/asf/pinot-site.git
The following commit(s) were added to refs/heads/new-site-dev by this push:
new 17939a33 ReactPlayer, use Head
xiangfu0 merged PR #115:
URL: https://github.com/apache/pinot-site/pull/115
--
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
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch clean-up-task-manager-for-tables
in repository https://gitbox.apache.org/repos/asf/pinot.git
commit 7068a246e9851329a7580cbba4479b8599aa64a2
Author: jlli_LinkedIn
AuthorDate: Mon Apr 15 15:39:13 20
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch clean-up-task-manager-for-tables
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 7068a246e9 Do task related cleanup for non-controller leader
This branch includes the followin
Jackie-Jiang commented on PR #12837:
URL: https://github.com/apache/pinot/pull/12837#issuecomment-2057916250
This change is introducing backward incompatibility because we rely on class
path during query plan ser/de, and `PinotRelExchangeType` is part of the ser/de.
We should fix the ser/
deemoliu commented on code in PR #12392:
URL: https://github.com/apache/pinot/pull/12392#discussion_r1566393961
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -570,6 +572,107 @@ public static String[] split(String input, String
deemoliu commented on code in PR #12392:
URL: https://github.com/apache/pinot/pull/12392#discussion_r1566496110
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -570,6 +572,107 @@ public static String[] split(String input, String
codecov-commenter commented on PR #12933:
URL: https://github.com/apache/pinot/pull/12933#issuecomment-2057865306
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12933?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
klsince merged PR #12863:
URL: https://github.com/apache/pinot/pull/12863
--
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.apac
This is an automated email from the ASF dual-hosted git repository.
xbli pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from c823430f11 Bump io.netty:netty-bom from 4.1.108.Final to 4.1.109.Final
(#12929)
add af8fd4089c Using local copy of
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch dependabot/maven/slf4j.version-2.0.13
in repository https://gitbox.apache.org/repos/asf/pinot.git
discard 6c539cc661 Bump slf4j.version from 2.0.12 to 2.0.13
add 2c6a84b8ec Improved seg
Jackie-Jiang merged PR #12929:
URL: https://github.com/apache/pinot/pull/12929
--
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
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/io.netty-netty-bom-4.1.109.Final
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 598ec59b76 Bump io.netty:netty-bom from 4.1.108.Final to 4.1.109.Final
The
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/pinot.git
from 2c6a84b8ec Improved segment build time for Lucene text index realtime
to offline conversion (#12744)
add c823430f
Jackie-Jiang commented on code in PR #12933:
URL: https://github.com/apache/pinot/pull/12933#discussion_r1566428022
##
pinot-common/src/main/java/org/apache/pinot/core/util/PeerServerSegmentFinder.java:
##
@@ -47,93 +46,73 @@ public class PeerServerSegmentFinder {
private Pee
klsince commented on code in PR #12933:
URL: https://github.com/apache/pinot/pull/12933#discussion_r1566387588
##
pinot-common/src/main/java/org/apache/pinot/core/util/PeerServerSegmentFinder.java:
##
@@ -47,93 +46,73 @@ public class PeerServerSegmentFinder {
private PeerServ
Jackie-Jiang opened a new pull request, #12933:
URL: https://github.com/apache/pinot/pull/12933
Separated from #12886 for easier review
- Refine and simplify the logic of `PeerServerSegmentFinder`
- Fix the bug of using netty port when admin port is not explicitly
configured
--
codecov-commenter commented on PR #12932:
URL: https://github.com/apache/pinot/pull/12932#issuecomment-2057616056
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12932?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
gio-startree opened a new pull request, #115:
URL: https://github.com/apache/pinot-site/pull/115
(no comment)
--
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
gio-startree closed pull request #114: Try with ReactPlayer
URL: https://github.com/apache/pinot-site/pull/114
--
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
PeterCorless commented on issue #10919:
URL: https://github.com/apache/pinot/issues/10919#issuecomment-2057576972
See [PR#11977](https://github.com/apache/pinot/pull/11977)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
lnbest0707-uber opened a new pull request, #12932:
URL: https://github.com/apache/pinot/pull/12932
tags: `feature` `bugfix`
Followup from PR https://github.com/apache/pinot/pull/12788 with:
- Getter/setter fix on SchemaConformingTransformerV2Config
- Schema and config validation
codecov-commenter commented on PR #12931:
URL: https://github.com/apache/pinot/pull/12931#issuecomment-2057527084
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12931?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
lnbest0707-uber opened a new pull request, #12931:
URL: https://github.com/apache/pinot/pull/12931
Instructions:
tags: `bugfix`
--
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 com
gio-startree opened a new pull request, #114:
URL: https://github.com/apache/pinot-site/pull/114
(no comment)
--
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
This is an automated email from the ASF dual-hosted git repository.
tingchen 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 2c6a84b8ec Improved segment build time for Luce
chenboat merged PR #12744:
URL: https://github.com/apache/pinot/pull/12744
--
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.apa
deepthi912 opened a new pull request, #12930:
URL: https://github.com/apache/pinot/pull/12930
Fix for the following issue:
--
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
sullis commented on PR #12928:
URL: https://github.com/apache/pinot/pull/12928#issuecomment-2056796309
LGTM
https://www.slf4j.org/news.html
https://github.com/apache/pinot/assets/30938/9518dd06-c722-475a-bd32-6dbfa823fef8";>
--
This is an automated message from the Apache
sullis commented on PR #12929:
URL: https://github.com/apache/pinot/pull/12929#issuecomment-2056794602
LGTM
https://netty.io/news/2024/04/15/4-1-109-Final.html
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/io.netty-netty-bom-4.1.109.Final
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 598ec59b76 Bump io.netty:netty-bom from 4.1.108.Final to 4.1.109.Final
dependabot[bot] opened a new pull request, #12929:
URL: https://github.com/apache/pinot/pull/12929
Bumps [io.netty:netty-bom](https://github.com/netty/netty) from
4.1.108.Final to 4.1.109.Final.
Commits
https://github.com/netty/netty/commit/43455dfe98202b8dc63d89ac3f04027c3e8f9
dependabot[bot] opened a new pull request, #12928:
URL: https://github.com/apache/pinot/pull/12928
Bumps `slf4j.version` from 2.0.12 to 2.0.13.
Updates `org.slf4j:jcl-over-slf4j` from 2.0.12 to 2.0.13
Updates `org.slf4j:slf4j-api` from 2.0.12 to 2.0.13
Dependabot will res
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/org.apache.maven.scm-maven-scm-provider-gitexe-2.1.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 068abcfc4d Bump org.apache.maven.scm:maven-scm-prov
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/org.apache.maven.plugins-maven-jar-plugin-3.4.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 3130039947 Bump org.apache.maven.plugins:maven-jar-plugi
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch dependabot/maven/aws.sdk.version-2.25.31
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 5863c220e8 Bump aws.sdk.version from 2.25.30 to 2.25.31
No new revisions were ad
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch dependabot/maven/slf4j.version-2.0.13
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 6c539cc661 Bump slf4j.version from 2.0.12 to 2.0.13
No new revisions were added by
dependabot[bot] opened a new pull request, #12927:
URL: https://github.com/apache/pinot/pull/12927
Bumps
[org.apache.maven.plugins:maven-jar-plugin](https://github.com/apache/maven-jar-plugin)
from 3.3.0 to 3.4.0.
Release notes
Sourced from https://github.com/apache/maven-jar-plug
dependabot[bot] opened a new pull request, #12926:
URL: https://github.com/apache/pinot/pull/12926
Bumps `aws.sdk.version` from 2.25.30 to 2.25.31.
Updates `software.amazon.awssdk:bom` from 2.25.30 to 2.25.31
Updates `software.amazon.awssdk:apache-client` from 2.25.30 to 2.25.31
dependabot[bot] opened a new pull request, #12925:
URL: https://github.com/apache/pinot/pull/12925
Bumps org.apache.maven.scm:maven-scm-provider-gitexe from 2.0.1 to 2.1.0.
[ {
@Overr
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565397287
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/mailbox/channel/MailboxContentObserver.java:
##
@@ -88,6 +71,8 @@ public void onNext(MailboxContent mailboxCo
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565395943
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/mailbox/channel/MailboxContentObserver.java:
##
@@ -62,22 +58,9 @@ public void onNext(MailboxContent mailboxC
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565391241
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/operator/LeafStageTransferableBlockOperator.java:
##
@@ -99,10 +105,32 @@ public
LeafStageTransferab
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565387972
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/mailbox/GrpcSendingMailbox.java:
##
@@ -64,25 +69,31 @@ public void send(TransferableBlock block)
retu
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565385608
##
pinot-query-planner/src/main/java/org/apache/pinot/calcite/rel/rules/PinotJoinToDynamicBroadcastRule.java:
##
@@ -164,11 +165,15 @@ public void onMatch(RelOptRuleCall
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565384127
##
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/ExecutionStatsAggregator.java:
##
@@ -364,25 +357,6 @@ public void setStats(@Nullable String rawTableName,
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565379309
##
pinot-common/src/main/java/org/apache/pinot/common/response/broker/BrokerResponseNativeV2.java:
##
@@ -61,36 +67,31 @@ public BrokerResponseNativeV2(List
exceptions)
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565358229
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/DataBlockUtils.java:
##
@@ -62,13 +67,13 @@ public static MetadataBlock getEndOfStreamDataBlock() {
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565356393
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/BaseDataBlock.java:
##
@@ -572,14 +585,9 @@ private Map
deserializeExceptions(ByteBuffer buffer)
@Ove
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565315967
##
pinot-broker/src/test/java/org/apache/pinot/broker/querylog/QueryLoggerTest.java:
##
@@ -259,11 +259,9 @@ private QueryLogger.QueryLogParams generateParams(boolean
i
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565313954
##
pinot-broker/src/test/java/org/apache/pinot/broker/querylog/QueryLoggerTest.java:
##
@@ -259,11 +259,9 @@ private QueryLogger.QueryLogParams generateParams(boolean
i
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1565308960
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/DataBlock.java:
##
@@ -80,8 +80,6 @@ byte[] toBytes()
@Nullable
RoaringBitmap getNullRowIds(int col
1 - 100 of 102 matches
Mail list logo
