codecov-commenter edited a comment on pull request #7064:
URL: https://github.com/apache/incubator-pinot/pull/7064#issuecomment-861995023
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7064?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter edited a comment on pull request #7116:
URL: https://github.com/apache/incubator-pinot/pull/7116#issuecomment-872543381
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7116?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter edited a comment on pull request #7064:
URL: https://github.com/apache/incubator-pinot/pull/7064#issuecomment-861995023
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7064?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
KKcorps commented on a change in pull request #7117:
URL: https://github.com/apache/incubator-pinot/pull/7117#discussion_r662759401
##
File path:
pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotPreparedStatement.java
##
@@ -33,20 +33,25 @@
public cl
This is an automated email from the ASF dual-hosted git repository.
kharekartik pushed a change to branch jdbc_tableau
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 5eeef7f fix bug: handle LIMIT using regex
add 82cba7e fix bug: handle LIMIT using regex
N
KKcorps commented on a change in pull request #7117:
URL: https://github.com/apache/incubator-pinot/pull/7117#discussion_r662759049
##
File path:
pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotPreparedStatement.java
##
@@ -33,20 +33,25 @@
public cl
This is an automated email from the ASF dual-hosted git repository.
kharekartik pushed a change to branch jdbc_tableau
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from ec228fa Replace java11 stripTrailing with java8 trim for backward
compat
add 5eeef7f fix
KKcorps commented on a change in pull request #7117:
URL: https://github.com/apache/incubator-pinot/pull/7117#discussion_r662749542
##
File path:
pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotPreparedStatement.java
##
@@ -33,20 +33,25 @@
public cl
This is an automated email from the ASF dual-hosted git repository.
kharekartik pushed a change to branch jdbc_tableau
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from ee2848e Make JDBC driver compatible for Timestamp datatype - Add
Timestamp datatype support in
codecov-commenter edited a comment on pull request #7070:
URL: https://github.com/apache/incubator-pinot/pull/7070#issuecomment-862946427
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7070?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter edited a comment on pull request #7070:
URL: https://github.com/apache/incubator-pinot/pull/7070#issuecomment-862946427
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7070?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
This is an automated email from the ASF dual-hosted git repository.
kharekartik pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new d4d0b93 Cancel running Kinesis co
KKcorps merged pull request #7109:
URL: https://github.com/apache/incubator-pinot/pull/7109
--
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: commit
This is an automated email from the ASF dual-hosted git repository.
kharekartik pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 70308f8 Fixing pinot k8s helm for jdk11 (#7118)
add 84c6a6e Add integration test for Kinesis rea
KKcorps merged pull request #7016:
URL: https://github.com/apache/incubator-pinot/pull/7016
--
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: commit
KKcorps commented on a change in pull request #7116:
URL: https://github.com/apache/incubator-pinot/pull/7116#discussion_r662740892
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/DateTruncTransformFunction.java
##
@@ -75,6 +76,7 @@
*
xiangfu0 merged pull request #7118:
URL: https://github.com/apache/incubator-pinot/pull/7118
--
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: commi
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 70308f8 Fixing pinot k8s helm for jdk
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 423ac35 Validate the length of preproces
jackjlli merged pull request #7119:
URL: https://github.com/apache/incubator-pinot/pull/7119
--
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: commi
codecov-commenter edited a comment on pull request #7119:
URL: https://github.com/apache/incubator-pinot/pull/7119#issuecomment-872652683
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7119?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter commented on pull request #7119:
URL: https://github.com/apache/incubator-pinot/pull/7119#issuecomment-872652683
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7119?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+commen
codecov-commenter edited a comment on pull request #7064:
URL: https://github.com/apache/incubator-pinot/pull/7064#issuecomment-861995023
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7064?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
Jackie-Jiang commented on a change in pull request #7116:
URL: https://github.com/apache/incubator-pinot/pull/7116#discussion_r662670808
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/DateTimeFunctions.java
##
@@ -593,6 +593,18 @@ public stati
Jackie-Jiang commented on a change in pull request #7117:
URL: https://github.com/apache/incubator-pinot/pull/7117#discussion_r662666988
##
File path:
pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotPreparedStatement.java
##
@@ -33,20 +33,25 @@
publ
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch fix-preprocessing-operations
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard a4e804e Validate the length of preprocessing operation from custom
config
add 2e8
Jackie-Jiang commented on pull request #7076:
URL: https://github.com/apache/incubator-pinot/pull/7076#issuecomment-872631434
@tdunning Thanks for the explanation.
I understand the error is unavoidable, but IMO the test is legit and just
mimics the behavior of utilizing t-digest to pre-g
codecov-commenter edited a comment on pull request #7118:
URL: https://github.com/apache/incubator-pinot/pull/7118#issuecomment-872628903
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7118?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter commented on pull request #7118:
URL: https://github.com/apache/incubator-pinot/pull/7118#issuecomment-872628903
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7118?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+commen
jackjlli opened a new pull request #7119:
URL: https://github.com/apache/incubator-pinot/pull/7119
## Description
This PR validates the length of preprocessing operations from custom config.
By default the value would be `""` if it's missing. Thus, check the length
of the string befor
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch fix-preprocessing-operations
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit a4e804e37b23b569d30981f59902e4bf3c73dab7
Author: Jack Li(Analytics Engineering)
AuthorDate:
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch fix-preprocessing-operations
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at a4e804e Validate the length of preprocessing operation from custom
config
This branch
mayankshriv edited a comment on issue #7084:
URL:
https://github.com/apache/incubator-pinot/issues/7084#issuecomment-872613610
Thanks for adding this feature. We also wanted to add rest-api's to make
these status queriable in a debug env. Hopefully we can reuse the code from
#7099 when ad
mayankshriv commented on issue #7084:
URL:
https://github.com/apache/incubator-pinot/issues/7084#issuecomment-872613610
Thanks for adding this feature. We also wanted to add rest-api's to make
these status queriable in a debug env. Hopefully I can resue the code from
#7099.
--
This is
xiangfu0 opened a new pull request #7118:
URL: https://github.com/apache/incubator-pinot/pull/7118
## Description
Move k8s helm example to use pinot jdk11 image and fix JVM options issues.
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order:
Co
codecov-commenter edited a comment on pull request #7016:
URL: https://github.com/apache/incubator-pinot/pull/7016#issuecomment-854638247
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7016?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter edited a comment on pull request #7016:
URL: https://github.com/apache/incubator-pinot/pull/7016#issuecomment-854638247
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7016?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
KKcorps commented on a change in pull request #7116:
URL: https://github.com/apache/incubator-pinot/pull/7116#discussion_r662596971
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/DateTruncTransformFunction.java
##
@@ -75,6 +76,7 @@
*
codecov-commenter edited a comment on pull request #7117:
URL: https://github.com/apache/incubator-pinot/pull/7117#issuecomment-872543128
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7117?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
npawar commented on a change in pull request #7109:
URL: https://github.com/apache/incubator-pinot/pull/7109#discussion_r662587462
##
File path:
pinot-plugins/pinot-stream-ingestion/pinot-kinesis/src/main/java/org/apache/pinot/plugin/stream/kinesis/KinesisConsumer.java
##
@@ -
codecov-commenter commented on pull request #7116:
URL: https://github.com/apache/incubator-pinot/pull/7116#issuecomment-872543381
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7116?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+commen
codecov-commenter commented on pull request #7117:
URL: https://github.com/apache/incubator-pinot/pull/7117#issuecomment-872543128
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7117?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+commen
KKcorps opened a new pull request #7117:
URL: https://github.com/apache/incubator-pinot/pull/7117
## Description
The PR addresses 2 issues with JDBC drivers -
- Adds support for Timestamp datatype so that function returning that
datatype can be queried by JDBC
- Append
xiangfu0 commented on a change in pull request #7116:
URL: https://github.com/apache/incubator-pinot/pull/7116#discussion_r662552153
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/DateTruncTransformFunction.java
##
@@ -75,6 +76,7 @@
This is an automated email from the ASF dual-hosted git repository.
kharekartik pushed a commit to branch jdbc_tableau
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit ee2848e8c265ac14f9b6797f3fcfa87ef87ef3c0
Author: KKcorps
AuthorDate: Fri Jul 2 00:46:46 2021 +0530
This is an automated email from the ASF dual-hosted git repository.
kharekartik pushed a change to branch jdbc_tableau
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at ee2848e Make JDBC driver compatible for Timestamp datatype - Add
Timestamp datatype support in
KKcorps opened a new pull request #7116:
URL: https://github.com/apache/incubator-pinot/pull/7116
## Description
Currently, the datetime functions requires a minimum of 3 arguments.
However, most databases offer the 2 argument implementation which is used by
services such as Tableau
tdunning commented on pull request #7076:
URL: https://github.com/apache/incubator-pinot/pull/7076#issuecomment-872488605
Yes. There is an update.
I think that there is a fundamental problem with the way that the test is
phrased.
Any digest that has limited memory will ultim
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/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 0255757 Fix build (#7115)
0255757 is d
Jackie-Jiang merged pull request #7115:
URL: https://github.com/apache/incubator-pinot/pull/7115
--
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: c
Jackie-Jiang opened a new pull request #7115:
URL: https://github.com/apache/incubator-pinot/pull/7115
Fix the conflict caused by merging multiple PRs
--
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
keweishang edited a comment on issue #6523:
URL:
https://github.com/apache/incubator-pinot/issues/6523#issuecomment-872455648
We at Bolt also need this feature. We use Kafka Streams to join different
Kafka topics. Kafka Streams' JOIN produces many Kafka records with the same
event-time co
keweishang commented on issue #6523:
URL:
https://github.com/apache/incubator-pinot/issues/6523#issuecomment-872455648
We at Bolt also need this feature. We use Kafka Streams to join different
Kafka topics. Kafka Streams' JOIN produces many Kafka records with the same
event-time column va
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/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 0fb9209 Implement Append merger for pa
Jackie-Jiang merged pull request #7087:
URL: https://github.com/apache/incubator-pinot/pull/7087
--
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: c
codecov-commenter edited a comment on pull request #7114:
URL: https://github.com/apache/incubator-pinot/pull/7114#issuecomment-872439509
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7114?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter commented on pull request #7114:
URL: https://github.com/apache/incubator-pinot/pull/7114#issuecomment-872439509
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/7114?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+commen
Jackie-Jiang commented on pull request #7076:
URL: https://github.com/apache/incubator-pinot/pull/7076#issuecomment-872433461
@tdunning 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
Jackie-Jiang merged pull request #7092:
URL: https://github.com/apache/incubator-pinot/pull/7092
--
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: c
Jackie-Jiang merged pull request #7104:
URL: https://github.com/apache/incubator-pinot/pull/7104
--
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: c
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/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new ea3566f Fix the memory issue for selec
Jackie-Jiang merged pull request #7112:
URL: https://github.com/apache/incubator-pinot/pull/7112
--
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: c
walterddr opened a new pull request #7114:
URL: https://github.com/apache/incubator-pinot/pull/7114
## Description
Support REGEXP_EXTRACT based on descriptions in #6599.
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order:
Controller, Broker, S
wuwenw opened a new pull request #7113:
URL: https://github.com/apache/incubator-pinot/pull/7113
## Description
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order:
Controller, Broker, Server, Minion)
* [ ] Yes (Please label as **backward-incom
MrNeocore commented on issue #7060:
URL:
https://github.com/apache/incubator-pinot/issues/7060#issuecomment-872309119
@mayankshriv I don't have that big table available right now, but I'll try
it out whenever possible :)
--
This is an automated message from the Apache Git Servi
65 matches
Mail list logo
