codecov-commenter edited a comment on pull request #7724:
URL: https://github.com/apache/pinot/pull/7724#issuecomment-963674976
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7724?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
mqliang closed pull request #7724:
URL: https://github.com/apache/pinot/pull/7724
--
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.
apucher 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 2075cbe correct way to send headers via url con
apucher merged pull request #7728:
URL: https://github.com/apache/pinot/pull/7728
--
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
codecov-commenter edited a comment on pull request #7728:
URL: https://github.com/apache/pinot/pull/7728#issuecomment-963846406
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7728?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 #7728:
URL: https://github.com/apache/pinot/pull/7728#issuecomment-963846406
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7728?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apac
klsince opened a new pull request #7728:
URL: https://github.com/apache/pinot/pull/7728
## Description
with this fix, the custom headers are sent out properly like below
```
$ nc -l 8000 // starting a dummy server
POST /query/sql HTTP/1.1
Authorization: . // a custom h
walterddr closed pull request #7726:
URL: https://github.com/apache/pinot/pull/7726
--
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-unsubs
walterddr commented on pull request #7726:
URL: https://github.com/apache/pinot/pull/7726#issuecomment-963800392
doesnt' work. closing
--
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 specifi
codecov-commenter edited a comment on pull request #7724:
URL: https://github.com/apache/pinot/pull/7724#issuecomment-963674976
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7724?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 #7724:
URL: https://github.com/apache/pinot/pull/7724#issuecomment-963674976
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7724?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 #7725:
URL: https://github.com/apache/pinot/pull/7725#issuecomment-963679063
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7725?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
mqliang closed pull request #7724:
URL: https://github.com/apache/pinot/pull/7724
--
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
codecov-commenter edited a comment on pull request #7678:
URL: https://github.com/apache/pinot/pull/7678#issuecomment-962252170
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7678?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 #7678:
URL: https://github.com/apache/pinot/pull/7678#issuecomment-962252170
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7678?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 #7678:
URL: https://github.com/apache/pinot/pull/7678#issuecomment-962252170
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7678?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
mqliang closed pull request #7724:
URL: https://github.com/apache/pinot/pull/7724
--
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
codecov-commenter edited a comment on pull request #7725:
URL: https://github.com/apache/pinot/pull/7725#issuecomment-963679063
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7725?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
jasperjiaguo edited a comment on pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#issuecomment-963705986
> Thank you for the updates.
>
> Would you mind still adding the unit test for a different keystore type?
It doesn't just serve robustness but similarly as docum
apucher commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745209268
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public interfac
apucher commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745209268
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public interfac
amrishlal commented on pull request #7568:
URL: https://github.com/apache/pinot/pull/7568#issuecomment-963714474
@mayankshriv @Jackie-Jiang @kishoreg Please review. Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
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 94a17f3 memoize literals in query context in order to deduplicate
them (#7720)
add 39daba6 format query parser
Jackie-Jiang merged pull request #7727:
URL: https://github.com/apache/pinot/pull/7727
--
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
xiangfu0 opened a new pull request #7727:
URL: https://github.com/apache/pinot/pull/7727
## Description
Format 3 tests
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order:
Controller, Broker, Server, Minion)
* [ ] Yes (Please label as **back
jasperjiaguo commented on pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#issuecomment-963705986
> Thank you for the updates.
>
> Would you mind still adding the unit test for a different keystore type?
It doesn't just serve robustness but similarly as documentatio
siddharthteotia commented on a change in pull request #7702:
URL: https://github.com/apache/pinot/pull/7702#discussion_r745199786
##
File path:
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/reader/SubStructureSelector.java
##
@@ -0,0 +1,25 @@
+/**
+ * Lice
siddharthteotia commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745198982
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public
siddharthteotia commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745198982
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public
xiangfu0 closed issue #7673:
URL: https://github.com/apache/pinot/issues/7673
--
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...@
walterddr commented on issue #7673:
URL: https://github.com/apache/pinot/issues/7673#issuecomment-963701902
I think we can close this one @xiangfu0
--
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
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 ada94c2 follow up with picocli migration (#7705)
add 94a17f3 memoize literals in query context in order to dedu
Jackie-Jiang merged pull request #7720:
URL: https://github.com/apache/pinot/pull/7720
--
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
jasperjiaguo commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745193796
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/transport/QueryServer.java
##
@@ -110,7 +119,9 @@ private void attachSSLHandler(SocketCha
codecov-commenter edited a comment on pull request #7725:
URL: https://github.com/apache/pinot/pull/7725#issuecomment-963679063
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7725?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
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 67b5b67 Fix the flaky SimpleMinionClusterIntegrationTest (#7723)
add ada94c2 follow up with picocli migration (
Jackie-Jiang merged pull request #7705:
URL: https://github.com/apache/pinot/pull/7705
--
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 closed issue #7673:
URL: https://github.com/apache/pinot/issues/7673
--
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
Jackie-Jiang commented on a change in pull request #7661:
URL: https://github.com/apache/pinot/pull/7661#discussion_r745184469
##
File path:
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/readers/forward/VarByteChunkSVForwardIndexReaderV4.java
#
codecov-commenter edited a comment on pull request #7725:
URL: https://github.com/apache/pinot/pull/7725#issuecomment-963679063
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7725?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 #7725:
URL: https://github.com/apache/pinot/pull/7725#issuecomment-963679063
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7725?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 #7724:
URL: https://github.com/apache/pinot/pull/7724#issuecomment-963674976
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7724?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 #7692:
URL: https://github.com/apache/pinot/pull/7692#issuecomment-961259336
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7692?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 #7724:
URL: https://github.com/apache/pinot/pull/7724#issuecomment-963674976
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7724?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 #7705:
URL: https://github.com/apache/pinot/pull/7705#issuecomment-961526091
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7705?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 #7705:
URL: https://github.com/apache/pinot/pull/7705#issuecomment-961526091
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7705?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 #7692:
URL: https://github.com/apache/pinot/pull/7692#issuecomment-961259336
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7692?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
apucher commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745172852
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public interfac
Jackie-Jiang opened a new pull request #7725:
URL: https://github.com/apache/pinot/pull/7725
## Description
All the functionality of `SegmentConverter` is replaced by
`SegmentProcessorFramework`.
## Release Notes
`MergeSegments` is no longer available in `pinot-admin`, use
`Se
mqliang opened a new pull request #7724:
URL: https://github.com/apache/pinot/pull/7724
## Description
This PR split the ThreadCpuTimeNs metric into 3 parts:
* threadCpuTimeNs
* systemActivitiesCpuTimeNs
* responseSerializationCpuTimeNs
Splitting will help us under
siddharthteotia commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745156486
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public
siddharthteotia commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745156486
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public
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 52ec46e Remove the AutomatonBasedRegexpPredicateEvaluator and use
regular RegexpLikePredicateEvaluator (#7715)
Jackie-Jiang merged pull request #7723:
URL: https://github.com/apache/pinot/pull/7723
--
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 merged pull request #7715:
URL: https://github.com/apache/pinot/pull/7715
--
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 2605122 Allow adding JSON data type for dimension column types (#7718)
add 52ec46e Remove the AutomatonBasedReg
apucher commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745150036
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public interfac
jasperjiaguo commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745140386
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public int
codecov-commenter edited a comment on pull request #7723:
URL: https://github.com/apache/pinot/pull/7723#issuecomment-963623525
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7723?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 #7723:
URL: https://github.com/apache/pinot/pull/7723#issuecomment-963623525
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7723?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apac
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 2605122 Allow adding JSON data type for dimensio
Jackie-Jiang merged pull request #7718:
URL: https://github.com/apache/pinot/pull/7718
--
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 closed issue #7717:
URL: https://github.com/apache/pinot/issues/7717
--
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.
xiangfu pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git.
from 7a24845 inline binary comparison ops to prevent function call
overhead (#7709)
add a17bdd7 Skip pinot.admin.s
xiangfu0 merged pull request #7722:
URL: https://github.com/apache/pinot/pull/7722
--
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
Jackie-Jiang opened a new pull request #7723:
URL: https://github.com/apache/pinot/pull/7723
When submitting a minion task, the actual task scheduling is generated
async, and might not be reflected immediately. Added a 10s wait for verifying
the task count.
Sample failure:
https://
jackjlli commented on a change in pull request #7721:
URL: https://github.com/apache/pinot/pull/7721#discussion_r745105710
##
File path:
pinot-plugins/pinot-batch-ingestion/v0_deprecated/pinot-hadoop/src/main/java/org/apache/pinot/hadoop/job/preprocess/DataPreprocessingHelper.java
###
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch fix-data-preprocess-sdf-pattern
in repository https://gitbox.apache.org/repos/asf/pinot.git.
discard 4c265fc Fix SDF pattern in DataPreprocessingHelper
add 751e8f6 Fix SDF pattern in DataPr
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch fix-data-preprocess-sdf-pattern
in repository https://gitbox.apache.org/repos/asf/pinot.git.
discard 2e3b0a0 Fix SDF pattern in DataPreprocessingHelper
add 4c265fc Fix SDF pattern in DataPr
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch fix-data-preprocess-sdf-pattern
in repository https://gitbox.apache.org/repos/asf/pinot.git.
discard 33ff293 Fix SDF pattern in DataPreprocessingHelper
add 2e3b0a0 Fix SDF pattern in DataPr
xiangfu0 commented on pull request #7722:
URL: https://github.com/apache/pinot/pull/7722#issuecomment-963576472
cc @kkrugler
--
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
Jackie-Jiang commented on a change in pull request #7692:
URL: https://github.com/apache/pinot/pull/7692#discussion_r745091691
##
File path:
pinot-connectors/pinot-spark-connector/src/test/scala/org/apache/pinot/connector/spark/connector/PinotUtilsTest.scala
##
@@ -19,12 +19,1
walterddr commented on a change in pull request #7722:
URL: https://github.com/apache/pinot/pull/7722#discussion_r745088200
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java
##
@@ -192,7 +192,11 @@ public static void main(String[] arg
walterddr commented on a change in pull request #7705:
URL: https://github.com/apache/pinot/pull/7705#discussion_r745087356
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/segment/converter/DictionaryToRawIndexConverter.java
##
@@ -262,20 +268,28 @@ private void
walterddr commented on a change in pull request #7705:
URL: https://github.com/apache/pinot/pull/7705#discussion_r745086708
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/segment/converter/DictionaryToRawIndexConverter.java
##
@@ -262,20 +268,28 @@ private void
xiangfu0 commented on a change in pull request #7722:
URL: https://github.com/apache/pinot/pull/7722#discussion_r745086080
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java
##
@@ -192,7 +192,11 @@ public static void main(String[] args
kkrugler commented on issue #7716:
URL: https://github.com/apache/pinot/issues/7716#issuecomment-963556599
I think the `StreamDataServerStartable` implementations are only needed for
tests and quickstart (aka demo), so part of this cleanup could be to move that
interface out of `pinot-spi`
codecov-commenter edited a comment on pull request #7721:
URL: https://github.com/apache/pinot/pull/7721#issuecomment-963511124
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7721?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
apucher commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745060458
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public interfac
Jackie-Jiang commented on a change in pull request #7722:
URL: https://github.com/apache/pinot/pull/7722#discussion_r745057862
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java
##
@@ -192,7 +192,11 @@ public static void main(String[]
jasperjiaguo commented on a change in pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#discussion_r745054917
##
File path:
pinot-core/src/main/java/org/apache/pinot/server/access/AccessControl.java
##
@@ -27,6 +28,13 @@
@InterfaceStability.Stable
public int
walterddr commented on a change in pull request #7722:
URL: https://github.com/apache/pinot/pull/7722#discussion_r745054684
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java
##
@@ -192,7 +192,11 @@ public static void main(String[] arg
jasperjiaguo edited a comment on pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#issuecomment-963519248
> Its not clear why we are changing the package name for
AllowAllAccessFactory
Hello @kishoreg, we moved the access control classes from pinot-server to
pinot-c
walterddr commented on a change in pull request #7722:
URL: https://github.com/apache/pinot/pull/7722#discussion_r745053508
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java
##
@@ -192,7 +192,11 @@ public static void main(String[] arg
xiangfu0 opened a new pull request #7722:
URL: https://github.com/apache/pinot/pull/7722
## Description
QuickStart commands starting using `bin/pinot-admin.sh QuickStart -type XXX`
will exit once the table is bootstrapped.
This is extremely annoying when running pinot from docker.
jasperjiaguo commented on pull request #7653:
URL: https://github.com/apache/pinot/pull/7653#issuecomment-963519248
> Its not clear why we are changing the package name for
AllowAllAccessFactory
Hello Kishore, we moved the access control classes from pinot-server to
pinot-core since
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 827c1ce lighter weight LiteralTransformFunction, avoid excessive
array fills (#7707)
add 7a24845 inline binary
Jackie-Jiang merged pull request #7709:
URL: https://github.com/apache/pinot/pull/7709
--
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
codecov-commenter commented on pull request #7721:
URL: https://github.com/apache/pinot/pull/7721#issuecomment-963511124
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7721?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 #7653:
URL: https://github.com/apache/pinot/pull/7653#issuecomment-953400988
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7653?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 #7653:
URL: https://github.com/apache/pinot/pull/7653#issuecomment-953400988
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/7653?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=T
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 f6d7a51 optimise getUnpaddedString with SWAR padding search (#7708)
add 827c1ce lighter weight LiteralTransform
Jackie-Jiang merged pull request #7707:
URL: https://github.com/apache/pinot/pull/7707
--
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
xiangfu0 commented on a change in pull request #7705:
URL: https://github.com/apache/pinot/pull/7705#discussion_r745026799
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/segment/converter/DictionaryToRawIndexConverter.java
##
@@ -262,20 +268,28 @@ private void
Jackie-Jiang merged pull request #7708:
URL: https://github.com/apache/pinot/pull/7708
--
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 3b0cce4 allow to carry headers with query requests (#7712)
add f6d7a51 optimise getUnpaddedString with SWAR pad
richardstartin commented on a change in pull request #7721:
URL: https://github.com/apache/pinot/pull/7721#discussion_r745020242
##
File path:
pinot-plugins/pinot-batch-ingestion/v0_deprecated/pinot-hadoop/src/main/java/org/apache/pinot/hadoop/job/preprocess/DataPreprocessingHelper.ja
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 91d2a95 Clean up controller-table related metrics in
ControllerPeriodicTask (#7557)
add 3b0cce4 allow to carry
Jackie-Jiang merged pull request #7712:
URL: https://github.com/apache/pinot/pull/7712
--
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
jackjlli opened a new pull request #7721:
URL: https://github.com/apache/pinot/pull/7721
## Description
The SDF pattern can be null if the time column isn't simple date format
(e.g. it can be timestamp), and this will make the set() method with the
following exception:
```
Caused
1 - 100 of 119 matches
Mail list logo
