Lvnszn closed issue #7818:
URL: https://github.com/apache/pinot/issues/7818
--
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...@pi
Lvnszn commented on issue #7818:
URL: https://github.com/apache/pinot/issues/7818#issuecomment-977606846
Thanks for your reply. IMO, the tableConfig is base on schemaConfig, so give
me a hint than the rule in tableConfig higher weight than schema.
--
This is an automated message from th
codecov-commenter edited a comment on pull request #7364:
URL: https://github.com/apache/pinot/pull/7364#issuecomment-945155571
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7364?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
siddharthteotia commented on a change in pull request #7823:
URL: https://github.com/apache/pinot/pull/7823#discussion_r755762444
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java
##
@@ -459,7 +459,13 @@ private Broke
codecov-commenter edited a comment on pull request #7364:
URL: https://github.com/apache/pinot/pull/7364#issuecomment-945155571
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7364?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7364:
URL: https://github.com/apache/pinot/pull/7364#issuecomment-945155571
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7364?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7364:
URL: https://github.com/apache/pinot/pull/7364#issuecomment-945155571
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7364?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7825:
URL: https://github.com/apache/pinot/pull/7825#issuecomment-977547948
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7825?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7825:
URL: https://github.com/apache/pinot/pull/7825#issuecomment-977547948
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7825?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7825:
URL: https://github.com/apache/pinot/pull/7825#issuecomment-977547948
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7825?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter commented on pull request #7825:
URL: https://github.com/apache/pinot/pull/7825#issuecomment-977547948
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7825?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apac
xiangfu0 opened a new pull request #7825:
URL: https://github.com/apache/pinot/pull/7825
## Description
Replaced with
```
bin/pinot-admin.sh LaunchDataIngestionJob -jobSpecFile
examples/batch/airlineStats/ingestionJobSpec.yaml
```
## Upgrade Notes
Does this PR prevent
kautshukla opened a new pull request #7824:
URL: https://github.com/apache/pinot/pull/7824
Prerequisite:
pinot/pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotDriver.java
headers map is not empty {JDBC with headers for Auth enabled client}
As of n
kautshukla closed pull request #7774:
URL: https://github.com/apache/pinot/pull/7774
--
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-unsub
kautshukla commented on pull request #7774:
URL: https://github.com/apache/pinot/pull/7774#issuecomment-977530469
@Jackie-Jiang : Will be closing and re creating pull request. Mistakenly
pushed to down stream.
--
This is an automated message from the Apache Git Service.
To respond to the
codecov-commenter edited a comment on pull request #7823:
URL: https://github.com/apache/pinot/pull/7823#issuecomment-977499036
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7823?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter commented on pull request #7823:
URL: https://github.com/apache/pinot/pull/7823#issuecomment-977499036
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7823?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apac
Jackie-Jiang commented on a change in pull request #7568:
URL: https://github.com/apache/pinot/pull/7568#discussion_r755656841
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java
##
@@ -519,6 +543,18 @@ private BrokerRe
Jackie-Jiang commented on a change in pull request #7568:
URL: https://github.com/apache/pinot/pull/7568#discussion_r755656841
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java
##
@@ -519,6 +543,18 @@ private BrokerRe
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 819cba0 Fix the broker routing when segment is deleted (#7817)
add 90f5881 Pass Pinot Connection properties fr
xiangfu0 merged pull request #7822:
URL: https://github.com/apache/pinot/pull/7822
--
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-unsubsc
jadami10 opened a new pull request #7823:
URL: https://github.com/apache/pinot/pull/7823
## Description
This fixes #7811. In #7264 we added exceptions when not all segments are
available, but this was one case that was missed. Specifically if all servers
are unavailable, both realti
codecov-commenter edited a comment on pull request #7822:
URL: https://github.com/apache/pinot/pull/7822#issuecomment-977308961
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7822?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7822:
URL: https://github.com/apache/pinot/pull/7822#issuecomment-977308961
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7822?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7822:
URL: https://github.com/apache/pinot/pull/7822#issuecomment-977308961
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7822?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#issuecomment-977259274
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7820?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7774:
URL: https://github.com/apache/pinot/pull/7774#issuecomment-977346867
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7774?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter commented on pull request #7774:
URL: https://github.com/apache/pinot/pull/7774#issuecomment-977346867
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7774?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apac
codecov-commenter edited a comment on pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#issuecomment-977259274
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7820?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#issuecomment-977259274
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7820?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7822:
URL: https://github.com/apache/pinot/pull/7822#issuecomment-977308961
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7822?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
Jackie-Jiang commented on a change in pull request #7822:
URL: https://github.com/apache/pinot/pull/7822#discussion_r755606608
##
File path:
pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotConnection.java
##
@@ -40,11 +41,20 @@
private PinotContro
Jackie-Jiang merged pull request #7817:
URL: https://github.com/apache/pinot/pull/7817
--
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-uns
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 a74d944 Adding pinot client connection config to allow skip fail on
broker response exception (#7816)
add 819cb
Jackie-Jiang commented on issue #7791:
URL: https://github.com/apache/pinot/issues/7791#issuecomment-977312406
Just went over the code, and we are actually using the proper metadata push
API (the other way is letting the controller download the segment and extract
the metadata). The ineffi
codecov-commenter commented on pull request #7822:
URL: https://github.com/apache/pinot/pull/7822#issuecomment-977308961
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7822?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apac
codecov-commenter edited a comment on pull request #7817:
URL: https://github.com/apache/pinot/pull/7817#issuecomment-977204199
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7817?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
Jackie-Jiang commented on issue #7818:
URL: https://github.com/apache/pinot/issues/7818#issuecomment-977303001
This is expected behavior. Pinot will skip the transform if the column
already exists
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
codecov-commenter edited a comment on pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#issuecomment-977259274
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7820?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7817:
URL: https://github.com/apache/pinot/pull/7817#issuecomment-977204199
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7817?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
xiangfu0 edited a comment on issue #7818:
URL: https://github.com/apache/pinot/issues/7818#issuecomment-977289127
I think by default pinot extract all the top level fields from JSON into a
PinotRecord for consuming, then evaluate each transform function.
So filed `sku` is already extract
codecov-commenter edited a comment on pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#issuecomment-977259274
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7820?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter edited a comment on pull request #7817:
URL: https://github.com/apache/pinot/pull/7817#issuecomment-977204199
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7817?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
xiangfu0 commented on issue #7818:
URL: https://github.com/apache/pinot/issues/7818#issuecomment-977289127
I think by default pinot extract all the top level fields into a record for
consuming, then evaluate each transform function.
So filed `sku` is already extracted then the transform
codecov-commenter edited a comment on pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#issuecomment-977259274
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7820?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
kkrugler edited a comment on issue #7818:
URL: https://github.com/apache/pinot/issues/7818#issuecomment-977282279
Hmm, others should weigh in here, but since the JSON record ALSO contains an
`sku` field at the top level, this looks like undefined behavior to me, not a
bug.
--
This is an
kkrugler commented on issue #7818:
URL: https://github.com/apache/pinot/issues/7818#issuecomment-977282279
Hmm, others should weigh in here, but since your record ALSO contains an
`sku` field at the top level, this looks like undefined behavior to me, not a
bug. Why not call your column so
xiangfu0 opened a new pull request #7822:
URL: https://github.com/apache/pinot/pull/7822
## Description
Passing info properties also to Pinot Connection.
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order:
Controller, Broker, Server, Minion)
richardstartin commented on pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#issuecomment-977264186
The try/catches can be removed if
https://github.com/json-path/JsonPath/pull/767 is merged and released.
--
This is an automated message from the Apache Git Service.
To r
codecov-commenter edited a comment on pull request #7781:
URL: https://github.com/apache/pinot/pull/7781#issuecomment-971204560
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7781?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter commented on pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#issuecomment-977259274
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7820?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apac
codecov-commenter edited a comment on pull request #7781:
URL: https://github.com/apache/pinot/pull/7781#issuecomment-971204560
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7781?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
richardstartin opened a new issue #7821:
URL: https://github.com/apache/pinot/issues/7821
I have made several improvements and bug fixes to the [JsonPath
library](https://github.com/json-path/JsonPath) since September, motivated by
improving Pinot, and these changes have been merged but no
kkrugler commented on issue #7791:
URL: https://github.com/apache/pinot/issues/7791#issuecomment-977233280
Hi @Jackie-Jiang - yes, re using a metadata push so that the controller
doesn't have to download the segment. But I'm curious what the 2 versions of
metadata push were. I only know of
mqliang edited a comment on issue #6720:
URL: https://github.com/apache/pinot/issues/6720#issuecomment-808974129
I write a bench mark here:
https://github.com/mqliang/pinot/commit/a32a61aad5dfa6b6c4a09064c75926b00495cd3a
The benchmark compares three ways to build a data table:
* `
richardstartin commented on a change in pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#discussion_r72508
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/common/DataBlockCache.java
##
@@ -120,6 +121,17 @@ public int getNumDocs() {
retur
richardstartin commented on a change in pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#discussion_r71967
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/common/DataBlockCache.java
##
@@ -120,6 +121,17 @@ public int getNumDocs() {
retur
xiangfu0 commented on issue #7818:
URL: https://github.com/apache/pinot/issues/7818#issuecomment-977228027
> Hi @Lvnszn - are you saying that because the "sku" field in the incoming
JSON record IS NOT in the `custom_props` sub-record, you don't think it should
wind up in the row?
ye
mayankshriv commented on a change in pull request #7820:
URL: https://github.com/apache/pinot/pull/7820#discussion_r70432
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/common/DataBlockCache.java
##
@@ -120,6 +121,17 @@ public int getNumDocs() {
return i
richardstartin opened a new pull request #7820:
URL: https://github.com/apache/pinot/pull/7820
Pushes JSON path evaluation down to the storage layer (giving direct access
to dictionaries and forward index) which avoids various intermediate
materialisations of strings, byte arrays and so on
codecov-commenter commented on pull request #7817:
URL: https://github.com/apache/pinot/pull/7817#issuecomment-977204199
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7817?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apac
lakshmanan-v commented on a change in pull request #7781:
URL: https://github.com/apache/pinot/pull/7781#discussion_r755529666
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/GapFillGroupByDataTableReducer.java
##
@@ -0,0 +1,490 @@
+/**
+ * Licensed t
weixiangsun commented on a change in pull request #7781:
URL: https://github.com/apache/pinot/pull/7781#discussion_r755528039
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/GapFillGroupByDataTableReducer.java
##
@@ -0,0 +1,490 @@
+/**
+ * Licensed to
kkrugler commented on issue #7818:
URL: https://github.com/apache/pinot/issues/7818#issuecomment-977187482
Hi @Lvnszn - are you saying that because the "sku" field in the incoming
JSON record IS NOT in the `custom_props` sub-record, you don't think it should
wind up in the row?
--
This
Jackie-Jiang commented on a change in pull request #7817:
URL: https://github.com/apache/pinot/pull/7817#discussion_r755511591
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/routing/RoutingManager.java
##
@@ -110,94 +110,99 @@ public void init(HelixManager he
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 4aef0ba change default range index version to v2 (#7815)
add a74d944 Adding pinot client connection config to
xiangfu0 closed issue #7770:
URL: https://github.com/apache/pinot/issues/7770
--
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...@
xiangfu0 merged pull request #7816:
URL: https://github.com/apache/pinot/pull/7816
--
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-unsubsc
xiangfu0 commented on a change in pull request #7817:
URL: https://github.com/apache/pinot/pull/7817#discussion_r754805996
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/routing/RoutingManager.java
##
@@ -110,94 +110,99 @@ public void init(HelixManager helixM
Jackie-Jiang commented on a change in pull request #7817:
URL: https://github.com/apache/pinot/pull/7817#discussion_r755459241
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/routing/RoutingManager.java
##
@@ -110,94 +110,99 @@ public void init(HelixManager he
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 767aa8a Proceed to next time window when no segment match for
RealtimeToOfflineTask (#7814)
add 4aef0ba change
Jackie-Jiang merged pull request #7815:
URL: https://github.com/apache/pinot/pull/7815
--
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-uns
Jackie-Jiang edited a comment on pull request #7815:
URL: https://github.com/apache/pinot/pull/7815#issuecomment-977083516
@siddharthteotia Yes, reader can read all different versions, and the reader
support is included in `0.9.0` release. Existing segments won't be modified
--
This is a
Jackie-Jiang commented on pull request #7815:
URL: https://github.com/apache/pinot/pull/7815#issuecomment-977083516
@siddharthteotia Yes, reader can read all different versions, and the reader
support is included in `0.9.0` release
--
This is an automated message from the Apache Git Serv
lfernandez93 commented on issue #5246:
URL: https://github.com/apache/pinot/issues/5246#issuecomment-977078950
+1 is there any update on this?
--
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
siddharthteotia commented on pull request #7815:
URL: https://github.com/apache/pinot/pull/7815#issuecomment-977068519
To confirm, reader should still be able to read existing segments with old
version right ? Are we planning to convert existing segments to V3 during
segment preprocessor /
lfernandez93 commented on issue #5246:
URL: https://github.com/apache/pinot/issues/5246#issuecomment-977027075
@kishoreg how do we up vote this 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 g
jackjlli commented on a change in pull request #7817:
URL: https://github.com/apache/pinot/pull/7817#discussion_r755425417
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/routing/RoutingManager.java
##
@@ -110,94 +110,99 @@ public void init(HelixManager helixM
weixiangsun commented on a change in pull request #7781:
URL: https://github.com/apache/pinot/pull/7781#discussion_r755366473
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/GapFillGroupByDataTableReducer.java
##
@@ -0,0 +1,490 @@
+/**
+ * Licensed to
weixiangsun commented on a change in pull request #7781:
URL: https://github.com/apache/pinot/pull/7781#discussion_r755367238
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/PostAggregationGapfillQueriesTest.java
##
@@ -0,0 +1,550 @@
+/**
+ * Licensed to the Ap
weixiangsun commented on a change in pull request #7781:
URL: https://github.com/apache/pinot/pull/7781#discussion_r755366473
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/GapFillGroupByDataTableReducer.java
##
@@ -0,0 +1,490 @@
+/**
+ * Licensed to
weixiangsun commented on a change in pull request #7781:
URL: https://github.com/apache/pinot/pull/7781#discussion_r755365577
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/GapFillGroupByDataTableReducer.java
##
@@ -0,0 +1,490 @@
+/**
+ * Licensed to
weixiangsun commented on a change in pull request #7781:
URL: https://github.com/apache/pinot/pull/7781#discussion_r755364417
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/GapFillGroupByDataTableReducer.java
##
@@ -0,0 +1,490 @@
+/**
+ * Licensed to
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/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 767aa8a Proceed to next time window when no s
npawar closed issue #6988:
URL: https://github.com/apache/pinot/issues/6988
--
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...@pi
npawar merged pull request #7814:
URL: https://github.com/apache/pinot/pull/7814
--
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.
mneedham commented on issue #7789:
URL: https://github.com/apache/pinot/issues/7789#issuecomment-976614503
This is where we've got to so far -
https://docs.pinot.apache.org/users/user-guide-query/filtering-with-idset
--
This is an automated message from the Apache Git Service.
To respond
codecov-commenter edited a comment on pull request #7819:
URL: https://github.com/apache/pinot/pull/7819#issuecomment-976515875
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7819?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
codecov-commenter commented on pull request #7819:
URL: https://github.com/apache/pinot/pull/7819#issuecomment-976515875
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7819?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apac
richardstartin commented on pull request #7802:
URL: https://github.com/apache/pinot/pull/7802#issuecomment-976509437
@Jackie-Jiang it depends. Constructing bitmaps from unordered values is
inefficient, and in general there's no point in using `RoaringBitmapWriter`,
so if the dictionary i
richardstartin edited a comment on pull request #7819:
URL: https://github.com/apache/pinot/pull/7819#issuecomment-976445015
It turns out there's a bug in the library which prevents this configuration
from being respected in all cases, which I have fixed in a [PR to the
library](https://gi
richardstartin commented on a change in pull request #7819:
URL: https://github.com/apache/pinot/pull/7819#discussion_r755082884
##
File path:
pinot-common/src/test/java/org/apache/pinot/common/function/scalar/ArrayAwareJacksonJsonProviderTest.java
##
@@ -48,15 +49,11 @@ publi
richardstartin commented on a change in pull request #7819:
URL: https://github.com/apache/pinot/pull/7819#discussion_r755082361
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/JsonFunctions.java
##
@@ -149,50 +159,45 @@ public static String js
richardstartin commented on pull request #7819:
URL: https://github.com/apache/pinot/pull/7819#issuecomment-976445015
It turns out there's a bug in the library which prevents this configuration
from being respected in all cases, which I have fixed in a PR to the library.
Converting to draf
codecov-commenter edited a comment on pull request #7815:
URL: https://github.com/apache/pinot/pull/7815#issuecomment-976097602
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7815?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
richardstartin opened a new pull request #7819:
URL: https://github.com/apache/pinot/pull/7819
By default, when a JSON path doesn't match a document, the JSONPath library
throws an exception, but this can be disabled with configuration. These
exceptions can be quite costly during ingestion
codecov-commenter commented on pull request #7816:
URL: https://github.com/apache/pinot/pull/7816#issuecomment-976405029
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7816?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apac
Lvnszn opened a new issue #7818:
URL: https://github.com/apache/pinot/issues/7818
## TableConfig
```
{
"REALTIME": {
"tableName": "tracker_analyze_REALTIME",
"tableType": "REALTIME",
"segmentsConfig": {
"schemaName": "tracker_analyze",
"timeC
richardstartin commented on pull request #7815:
URL: https://github.com/apache/pinot/pull/7815#issuecomment-976393298
@Jackie-Jiang I see, that's a different error, it didn't get that far before
I force pushed the first time.
--
This is an automated message from the Apache Git Service.
T
99 matches
Mail list logo
