Jackie-Jiang commented on a change in pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#discussion_r627883138
##
File path:
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/creator/impl/inv/json/BaseJsonIndexCreator.java
##
@@
Jackie-Jiang commented on a change in pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#discussion_r627883425
##
File path:
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/readers/json/ImmutableJsonIndexReader.java
##
@
Jackie-Jiang commented on a change in pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#discussion_r627883534
##
File path:
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/readers/json/ImmutableJsonIndexReader.java
##
@
icefury71 opened a new pull request #6890:
URL: https://github.com/apache/incubator-pinot/pull/6890
… table
## Description
This is part 1 of #6524 . This PR provides an overall ingestion status for
Pinot realtime table. The status is HEALTHY if all consuming segments are in
the C
Jackie-Jiang commented on a change in pull request #6885:
URL: https://github.com/apache/incubator-pinot/pull/6885#discussion_r627915336
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/perf/QueryRunner.java
##
@@ -686,16 +686,16 @@ public static void
increasing
Jackie-Jiang commented on a change in pull request #6885:
URL: https://github.com/apache/incubator-pinot/pull/6885#discussion_r627915394
##
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/perf/QueryRunner.java
##
@@ -542,16 +544,16 @@ public static void
targetQPSQ
yupeng9 commented on a change in pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#discussion_r627921488
##
File path:
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/creator/impl/inv/json/BaseJsonIndexCreator.java
##
@@ -130
Jackie-Jiang removed a comment on pull request #6885:
URL: https://github.com/apache/incubator-pinot/pull/6885#issuecomment-834061611
For clients that call the QueryRunner methods directly (not through command
line), added return objects for the methods so that that the client does not
hav
Jackie-Jiang commented on pull request #6885:
URL: https://github.com/apache/incubator-pinot/pull/6885#issuecomment-834061611
For clients that call the QueryRunner methods directly (not through command
line), added return objects for the methods so that that the client does not
have to col
Jackie-Jiang merged pull request #6885:
URL: https://github.com/apache/incubator-pinot/pull/6885
--
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.
For queries about this se
codecov-commenter commented on pull request #6890:
URL: https://github.com/apache/incubator-pinot/pull/6890#issuecomment-834062294
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6890?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 #6890:
URL: https://github.com/apache/incubator-pinot/pull/6890#issuecomment-834062294
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6890?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
xiangfu0 merged pull request #6888:
URL: https://github.com/apache/incubator-pinot/pull/6888
--
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.
For queries about this servic
jtao15 opened a new pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891
This pr add supports to allow selecting same column multiple times with
different alias (#6844 ).
E.g. `Select colA AS xx, colA AS yy from T`.
--
This is an automated message from the Apa
codecov-commenter commented on pull request #6889:
URL: https://github.com/apache/incubator-pinot/pull/6889#issuecomment-834104502
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6889?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+commen
codecov-commenter commented on pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#issuecomment-834125903
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6891?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 #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#issuecomment-832453091
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6878?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter edited a comment on pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#issuecomment-832453091
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6878?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628351204
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonDatatypeTest.java
##
@@ -0,0 +1,293 @@
+/**
+ * Licensed to the Apache Soft
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628351400
##
File path: pinot-spi/src/main/java/org/apache/pinot/spi/data/FieldSpec.java
##
@@ -52,6 +52,7 @@
public static final Integer DEFAULT_DIMEN
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628354108
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/transport/QueryRoutingTest.java
##
@@ -211,7 +211,7 @@ public void testServerDown(
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628354233
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/HavingFilterHandler.java
##
@@ -154,6 +154,7 @@ public boolean isMatc
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628354318
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/CastTransformFunction.java
##
@@ -130,6 +132,9 @@ publ
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628354478
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/CastTransformFunction.java
##
@@ -76,6 +76,8 @@ public
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628358014
##
File path:
pinot-common/src/test/java/org/apache/pinot/common/utils/PinotDataTypeTest.java
##
@@ -102,43 +127,64 @@ public void testObject()
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628358171
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/utils/PinotDataType.java
##
@@ -538,6 +540,64 @@ public String convert(Object
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628358409
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/utils/PinotDataType.java
##
@@ -538,6 +540,64 @@ public String convert(Object
siddharthteotia commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628362469
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonDatatypeTest.java
##
@@ -0,0 +1,308 @@
+/**
+ * Licensed to the Apach
siddharthteotia commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628365082
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonDatatypeTest.java
##
@@ -0,0 +1,308 @@
+/**
+ * Licensed to the Apach
siddharthteotia commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628366484
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonDatatypeTest.java
##
@@ -0,0 +1,308 @@
+/**
+ * Licensed to the Apach
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628354108
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/transport/QueryRoutingTest.java
##
@@ -211,7 +211,7 @@ public void testServerDown(
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628354108
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/transport/QueryRoutingTest.java
##
@@ -211,7 +211,7 @@ public void testServerDown(
Jackie-Jiang commented on a change in pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#discussion_r628388394
##
File path:
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/creator/impl/inv/json/BaseJsonIndexCreator.java
##
@@
desaijay230592 commented on a change in pull request #6842:
URL: https://github.com/apache/incubator-pinot/pull/6842#discussion_r628396786
##
File path:
pinot-server/src/main/java/org/apache/pinot/server/starter/helix/HelixServerStarter.java
##
@@ -261,12 +270,39 @@ private vo
Jackie-Jiang commented on a change in pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#discussion_r628407710
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/BrokerReduceService.java
##
@@ -91,7 +91,8 @@ private static void
apucher opened a new pull request #6892:
URL: https://github.com/apache/incubator-pinot/pull/6892
## Description
Add a shell-script for AuthQuickstart
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order:
Controller, Broker, Server, Minion)
*
amrishlal opened a new pull request #6893:
URL: https://github.com/apache/incubator-pinot/pull/6893
## Description
To avoid flakyness, set timeout to 2000 msec. For some test runs, it can
take up to 1400 msec to mark request as failed.
## Upgrade Notes
Does this PR prevent a zero d
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628450306
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/transport/QueryRoutingTest.java
##
@@ -211,7 +211,7 @@ public void testServerDown(
amrishlal commented on pull request #6893:
URL: https://github.com/apache/incubator-pinot/pull/6893#issuecomment-834713572
@Jackie-Jiang @siddharthteotia Please review.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
desaijay230592 commented on a change in pull request #6842:
URL: https://github.com/apache/incubator-pinot/pull/6842#discussion_r628467875
##
File path:
pinot-spi/src/main/java/org/apache/pinot/spi/environmentprovider/PinotEnvironmentProvider.java
##
@@ -0,0 +1,43 @@
+/**
+ *
codecov-commenter commented on pull request #6892:
URL: https://github.com/apache/incubator-pinot/pull/6892#issuecomment-834731876
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6892?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+commen
codecov-commenter commented on pull request #6893:
URL: https://github.com/apache/incubator-pinot/pull/6893#issuecomment-834734222
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6893?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+commen
amrishlal commented on a change in pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#discussion_r628472654
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonMatchQueriesTest.java
##
@@ -0,0 +1,199 @@
+/**
+ * Licensed to the Apache
richardstartin commented on a change in pull request #6886:
URL: https://github.com/apache/incubator-pinot/pull/6886#discussion_r628473211
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/InstanceResponseOperator.java
##
@@ -45,8 +46,39 @@ protected Instan
richardstartin commented on a change in pull request #6886:
URL: https://github.com/apache/incubator-pinot/pull/6886#discussion_r628477390
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/InstanceResponseOperator.java
##
@@ -45,8 +46,39 @@ protected Instan
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628488178
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonDatatypeTest.java
##
@@ -0,0 +1,308 @@
+/**
+ * Licensed to the Apache Soft
desaijay230592 commented on a change in pull request #6842:
URL: https://github.com/apache/incubator-pinot/pull/6842#discussion_r628488447
##
File path:
pinot-server/src/main/java/org/apache/pinot/server/starter/helix/HelixServerStarter.java
##
@@ -261,12 +270,39 @@ private vo
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628488701
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonDatatypeTest.java
##
@@ -0,0 +1,308 @@
+/**
+ * Licensed to the Apache Soft
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628488178
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonDatatypeTest.java
##
@@ -0,0 +1,308 @@
+/**
+ * Licensed to the Apache Soft
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628493077
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonDatatypeTest.java
##
@@ -0,0 +1,308 @@
+/**
+ * Licensed to the Apache Soft
mcvsubbu commented on a change in pull request #6887:
URL: https://github.com/apache/incubator-pinot/pull/6887#discussion_r628489699
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/metrics/AbstractMetrics.java
##
@@ -462,16 +463,48 @@ public void addCallbackGa
awishnick opened a new issue #6894:
URL: https://github.com/apache/incubator-pinot/issues/6894
I tried to run a SegmentCreation job to ingest some Parquet files written by
Trino. I got some confusing error messages that made it look like the file was
corrupted. It turns out that this is be
mcvsubbu commented on a change in pull request #6890:
URL: https://github.com/apache/incubator-pinot/pull/6890#discussion_r628548807
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java
##
@@ -2117,6 +2117,17 @@ pub
jtao15 commented on a change in pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#discussion_r628564212
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/BrokerReduceService.java
##
@@ -91,7 +91,8 @@ private static void update
jtao15 commented on a change in pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#discussion_r628564298
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/AliasQueriesTest.java
##
@@ -0,0 +1,249 @@
+/**
+ * Licensed to the Apache Softwar
Jackie-Jiang commented on a change in pull request #6893:
URL: https://github.com/apache/incubator-pinot/pull/6893#discussion_r628564151
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/transport/QueryRoutingTest.java
##
@@ -187,6 +187,9 @@ public void testNonMatch
Jackie-Jiang commented on a change in pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#discussion_r628572042
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonMatchQueriesTest.java
##
@@ -0,0 +1,199 @@
+/**
+ * Licensed to the Apac
xiangfu0 commented on issue #6894:
URL:
https://github.com/apache/incubator-pinot/issues/6894#issuecomment-834822896
Have you tried using
`org.apache.pinot.plugin.inputformat.parquet.ParquetNativeRecordReader` as
record reader? It's using native parquet reader. Meanwhile we can look at ho
Jackie-Jiang commented on a change in pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#discussion_r628572042
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonMatchQueriesTest.java
##
@@ -0,0 +1,199 @@
+/**
+ * Licensed to the Apac
Jackie-Jiang merged pull request #6892:
URL: https://github.com/apache/incubator-pinot/pull/6892
--
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.
For queries about this se
Jackie-Jiang commented on a change in pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#discussion_r628581113
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/BrokerReduceService.java
##
@@ -91,10 +91,7 @@ private static void
Jackie-Jiang commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628585502
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonDatatypeTest.java
##
@@ -0,0 +1,308 @@
+/**
+ * Licensed to the Apache S
amrishlal commented on a change in pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#discussion_r628589659
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonDatatypeTest.java
##
@@ -0,0 +1,308 @@
+/**
+ * Licensed to the Apache Soft
codecov-commenter commented on pull request #6842:
URL: https://github.com/apache/incubator-pinot/pull/6842#issuecomment-834835328
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6842?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 #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#issuecomment-834125903
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6891?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter edited a comment on pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#issuecomment-832402698
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6877?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
amrishlal commented on a change in pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#discussion_r628602361
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonMatchQueriesTest.java
##
@@ -0,0 +1,199 @@
+/**
+ * Licensed to the Apache
mcvsubbu commented on pull request #6893:
URL: https://github.com/apache/incubator-pinot/pull/6893#issuecomment-834852938
I am a little confused by this test. How can we guarantee that the
`queryServer.shutDown()` happens before the query actually makes it to the
server?
--
This is an
apucher merged pull request #6884:
URL: https://github.com/apache/incubator-pinot/pull/6884
--
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.
For queries about this service
amrishlal commented on a change in pull request #6893:
URL: https://github.com/apache/incubator-pinot/pull/6893#discussion_r628610393
##
File path:
pinot-core/src/test/java/org/apache/pinot/core/transport/QueryRoutingTest.java
##
@@ -187,6 +187,9 @@ public void testNonMatching
amrishlal commented on pull request #6893:
URL: https://github.com/apache/incubator-pinot/pull/6893#issuecomment-834862009
> I am a little confused by this test. How can we guarantee that the
`queryServer.shutDown()` happens before the query actually makes it to the
server?
`querySe
amrishlal edited a comment on pull request #6893:
URL: https://github.com/apache/incubator-pinot/pull/6893#issuecomment-834862009
> I am a little confused by this test. How can we guarantee that the
`queryServer.shutDown()` happens before the query actually makes it to the
server?
`
Jackie-Jiang commented on a change in pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#discussion_r628619787
##
File path:
pinot-core/src/test/java/org/apache/pinot/queries/JsonMatchQueriesTest.java
##
@@ -0,0 +1,199 @@
+/**
+ * Licensed to the Apac
Jackie-Jiang merged pull request #6893:
URL: https://github.com/apache/incubator-pinot/pull/6893
--
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.
For queries about this se
snleee commented on a change in pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#discussion_r628619949
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/selection/SelectionOperatorUtils.java
##
@@ -464,21 +464,40 @@ public static Se
snleee commented on a change in pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#discussion_r628619949
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/selection/SelectionOperatorUtils.java
##
@@ -464,21 +464,40 @@ public static Se
Jackie-Jiang merged pull request #6883:
URL: https://github.com/apache/incubator-pinot/pull/6883
--
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.
For queries about this se
codecov-commenter edited a comment on pull request #6877:
URL: https://github.com/apache/incubator-pinot/pull/6877#issuecomment-832402698
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6877?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
mcvsubbu commented on a change in pull request #6886:
URL: https://github.com/apache/incubator-pinot/pull/6886#discussion_r628636851
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/InstanceResponseOperator.java
##
@@ -45,8 +46,39 @@ protected InstanceResp
mcvsubbu commented on a change in pull request #6886:
URL: https://github.com/apache/incubator-pinot/pull/6886#discussion_r628637817
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/InstanceResponseOperator.java
##
@@ -45,8 +46,39 @@ protected InstanceResp
siddharthteotia commented on a change in pull request #6886:
URL: https://github.com/apache/incubator-pinot/pull/6886#discussion_r628638182
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/InstanceResponseOperator.java
##
@@ -45,8 +46,39 @@ protected Insta
xiangfu0 commented on pull request #6889:
URL: https://github.com/apache/incubator-pinot/pull/6889#issuecomment-834907244
> I don't fully understand the intention of this PR. Do we want to pick the
port from a random kafka server, or kafka might change the passed-in port if it
is unavailab
sajjad-moradi commented on a change in pull request #6869:
URL: https://github.com/apache/incubator-pinot/pull/6869#discussion_r628640366
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/recommender/rules/impl/PinotTablePartitionRule.java
##
@@ -119,12
xiangfu0 merged pull request #6889:
URL: https://github.com/apache/incubator-pinot/pull/6889
--
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.
For queries about this servic
sajjad-moradi commented on a change in pull request #6869:
URL: https://github.com/apache/incubator-pinot/pull/6869#discussion_r628640546
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/recommender/rules/io/params/SegmentSizeRuleParams.java
##
@@ -0,0
sajjad-moradi commented on a change in pull request #6869:
URL: https://github.com/apache/incubator-pinot/pull/6869#discussion_r628640710
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/recommender/rules/RulesToExecute.java
##
@@ -160,9 +169,14 @@ publ
sajjad-moradi commented on a change in pull request #6869:
URL: https://github.com/apache/incubator-pinot/pull/6869#discussion_r628640767
##
File path:
pinot-controller/src/test/java/org/apache/pinot/controller/recommender/TestConfigEngine.java
##
@@ -229,6 +230,10 @@ void tes
codecov-commenter edited a comment on pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#issuecomment-832453091
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6878?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
mcvsubbu commented on a change in pull request #6842:
URL: https://github.com/apache/incubator-pinot/pull/6842#discussion_r628638866
##
File path:
pinot-plugins/pinot-environment/pinot-azure/src/main/java/org/apache/pinot/plugin/provider/AzureEnvironmentProvider.java
##
@@ -0,
jtao15 commented on a change in pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#discussion_r628664205
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/BrokerReduceService.java
##
@@ -91,10 +91,7 @@ private static void updat
jtao15 commented on a change in pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#discussion_r628665006
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/selection/SelectionOperatorUtils.java
##
@@ -464,21 +464,40 @@ public static Se
codecov-commenter edited a comment on pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#issuecomment-832453091
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6878?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter edited a comment on pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#issuecomment-834125903
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6891?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter edited a comment on pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878#issuecomment-832453091
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6878?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
siddharthteotia merged pull request #6878:
URL: https://github.com/apache/incubator-pinot/pull/6878
--
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.
For queries about this
snleee commented on a change in pull request #6891:
URL: https://github.com/apache/incubator-pinot/pull/6891#discussion_r628707966
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/selection/SelectionOperatorUtils.java
##
@@ -464,21 +464,50 @@ public static Se
codecov-commenter edited a comment on pull request #6869:
URL: https://github.com/apache/incubator-pinot/pull/6869#issuecomment-830205799
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6869?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
codecov-commenter edited a comment on pull request #6869:
URL: https://github.com/apache/incubator-pinot/pull/6869#issuecomment-830205799
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6869?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr
troywinter opened a new pull request #6895:
URL: https://github.com/apache/incubator-pinot/pull/6895
Problem to solve:
Currently when updating a table config, api only return invalid table
config, not showing any useful error msg, it's hard for user to debug without
first request table
apucher opened a new issue #6896:
URL: https://github.com/apache/incubator-pinot/issues/6896
The controller UI crashes ungracefully to a white screen if all brokers are
offline and/or all servers are offline.
https://user-images.githubusercontent.com/25439965/117563140-bed25c80-b058-
1 - 100 of 55177 matches
Mail list logo