codecov-commenter edited a comment on pull request #8312:
URL: https://github.com/apache/pinot/pull/8312#issuecomment-1061430158
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8312?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
codecov-commenter commented on pull request #8312:
URL: https://github.com/apache/pinot/pull/8312#issuecomment-1061430158
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8312?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
codecov-commenter edited a comment on pull request #8311:
URL: https://github.com/apache/pinot/pull/8311#issuecomment-1061348277
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8311?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
xiangfu0 closed pull request #5719:
URL: https://github.com/apache/pinot/pull/5719
--
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
This is an automated email from the ASF dual-hosted git repository.
sajjad 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 a208954 Exclude protobuf-java explicitly in tran
sajjad-moradi merged pull request #8307:
URL: https://github.com/apache/pinot/pull/8307
--
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-un
codecov-commenter edited a comment on pull request #8307:
URL: https://github.com/apache/pinot/pull/8307#issuecomment-1061392302
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8307?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
dongxiaoman opened a new pull request #8313:
URL: https://github.com/apache/pinot/pull/8313
## Description
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order:
Controller, Broker, Server, Minion)
* [ ] Yes (Please label as **backward-incompat**
codecov-commenter edited a comment on pull request #8307:
URL: https://github.com/apache/pinot/pull/8307#issuecomment-1061392302
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8307?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
codecov-commenter edited a comment on pull request #8307:
URL: https://github.com/apache/pinot/pull/8307#issuecomment-1061392302
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8307?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
codecov-commenter edited a comment on pull request #8311:
URL: https://github.com/apache/pinot/pull/8311#issuecomment-1061348277
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8311?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
codecov-commenter commented on pull request #8307:
URL: https://github.com/apache/pinot/pull/8307#issuecomment-1061392302
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8307?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
codecov-commenter edited a comment on pull request #8311:
URL: https://github.com/apache/pinot/pull/8311#issuecomment-1061348277
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8311?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
codecov-commenter commented on pull request #8309:
URL: https://github.com/apache/pinot/pull/8309#issuecomment-1061391619
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8309?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
codecov-commenter edited a comment on pull request #8305:
URL: https://github.com/apache/pinot/pull/8305#issuecomment-1061382612
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8305?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
codecov-commenter commented on pull request #8305:
URL: https://github.com/apache/pinot/pull/8305#issuecomment-1061382612
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8305?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
npawar commented on a change in pull request #8309:
URL: https://github.com/apache/pinot/pull/8309#discussion_r821288753
##
File path:
pinot-plugins/pinot-stream-ingestion/pinot-kafka-2.0/src/test/java/org/apache/pinot/plugin/stream/kafka20/utils/MiniKafkaCluster.java
##
@@ -1
npawar commented on a change in pull request #8309:
URL: https://github.com/apache/pinot/pull/8309#discussion_r821288060
##
File path:
pinot-plugins/pinot-stream-ingestion/pinot-kafka-2.0/src/main/java/org/apache/pinot/plugin/stream/kafka20/KafkaPartitionLevelConnectionHandler.java
##
npawar commented on a change in pull request #8309:
URL: https://github.com/apache/pinot/pull/8309#discussion_r821287802
##
File path:
pinot-plugins/pinot-stream-ingestion/pinot-kafka-2.0/src/main/java/org/apache/pinot/plugin/stream/kafka20/KafkaPartitionLevelConnectionHandler.java
##
npawar commented on a change in pull request #8309:
URL: https://github.com/apache/pinot/pull/8309#discussion_r821287602
##
File path:
pinot-plugins/pinot-stream-ingestion/pinot-kafka-2.0/src/main/java/org/apache/pinot/plugin/stream/kafka20/KafkaPartitionLevelConnectionHandler.java
##
This is an automated email from the ASF dual-hosted git repository.
mayanks pushed a change to branch dev
in repository https://gitbox.apache.org/repos/asf/pinot-site.git.
from cb06927 Merge pull request #67 from apache/0.9.3-release-website
new b67c148 New company logos
new 002f
mayankshriv merged pull request #68:
URL: https://github.com/apache/pinot-site/pull/68
--
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 edited a comment on pull request #8311:
URL: https://github.com/apache/pinot/pull/8311#issuecomment-1061348277
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8311?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
codecov-commenter commented on pull request #8311:
URL: https://github.com/apache/pinot/pull/8311#issuecomment-1061348277
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8311?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
walterddr commented on a change in pull request #8305:
URL: https://github.com/apache/pinot/pull/8305#discussion_r821269900
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/ZookeeperResource.java
##
@@ -84,14 +85,7 @@ public String getData
tuor713 commented on issue #7849:
URL: https://github.com/apache/pinot/issues/7849#issuecomment-1061342606
The issue of rows missing during segment flush seem to come from the fact
that there is period where there are two consuming segments (the old one being
rolled into immutable and the
navina commented on a change in pull request #8309:
URL: https://github.com/apache/pinot/pull/8309#discussion_r821269269
##
File path:
pinot-plugins/pinot-stream-ingestion/pinot-kafka-2.0/src/main/java/org/apache/pinot/plugin/stream/kafka20/KafkaPartitionLevelConnectionHandler.java
##
navina commented on a change in pull request #8309:
URL: https://github.com/apache/pinot/pull/8309#discussion_r821266312
##
File path:
pinot-plugins/pinot-stream-ingestion/pinot-kafka-2.0/src/main/java/org/apache/pinot/plugin/stream/kafka20/KafkaPartitionLevelConnectionHandler.java
##
mayankshriv opened a new pull request #8311:
URL: https://github.com/apache/pinot/pull/8311
## Description
Use the appAssembler plugin to publish `query-runner.sh` as a standalone
tool, based on the `QueryRunner` class.
## Upgrade Notes
Does this PR prevent a zero down-time up
weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r821252308
##
File path:
pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java
##
@@ -369,6 +369,9 @@ private static PinotQuery
compileCa
weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r821252308
##
File path:
pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java
##
@@ -369,6 +369,9 @@ private static PinotQuery
compileCa
joshigaurava opened a new pull request #68:
URL: https://github.com/apache/pinot-site/pull/68
https://user-images.githubusercontent.com/68965367/157148599-fc17aa5e-dfb5-4c80-962e-27f3cdc2a739.png";>
--
This is an automated message from the Apache Git Service.
To respond to the message
Jackie-Jiang commented on issue #8254:
URL: https://github.com/apache/pinot/issues/8254#issuecomment-1061306192
#8310
--
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 u
Jackie-Jiang opened a new pull request #8310:
URL: https://github.com/apache/pinot/pull/8310
In #7269 we added support for null time handling, but with 2 limitations:
- It is not enabled by default
- It does not support custom default value
This PR addressed these 2 limitations
npawar commented on issue #8219:
URL: https://github.com/apache/pinot/issues/8219#issuecomment-1061300556
@lfernandez93 yes, i was able to reproduce. I've submitted a fix
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
GSharayu commented on pull request #8307:
URL: https://github.com/apache/pinot/pull/8307#issuecomment-1061293754
Thank you for this 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 to the specif
GSharayu commented on pull request #8307:
URL: https://github.com/apache/pinot/pull/8307#issuecomment-1061293670
LGTM
--
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 un
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch exclude-protobuf-java
in repository https://gitbox.apache.org/repos/asf/pinot.git
commit cdfe6066536c93ff67ce4ed460a6f6ca624a0233
Author: Jack Li(Analytics Engineering)
AuthorDate: Mon Mar 7 11:58:
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch exclude-protobuf-java
in repository https://gitbox.apache.org/repos/asf/pinot.git.
discard 6a4fc1f Exclude protobuf-java explicitly in transitive dependencies
add 890b9be During rebalance,
npawar opened a new pull request #8309:
URL: https://github.com/apache/pinot/pull/8309
For https://github.com/apache/pinot/issues/8219
The root cause of issue was that the kafka sets the OffsetResetStrategy in
SubscriptionState to "latest" by default.
From `KafkaConsumer`
```
jasperjiaguo commented on pull request #8307:
URL: https://github.com/apache/pinot/pull/8307#issuecomment-1061225307
> The lint tests were failing due to dependency issues after merging PR
#8300 cc @jasperjiaguo I think we can just upgrade to a single version to avoid
the conflict ? The fl
colemast commented on issue #7849:
URL: https://github.com/apache/pinot/issues/7849#issuecomment-1061223857
This is a very interesting thread and great analysis. I can see issue 1 is
being addressed and issue 3 seems like a limitation with the trino query
pattern of hitting segments. Has a
Jackie-Jiang opened a new pull request #8308:
URL: https://github.com/apache/pinot/pull/8308
## Description
Allow broker to automatically rewrite query with transform to derived column
if exists
## Release Notes
Add a new config `enableQueryRewrite` to `TransformConfig` to enab
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch exclude-protobuf-java
in repository https://gitbox.apache.org/repos/asf/pinot.git.
discard 6a4fc1f Exclude protobuf-java explicitly in transitive dependencies
add 890b9be During rebalance,
Jackie-Jiang commented on a change in pull request #8305:
URL: https://github.com/apache/pinot/pull/8305#discussion_r821131260
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java
##
@@ -1539,7 +1539,11 @@ public ZN
siddharthteotia commented on pull request #8295:
URL: https://github.com/apache/pinot/pull/8295#issuecomment-1061176898
Looking at the build failures
--
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
siddharthteotia commented on pull request #8295:
URL: https://github.com/apache/pinot/pull/8295#issuecomment-1061177302
Related PR by @jackjlli to exclude the vulnerability related version being
transitively pulled in https://github.com/apache/pinot/pull/8307
--
This is an automated mes
siddharthteotia commented on pull request #8307:
URL: https://github.com/apache/pinot/pull/8307#issuecomment-1061174821
The lint tests were failing due to dependency issues after merging PR
https://github.com/apache/pinot/pull/8300
cc @jasperjiaguo
I think we can just upgrade to a si
This is an automated email from the ASF dual-hosted git repository.
richardstartin 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 d3fe951 Make realtime indexes pluggable
richardstartin merged pull request #8279:
URL: https://github.com/apache/pinot/pull/8279
--
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-u
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 308b371 In TableCache, invoke callback on config
richardstartin commented on issue #8153:
URL: https://github.com/apache/pinot/issues/8153#issuecomment-1061138781
🤞🏻
--
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 un
Jackie-Jiang closed issue #8153:
URL: https://github.com/apache/pinot/issues/8153
--
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 merged pull request #8302:
URL: https://github.com/apache/pinot/pull/8302
--
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 #8307:
URL: https://github.com/apache/pinot/pull/8307
## Description
This PR excludes protobuf-java explicitly in transitive dependencies in the
repo, as some lower version of this jar which is marked as vulnerable gets
pulled transparently.
E.g
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch exclude-protobuf-java
in repository https://gitbox.apache.org/repos/asf/pinot.git
commit 6a4fc1f1c071341841018fb92355d74000a362f3
Author: Jack Li(Analytics Engineering)
AuthorDate: Mon Mar 7 11:58:
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch exclude-protobuf-java
in repository https://gitbox.apache.org/repos/asf/pinot.git.
at 6a4fc1f Exclude protobuf-java explicitly in transitive dependencies
This branch includes the following
codecov-commenter edited a comment on pull request #8302:
URL: https://github.com/apache/pinot/pull/8302#issuecomment-1059628851
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8302?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
codecov-commenter edited a comment on pull request #8302:
URL: https://github.com/apache/pinot/pull/8302#issuecomment-1059628851
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8302?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
amrishlal commented on pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#issuecomment-1061051085
> Do you have an example about how to add the integration test I can add? I
do not see any query-related test against broker and server
Please see `OfflineClusterIntegrat
amrishlal commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r821031097
##
File path:
pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java
##
@@ -369,6 +369,9 @@ private static PinotQuery
compileCalc
This is an automated email from the ASF dual-hosted git repository.
siddteotia pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git.
from e41d86b Add Flink Pinot connector (#8233)
add 425055d Add support for native transports #4427 (#8300)
No n
siddharthteotia merged pull request #8300:
URL: https://github.com/apache/pinot/pull/8300
--
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-
jackjlli opened a new issue #8306:
URL: https://github.com/apache/pinot/issues/8306
We’re planning to upgrade the Apache Helix version from 0.9.8 to 1.0+ in
Pinot repo. This will not only address the issues we’ve seen in the current
0.9.8 release (which have been addressed in Helix 1.0+ an
yupeng9 merged pull request #8233:
URL: https://github.com/apache/pinot/pull/8233
--
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
0902horn commented on a change in pull request #8274:
URL: https://github.com/apache/pinot/pull/8274#discussion_r820520408
##
File path:
pinot-plugins/pinot-stream-ingestion/pinot-rocketmq/src/main/java/org/apache/pinot/plugin/stream/rocketmq/RocketMQConsumerFactory.java
##
@@
richardstartin commented on a change in pull request #8304:
URL: https://github.com/apache/pinot/pull/8304#discussion_r820500243
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/TrigonometricFunctions.java
##
@@ -0,0 +1,74 @@
+package org.apache
richardstartin commented on a change in pull request #8304:
URL: https://github.com/apache/pinot/pull/8304#discussion_r820497946
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/ArithmeticFunctions.java
##
@@ -104,4 +111,30 @@ public static doub
richardstartin commented on a change in pull request #8304:
URL: https://github.com/apache/pinot/pull/8304#discussion_r820497440
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/ArithmeticFunctions.java
##
@@ -104,4 +111,30 @@ public static doub
codecov-commenter edited a comment on pull request #8300:
URL: https://github.com/apache/pinot/pull/8300#issuecomment-1059516766
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8300?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
70 matches
Mail list logo
