gortiz opened a new pull request, #12864:
URL: https://github.com/apache/pinot/pull/12864
Versions below 3.5.1 may have issues with classes compiled in Java 20 and
21, which we may find in multi-release jar dependencies
--
This is an automated message from the Apache Git Service.
To r
shounakmk219 commented on code in PR #12856:
URL: https://github.com/apache/pinot/pull/12856#discussion_r1558934875
##
docker/images/pinot/etc/jmx_prometheus_javaagent/configs/broker.yml:
##
@@ -1,173 +1,173 @@
rules:
-- pattern: "\"org.apache.pinot.common.metrics\"<>(\\w+)"
+-
shounakmk219 commented on PR #12856:
URL: https://github.com/apache/pinot/pull/12856#issuecomment-2046641801
> > It also replaces *? with * for custom metric rules as the surrounding
pattern matching is well defined.
>
> Can you please elaborate more on this? Maybe you can use an exam
gortiz commented on PR #12808:
URL: https://github.com/apache/pinot/pull/12808#issuecomment-2046628265
> I feel we can remove the method level `synchronize` and declare the field
as `volatile` ?
I think that would be safe as well, although the reasoning may be a bit more
complex. Giv
shounakmk219 commented on code in PR #12856:
URL: https://github.com/apache/pinot/pull/12856#discussion_r1558915239
##
docker/images/pinot/etc/jmx_prometheus_javaagent/configs/broker.yml:
##
@@ -1,173 +1,173 @@
rules:
-- pattern: "\"org.apache.pinot.common.metrics\"<>(\\w+)"
+-
gortiz commented on code in PR #12808:
URL: https://github.com/apache/pinot/pull/12808#discussion_r1558916116
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/memory/unsafe/DirectMemory.java:
##
@@ -56,8 +56,12 @@ public void flush() {
@Override
public synch
itschrispeck commented on code in PR #12744:
URL: https://github.com/apache/pinot/pull/12744#discussion_r1558836542
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/mutable/MutableIndex.java:
##
@@ -60,4 +60,12 @@ public interface MutableIndex extends IndexR
itschrispeck commented on code in PR #12744:
URL: https://github.com/apache/pinot/pull/12744#discussion_r1558835084
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/creator/impl/text/LuceneTextIndexCreator.java:
##
@@ -90,14 +102,17 @@ public static Ha
itschrispeck commented on code in PR #12744:
URL: https://github.com/apache/pinot/pull/12744#discussion_r1558833921
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/realtime/impl/invertedindex/RealtimeLuceneTextIndex.java:
##
@@ -181,6 +181,17 @@ private Mutab
deepthi912 commented on issue #12860:
URL: https://github.com/apache/pinot/issues/12860#issuecomment-2046526106
I think failures are due to Lucene95Codec, Lucene95HnswVectorsFormat which
are currently not in 9.10.0 version.
--
This is an automated message from the Apache Git Service.
To
codecov-commenter commented on PR #12863:
URL: https://github.com/apache/pinot/pull/12863#issuecomment-2046499970
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12863?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
swaminathanmanish commented on code in PR #12812:
URL: https://github.com/apache/pinot/pull/12812#discussion_r1558784988
##
pinot-plugins/pinot-stream-ingestion/pinot-pulsar/src/main/java/org/apache/pinot/plugin/stream/pulsar/PulsarPartitionLevelConsumer.java:
##
@@ -45,93 +40,6
swaminathanmanish opened a new pull request, #12863:
URL: https://github.com/apache/pinot/pull/12863
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
codecov-commenter commented on PR #12862:
URL: https://github.com/apache/pinot/pull/12862#issuecomment-2046460827
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12862?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
sullis opened a new pull request, #12862:
URL: https://github.com/apache/pinot/pull/12862
This project uses Mockito for mock objects. Therefore, we will ban EasyMock
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
abhioncbr commented on PR #12440:
URL: https://github.com/apache/pinot/pull/12440#issuecomment-2046294955
@Jackie-Jiang / @klsince, please review it once more. Thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
abhioncbr commented on issue #12858:
URL: https://github.com/apache/pinot/issues/12858#issuecomment-2046251639
I will look into it. Thanks
--
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 spec
abhioncbr commented on issue #12860:
URL: https://github.com/apache/pinot/issues/12860#issuecomment-2046251450
I can take a look. Assign it to me. Thanks
--
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
abhioncbr commented on issue #12861:
URL: https://github.com/apache/pinot/issues/12861#issuecomment-2046250619
Assign it to me please thanks
--
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 s
swaminathanmanish commented on code in PR #12812:
URL: https://github.com/apache/pinot/pull/12812#discussion_r1558500972
##
pinot-plugins/pinot-stream-ingestion/pinot-pulsar/src/main/java/org/apache/pinot/plugin/stream/pulsar/PulsarPartitionLevelConsumer.java:
##
@@ -45,93 +40,6
jackjlli commented on PR #12838:
URL: https://github.com/apache/pinot/pull/12838#issuecomment-2046224113
I did test out the cases when forward index is disabled for the realtime
table. And here is the exception messages:
```
2024/04/09 16:31:42.220 INFO [FixedByteSVMutableForwardIndex]
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 13673f1150 Add SchemaConformingTransformerV2 to
chenboat merged PR #12788:
URL: https://github.com/apache/pinot/pull/12788
--
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
Jackie-Jiang commented on PR #12838:
URL: https://github.com/apache/pinot/pull/12838#issuecomment-2046124115
My point is that forward index cannot be disabled for consuming segment
anyway, regardless of whether `columnMajorSegmentBuilderEnabled` is enabled or
not. Have you tried disabling f
jackjlli commented on PR #12838:
URL: https://github.com/apache/pinot/pull/12838#issuecomment-2046113187
Right, I was about to paste you the same
[link](https://github.com/apache/pinot/blob/master/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/indexsegment/mutable/MutableSe
Jackie-Jiang merged PR #12859:
URL: https://github.com/apache/pinot/pull/12859
--
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 PR #12812:
URL: https://github.com/apache/pinot/pull/12812#issuecomment-2046102481
> Thanks Jackie. From the code, it does not look like there are any changes
in behavior except that its done in a different way without the executor. Just
wanted to confirm.
I
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 425eea9e80 Bump org.apache.maven.plugins:maven-javadoc-plugin from
3.2.0 to 3.6.3 (#12852)
add c93de37765 remove
Jackie-Jiang commented on code in PR #12812:
URL: https://github.com/apache/pinot/pull/12812#discussion_r1558332571
##
pinot-plugins/pinot-stream-ingestion/pinot-pulsar/src/main/java/org/apache/pinot/plugin/stream/pulsar/PulsarPartitionLevelConsumer.java:
##
@@ -45,93 +40,66 @@
Jackie-Jiang commented on code in PR #12812:
URL: https://github.com/apache/pinot/pull/12812#discussion_r1558333101
##
pinot-plugins/pinot-stream-ingestion/pinot-pulsar/src/main/java/org/apache/pinot/plugin/stream/pulsar/PulsarPartitionLevelConsumer.java:
##
@@ -45,93 +40,66 @@
Jackie-Jiang commented on PR #12838:
URL: https://github.com/apache/pinot/pull/12838#issuecomment-2046098039
I don't think forward index can be disabled for mutable segment (see this
[line](https://github.com/apache/pinot/blob/425eea9e80dd81c633481488a12c1c62cc7a2c91/pinot-segment-local/src/
Jackie-Jiang commented on PR #12838:
URL: https://github.com/apache/pinot/pull/12838#issuecomment-2046090959
What I was asking is when we disable forward index, will the config be
picked up for consuming segment? Will the consuming segment only create
inverted index, but not forward index?
jackjlli commented on PR #12838:
URL: https://github.com/apache/pinot/pull/12838#issuecomment-2046083662
> Do we allow disabling forward index for consuming segment? Can we add a
test to verify if it works?
With the change in this current PR no we won't allow disabling forward index
swaminathanmanish commented on code in PR #12812:
URL: https://github.com/apache/pinot/pull/12812#discussion_r1558308899
##
pinot-plugins/pinot-stream-ingestion/pinot-pulsar/src/main/java/org/apache/pinot/plugin/stream/pulsar/PulsarPartitionLevelConsumer.java:
##
@@ -45,93 +40,6
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-javadoc-plugin-3.6.3
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 1c37415bab Bump org.apache.maven.plugins:maven-javadoc-p
Jackie-Jiang merged PR #12852:
URL: https://github.com/apache/pinot/pull/12852
--
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 7e4ea06b99 use static TypeReference object (#12776)
add 425eea9e80 Bump org.apache.maven.plugins:maven-javadoc-plu
Jackie-Jiang commented on code in PR #12808:
URL: https://github.com/apache/pinot/pull/12808#discussion_r1558289310
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/memory/unsafe/DirectMemory.java:
##
@@ -56,8 +56,12 @@ public void flush() {
@Override
public
Jackie-Jiang merged PR #12776:
URL: https://github.com/apache/pinot/pull/12776
--
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 07682d24f8 Bump com.azure:azure-identity from 1.11.3 to 1.12.0 (#12855)
add 7e4ea06b99 use static TypeReference ob
Jackie-Jiang commented on issue #12801:
URL: https://github.com/apache/pinot/issues/12801#issuecomment-2046039885
cc @snleee @swaminathanmanish
--
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 th
Jackie-Jiang commented on code in PR #12813:
URL: https://github.com/apache/pinot/pull/12813#discussion_r1558283670
##
pinot-spi/src/main/java/org/apache/pinot/spi/stream/StreamMessageDecoder.java:
##
@@ -44,9 +45,10 @@ public interface StreamMessageDecoder {
* @param fields
Jackie-Jiang merged PR #12855:
URL: https://github.com/apache/pinot/pull/12855
--
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/com.azure-azure-identity-1.12.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 64fa753f38 Bump com.azure:azure-identity from 1.11.3 to 1.12.0
The revision
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 3a7cef3155 Bump net.openhft:chronicle-core from 2.25ea10 to 2.25ea13
(#12854)
add 07682d24f8 Bump com.azure:azure
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 d9b1f040bf Bump com.microsoft.azure:msal4j from 1.14.3 to 1.15.0
(#12853)
add 3a7cef3155 Bump net.openhft:chronic
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/net.openhft-chronicle-core-2.25ea13
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 22ef79e8e5 Bump net.openhft:chronicle-core from 2.25ea10 to 2.25ea13
Th
Jackie-Jiang merged PR #12854:
URL: https://github.com/apache/pinot/pull/12854
--
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/com.microsoft.azure-msal4j-1.15.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was cd6fee632f Bump com.microsoft.azure:msal4j from 1.14.3 to 1.15.0
The revi
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 d9b1f040bf Bump com.microsoft.azure:msal4j from 1
Jackie-Jiang merged PR #12853:
URL: https://github.com/apache/pinot/pull/12853
--
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 e001a884e4 Bump com.gradle:common-custom-user-dat
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/com.gradle-common-custom-user-data-maven-extension-2
in repository https://gitbox.apache.org/repos/asf/pinot.git
was f203e8b4b5 Bump com.gradle:common-custom-user-data-mave
Jackie-Jiang merged PR #12850:
URL: https://github.com/apache/pinot/pull/12850
--
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/com.google.apis-google-api-services-storage-v1-rev20240319-2.0.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was c5bcdf0f1b Bump com.google.apis:google-api-
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 abf45fc6e0 Bump com.google.errorprone:error_prone_annotations from
2.25.0 to 2.26.1 (#12847)
add df5fcb7b0f Bump
Jackie-Jiang merged PR #12848:
URL: https://github.com/apache/pinot/pull/12848
--
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/com.google.errorprone-error_prone_annotations-2.26.1
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 5b17efc281 Bump com.google.errorprone:error_prone_annot
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 73ef57b3af Bump org.apache:apache from 21 to 31 (#12846)
add abf45fc6e0 Bump com.google.errorprone:error_prone_ann
Jackie-Jiang merged PR #12847:
URL: https://github.com/apache/pinot/pull/12847
--
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/org.apache-apache-31
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 5685a2da34 Bump org.apache:apache from 21 to 31
The revisions that were on this branch
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch dependabot/maven/pulsar.version-3.2.2
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 986277bdc5 Bump pulsar.version from 2.11.4 to 3.2.2
The revisions that were on this
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 c722607743 Bump aws.sdk.version from 2.25.17 to 2.25.27 (#12843)
add 73ef57b3af Bump org.apache:apache from 21 to
Jackie-Jiang merged PR #12846:
URL: https://github.com/apache/pinot/pull/12846
--
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
dependabot[bot] commented on PR #12845:
URL: https://github.com/apache/pinot/pull/12845#issuecomment-2046007646
OK, I won't notify you again about this release, but will get in touch when
a new version is available. You can also ignore all major, minor, or patch
releases for a dependency by
Jackie-Jiang commented on PR #12845:
URL: https://github.com/apache/pinot/pull/12845#issuecomment-2046007590
Tracked in #12713, need some manual fix
--
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 t
Jackie-Jiang closed pull request #12845: Bump pulsar.version from 2.11.4 to
3.2.2
URL: https://github.com/apache/pinot/pull/12845
--
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.
github-bot pushed a change to branch
dependabot/maven/org.codehaus.woodstox-stax2-api-4.2.2
in repository https://gitbox.apache.org/repos/asf/pinot.git
discard e2311b8272 Bump org.codehaus.woodstox:stax2-api from 4.1 to 4.2.2
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 101af7c583 add SegmentContext to collect validDocIds bitmaps for many
segments together (#12694)
add b5148ebfeb B
Jackie-Jiang merged PR #12843:
URL: https://github.com/apache/pinot/pull/12843
--
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 #12842:
URL: https://github.com/apache/pinot/pull/12842
--
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/aws.sdk.version-2.25.27
in repository https://gitbox.apache.org/repos/asf/pinot.git
was ff444303fe Bump aws.sdk.version from 2.25.17 to 2.25.27
The revisions that were on 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 b5148ebfeb Bump cloud.localstack:localstack-utils from 0.2.19 to
0.2.23 (#12842)
add c722607743 Bump aws.sdk.vers
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/cloud.localstack-localstack-utils-0.2.23
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 5250ac04c3 Bump cloud.localstack:localstack-utils from 0.2.19 to 0.
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 620c46acd3 Move package org.apache.calcite to org.apache.pinot.calcite
(#12837)
add 101af7c583 add SegmentContext t
klsince merged PR #12694:
URL: https://github.com/apache/pinot/pull/12694
--
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
hpvd commented on issue #12858:
URL: https://github.com/apache/pinot/issues/12858#issuecomment-2045910970
notable changes:
https://orc.apache.org/releases/
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
chenboat commented on code in PR #12744:
URL: https://github.com/apache/pinot/pull/12744#discussion_r1558152978
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/mutable/MutableIndex.java:
##
@@ -60,4 +60,12 @@ public interface MutableIndex extends IndexReade
codecov-commenter commented on PR #12859:
URL: https://github.com/apache/pinot/pull/12859#issuecomment-2045873576
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12859?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
chenboat commented on code in PR #12744:
URL: https://github.com/apache/pinot/pull/12744#discussion_r1558146827
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/creator/impl/text/LuceneTextIndexCreator.java:
##
@@ -90,14 +102,17 @@ public static HashSe
chenboat commented on code in PR #12744:
URL: https://github.com/apache/pinot/pull/12744#discussion_r1558134729
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/realtime/impl/invertedindex/RealtimeLuceneTextIndex.java:
##
@@ -181,6 +181,17 @@ private MutableRo
lnbest0707-uber commented on PR #12788:
URL: https://github.com/apache/pinot/pull/12788#issuecomment-2045820585
Some extreme column name corner case checks would be taken care in future
patch, e.g., column name with `a.`, input data with `{"a": {"b":1}, "a.b":2,
"a.":3, "a.b.":4}`
--
Thi
lnbest0707-uber commented on code in PR #12788:
URL: https://github.com/apache/pinot/pull/12788#discussion_r1558107343
##
pinot-spi/src/main/java/org/apache/pinot/spi/config/table/ingestion/SchemaConformingTransformerV2Config.java:
##
@@ -0,0 +1,253 @@
+/**
+ * Licensed to the A
sullis opened a new pull request, #12859:
URL: https://github.com/apache/pinot/pull/12859
This project uses Mockito for mock objects. We don't need to use EasyMock
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
chenboat commented on code in PR #12744:
URL: https://github.com/apache/pinot/pull/12744#discussion_r1558089422
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/realtime/converter/RealtimeSegmentConverter.java:
##
@@ -118,6 +118,9 @@ public void build(@Nullabl
Jackie-Jiang commented on code in PR #12856:
URL: https://github.com/apache/pinot/pull/12856#discussion_r1558089252
##
docker/images/pinot/etc/jmx_prometheus_javaagent/configs/broker.yml:
##
@@ -1,173 +1,173 @@
rules:
-- pattern: "\"org.apache.pinot.common.metrics\"<>(\\w+)"
+-
chenboat commented on code in PR #12744:
URL: https://github.com/apache/pinot/pull/12744#discussion_r1558086670
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/indexsegment/mutable/MutableSegmentImpl.java:
##
@@ -932,6 +932,19 @@ public Object getValue(int do
zhtaoxiang commented on PR #12856:
URL: https://github.com/apache/pinot/pull/12856#issuecomment-2045767927
> It also replaces *? with * for custom metric rules as the surrounding
pattern matching is well defined.
Can you please elaborate more on this? Maybe you can use an example to
aditya0811 commented on issue #12367:
URL: https://github.com/apache/pinot/issues/12367#issuecomment-2045759962
Hello Jackie, I was able to reproduce the error by changing `protected
static final int NUM_ROWS = 1` and below in `setUp()` in
`BaseTransformFunctionTest.java`.
```
floatS
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch
add-validation-check-for-forward-index-disabled-when-enabling-columnar-segment-creation
in repository https://gitbox.apache.org/repos/asf/pinot.git
discard 4c2fca142d Add validation check for for
robertzych commented on issue #12805:
URL: https://github.com/apache/pinot/issues/12805#issuecomment-2045708477
GeoServer doesn't strictly require the missing functions
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
swaminathanmanish opened a new issue, #12857:
URL: https://github.com/apache/pinot/issues/12857
**Problem**:
Currently [RealtimeToOfflineSegmentsTask that is used to move real time
segments to offline
segments](https://docs.pinot.apache.org/operators/operating-pinot/pinot-managed-offline
swaminathanmanish commented on issue #12857:
URL: https://github.com/apache/pinot/issues/12857#issuecomment-2045650083
cc @aishikbh - Please take a look
--
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
gio-startree opened a new pull request, #112:
URL: https://github.com/apache/pinot-site/pull/112
(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
davecromberge commented on PR #12685:
URL: https://github.com/apache/pinot/pull/12685#issuecomment-2045245091
> Can you please help update the Pinot doc about this function?
@Jackie-Jiang here is the accompanying pull request to the documentation
site:
https://github.com/pinot-cont
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 278bf8c9 Wb 280 add webex (#111
xiangfu0 merged PR #111:
URL: https://github.com/apache/pinot-site/pull/111
--
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
deepthi912 commented on issue #10375:
URL: https://github.com/apache/pinot/issues/10375#issuecomment-2045045145
Is anyone still working on this issue? I can take a look if not.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
hpvd commented on issue #11441:
URL: https://github.com/apache/pinot/issues/11441#issuecomment-2045052419
Perhaps this is something to keep in mind when futher refining the metrics:
https://github.com/apache/pinot/issues/12840
--
This is an automated message from the Apache Git Servic
shounakmk219 opened a new pull request, #12856:
URL: https://github.com/apache/pinot/pull/12856
# Description
The `.` requires escaping throughout the pattern string if it is intended to
represent a `.` literal. The only reason its working right now is because its
surrounded by specific
1 - 100 of 162 matches
Mail list logo
