codecov-commenter edited a comment on pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#issuecomment-1013577523
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8029?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 #8029:
URL: https://github.com/apache/pinot/pull/8029#issuecomment-1013577523
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8029?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 #8029:
URL: https://github.com/apache/pinot/pull/8029#issuecomment-1013577523
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8029?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=
weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r819278835
##
File path: pinot-core/src/main/java/org/apache/pinot/core/util/GapfillUtils.java
##
@@ -119,4 +130,138 @@ static public Serializable
getDefaultValue
weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r819267921
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/request/context/utils/BrokerRequestToQueryContextConverter.java
##
@@ -53,12 +55,81
walterddr opened a new pull request #8293:
URL: https://github.com/apache/pinot/pull/8293
## Description
create HttpUtils out of ControllerTestUtils that are generically useful.
## Release Notes
TBA
## Documentation
TBA
--
This is an automated message from the A
weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r819266353
##
File path:
pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java
##
@@ -369,6 +369,9 @@ private static PinotQuery
compileCa
npawar closed issue #7988:
URL: https://github.com/apache/pinot/issues/7988
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pi
npawar closed issue #4589:
URL: https://github.com/apache/pinot/issues/4589
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pi
weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r819261079
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/request/context/utils/BrokerRequestToQueryContextConverter.java
##
@@ -53,12 +55,81
npawar commented on a change in pull request #8233:
URL: https://github.com/apache/pinot/pull/8233#discussion_r819260880
##
File path:
pinot-connectors/pinot-flink-connector/src/main/java/org/apache/pinot/connector/flink/sink/FlinkSegmentWriter.java
##
@@ -0,0 +1,266 @@
+/**
+
weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r819260278
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/GapfillQueriesTest.java
##
@@ -0,0 +1,3615 @@
+/**
+ * Licensed to the Apache Software
weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r819259743
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/request/context/QueryContext.java
##
@@ -133,6 +137,8 @@ private QueryContext(String
weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r819258819
##
File path: pinot-core/src/main/java/org/apache/pinot/core/util/GapfillUtils.java
##
@@ -31,7 +36,10 @@
*/
public class GapfillUtils {
private s
weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r817134430
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/request/context/utils/BrokerRequestToQueryContextConverter.java
##
@@ -53,12 +55,82
yupeng9 commented on a change in pull request #8233:
URL: https://github.com/apache/pinot/pull/8233#discussion_r819252986
##
File path:
pinot-connectors/pinot-flink-connector/src/main/java/org/apache/pinot/connector/flink/sink/FlinkSegmentWriter.java
##
@@ -0,0 +1,266 @@
+/**
yupeng9 commented on a change in pull request #8233:
URL: https://github.com/apache/pinot/pull/8233#discussion_r819142901
##
File path:
pinot-connectors/pinot-flink-connector/src/main/java/org/apache/pinot/connector/flink/FlinkQuickStart.java
##
@@ -0,0 +1,88 @@
+/**
+ * Licen
codecov-commenter edited a comment on pull request #8292:
URL: https://github.com/apache/pinot/pull/8292#issuecomment-1058749310
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8292?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 #8292:
URL: https://github.com/apache/pinot/pull/8292#issuecomment-1058749310
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8292?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 #8292:
URL: https://github.com/apache/pinot/pull/8292#issuecomment-1058749310
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8292?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 #8292:
URL: https://github.com/apache/pinot/pull/8292#issuecomment-1058749310
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8292?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
This is an automated email from the ASF dual-hosted git repository.
sajjad pushed a change to branch release-0.10.0-rc
in repository https://gitbox.apache.org/repos/asf/pinot.git.
at fd9c58a Adding NoopPinotMetricFactory and corresponding changes
(#8270)
No new revisions were added by t
walterddr commented on issue #8246:
URL: https://github.com/apache/pinot/issues/8246#issuecomment-1058636542
+1 for this proposal. I am also working on factoring out the helper
functions from ControllerTestUtils.
Here is what I proposed to do, please kindly let me know if this is th
Jackie-Jiang opened a new pull request #8291:
URL: https://github.com/apache/pinot/pull/8291
In no-downtime mode, when the ideal state is changed (potentially by another
rebalance request), re-calculate the instance partitions before re-calculating
the target assignment.
Currently we do
icefury71 commented on a change in pull request #8273:
URL: https://github.com/apache/pinot/pull/8273#discussion_r819083193
##
File path:
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/ComplexTypeTransformer.java
##
@@ -279,6 +294,23 @@ prot
amrishlal commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r819020015
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/request/context/QueryContext.java
##
@@ -133,6 +137,8 @@ private QueryContext(String t
ksnijjer opened a new issue #8290:
URL: https://github.com/apache/pinot/issues/8290
Pinot currently supports aggregation functions for e.g max() on numeric
values only, however if you run a query where a non-numeric column is passed as
a value then error message generated is really vague a
ksnijjer opened a new issue #8289:
URL: https://github.com/apache/pinot/issues/8289
Currently if any query fired from the Query console times out then below
message is shown in the UI:
https://user-images.githubusercontent.com/85707391/156644706-8d3d16cc-fbd4-453f-b866-2fd1f8a07c56.p
npawar commented on a change in pull request #8233:
URL: https://github.com/apache/pinot/pull/8233#discussion_r819011364
##
File path:
pinot-connectors/pinot-flink-connector/src/main/java/org/apache/pinot/connector/flink/sink/FlinkSegmentWriter.java
##
@@ -0,0 +1,266 @@
+/**
+
yupeng9 commented on a change in pull request #8233:
URL: https://github.com/apache/pinot/pull/8233#discussion_r818999292
##
File path:
pinot-connectors/pinot-flink-connector/src/main/java/org/apache/pinot/connector/flink/sink/FlinkSegmentWriter.java
##
@@ -0,0 +1,266 @@
+/**
xiangfu0 closed issue #8267:
URL: https://github.com/apache/pinot/issues/8267
--
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 235e21b Fix bug in GrpcBrokerRequestHandler: exceptions are
suppressed (#8272)
add 46ed731 Fail process when
xiangfu0 merged pull request #8282:
URL: https://github.com/apache/pinot/pull/8282
--
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 change to branch release-0.10.0
in repository https://gitbox.apache.org/repos/asf/pinot.git.
at fd9c58a Adding NoopPinotMetricFactory and corresponding changes
(#8270)
No new revisions were added by this
walterddr commented on a change in pull request #7358:
URL: https://github.com/apache/pinot/pull/7358#discussion_r818957908
##
File path:
pinot-plugins/pinot-input-format/pinot-avro-base/src/main/java/org/apache/pinot/plugin/inputformat/avro/AvroLogicalTypeConversionRegistry.java
siddharthteotia commented on issue #8164:
URL: https://github.com/apache/pinot/issues/8164#issuecomment-1058362750
@jasperjiaguo and I reviewed the proposal with @Jackie-Jiang
Overall we are aligned with the proposed algorithm and there were some
questions on implementation details.
richardstartin commented on a change in pull request #7358:
URL: https://github.com/apache/pinot/pull/7358#discussion_r818946683
##
File path:
pinot-plugins/pinot-input-format/pinot-avro-base/src/main/java/org/apache/pinot/plugin/inputformat/avro/AvroLogicalTypeConversionRegistry.java
richardstartin merged pull request #8272:
URL: https://github.com/apache/pinot/pull/8272
--
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.
richardstartin pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git.
from eaf656b add more testing for MV var-length columns, detect overflow
scenarios (#8286)
add 235e21b Fix
This is an automated email from the ASF dual-hosted git repository.
richardstartin pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git.
from 2edcb6b Upgrade protobuf as the current version has security
vulnerability (#8287)
add eaf656b add mor
richardstartin merged pull request #8286:
URL: https://github.com/apache/pinot/pull/8286
--
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
dongxiaoman commented on a change in pull request #8272:
URL: https://github.com/apache/pinot/pull/8272#discussion_r818940174
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/query/reduce/StreamingReduceServiceTest.java
##
@@ -0,0 +1,114 @@
+/**
+ * Licensed to the
dongxiaoman commented on a change in pull request #8272:
URL: https://github.com/apache/pinot/pull/8272#discussion_r818938704
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/query/reduce/StreamingReduceServiceTest.java
##
@@ -0,0 +1,114 @@
+/**
+ * Licensed to the
dongxiaoman commented on a change in pull request #8272:
URL: https://github.com/apache/pinot/pull/8272#discussion_r818938704
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/query/reduce/StreamingReduceServiceTest.java
##
@@ -0,0 +1,114 @@
+/**
+ * Licensed to the
dongxiaoman commented on a change in pull request #8272:
URL: https://github.com/apache/pinot/pull/8272#discussion_r818933915
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/query/reduce/StreamingReduceServiceTest.java
##
@@ -0,0 +1,114 @@
+/**
+ * Licensed to the
dongxiaoman commented on a change in pull request #8272:
URL: https://github.com/apache/pinot/pull/8272#discussion_r818933764
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/query/reduce/StreamingReduceServiceTest.java
##
@@ -0,0 +1,114 @@
+/**
+ * Licensed to the
richardstartin commented on a change in pull request #8279:
URL: https://github.com/apache/pinot/pull/8279#discussion_r818921189
##
File path:
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/mutable/provider/MutableForwardIndexProvider.java
##
@@ -0,0 +1,26
Jackie-Jiang commented on a change in pull request #8279:
URL: https://github.com/apache/pinot/pull/8279#discussion_r818913738
##
File path:
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/mutable/provider/MutableForwardIndexProvider.java
##
@@ -0,0 +1,26 @@
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 e498a71 move mutable indexes into dedicated package (#8278)
add 2edcb6b Upgrade protobuf as the current ver
siddharthteotia merged pull request #8287:
URL: https://github.com/apache/pinot/pull/8287
--
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-
aashitk opened a new issue #8288:
URL: https://github.com/apache/pinot/issues/8288
After consuming `access_token`, it should redirect to the desired route
For instance, `/#/access_token=&query` should redirect
to `/#/query` instead of `/#/`
--
This is an automated message from the
codecov-commenter commented on pull request #8287:
URL: https://github.com/apache/pinot/pull/8287#issuecomment-1058218276
#
[Codecov](https://codecov.io/gh/apache/pinot/pull/8287?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apa
siddharthteotia opened a new pull request #8287:
URL: https://github.com/apache/pinot/pull/8287
Our internal build system failed after flagging a security vulenrability in
protobuf version
com.google.protobuf:protobuf-java:3.17.1
Notes: Vulnerability found and is blocked -- vulner
GXM2333 closed issue #8258:
URL: https://github.com/apache/pinot/issues/8258
--
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...@p
richardstartin opened a new pull request #8286:
URL: https://github.com/apache/pinot/pull/8286
This addresses a bug report of a buffer overflow creating MV string raw
forward index, with the following stack trace:
```
Start building IndexCreator!
Could not build segment
java
aashitk opened a new pull request #8285:
URL: https://github.com/apache/pinot/pull/8285
## 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**, an
56 matches
Mail list logo
