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 3e794ca Fixing segment push uri
fx19880617 merged pull request #5897:
URL: https://github.com/apache/incubator-pinot/pull/5897
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
jasonyanwenl commented on a change in pull request #5866:
URL: https://github.com/apache/incubator-pinot/pull/5866#discussion_r473085145
##
File path:
thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/api/detection/AnomalyDetectionResource.java
##
@@ -345,18
cyrilou242 edited a comment on pull request #5868:
URL: https://github.com/apache/incubator-pinot/pull/5868#issuecomment-676488023
Hello, I've added a change to make integration with google apis easier.
A recent commit
cyrilou242 commented on pull request #5868:
URL: https://github.com/apache/incubator-pinot/pull/5868#issuecomment-676488023
Hello, I've added a change to make integration with google apis easier.
A recent commit
jasonyanwenl commented on a change in pull request #5866:
URL: https://github.com/apache/incubator-pinot/pull/5866#discussion_r473093021
##
File path:
thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/api/detection/AnomalyDetectionResource.java
##
@@ -464,6
jasonyanwenl commented on a change in pull request #5866:
URL: https://github.com/apache/incubator-pinot/pull/5866#discussion_r473202402
##
File path:
thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/api/detection/AnomalyDetectionResource.java
##
@@ -511,24
kishoreg commented on issue #4484:
URL:
https://github.com/apache/incubator-pinot/issues/4484#issuecomment-676603896
There was another issue recently where a host was unreachable and the entire
query would fail.
High-level design sounds good. With Jackie changes to computing
lgo opened a new issue #5899:
URL: https://github.com/apache/incubator-pinot/issues/5899
Just a usability thing: when writing out some configuration, there was a
typo in a schema's field name so the star tree configuration did not have a
list of all valid columns. It took a bit of poking
icefury71 commented on issue #4484:
URL:
https://github.com/apache/incubator-pinot/issues/4484#issuecomment-676597798
Thanks for the comments Subbu and Ting. Based on what @mcvsubbu mentioned,
I'm thinking of adding a failure detector capability in the Pinot broker so
that it can
KKcorps commented on a change in pull request #5892:
URL: https://github.com/apache/incubator-pinot/pull/5892#discussion_r472427252
##
File path:
pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/PreparedStatement.java
##
@@ -72,6 +72,15 @@ public
KKcorps commented on a change in pull request #5892:
URL: https://github.com/apache/incubator-pinot/pull/5892#discussion_r473161253
##
File path:
pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/Connection.java
##
@@ -96,6 +96,18 @@ public ResultSetGroup
This is an automated email from the ASF dual-hosted git repository.
tingchen pushed a commit to branch release-0.5.0-rc
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/release-0.5.0-rc by this push:
new 94b36b6 Fix
KKcorps commented on a change in pull request #5892:
URL: https://github.com/apache/incubator-pinot/pull/5892#discussion_r473161253
##
File path:
pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/Connection.java
##
@@ -96,6 +96,18 @@ public ResultSetGroup
fx19880617 commented on issue #5877:
URL:
https://github.com/apache/incubator-pinot/issues/5877#issuecomment-676569119
I've updated the ingestion uri push job in:
https://github.com/apache/incubator-pinot/pull/5897
So users don't need to set `segmentUriPrefix` for push jobs
lgo opened a new issue #5900:
URL: https://github.com/apache/incubator-pinot/issues/5900
(also non-critical, mostly just usability for being able to flexibility
configure Pinot)
While experimenting with some schemas, I tried setting up a star-index to
use an ingest transform
icefury71 edited a comment on issue #4484:
URL:
https://github.com/apache/incubator-pinot/issues/4484#issuecomment-676597798
Thanks for the comments Subbu and Ting. Based on what @mcvsubbu mentioned,
I'm thinking of adding a failure detector capability in the Pinot broker so
that it can
npawar commented on issue #5900:
URL:
https://github.com/apache/incubator-pinot/issues/5900#issuecomment-676692376
Column name in ingestionCOnfigs should match the column name in the schema.
Schema will contain the final names of the columns, post the transformation.
In your example
jasonyanwenl commented on a change in pull request #5866:
URL: https://github.com/apache/incubator-pinot/pull/5866#discussion_r473375909
##
File path: thirdeye/thirdeye-pinot/src/main/resources/schema/create-schema.sql
##
@@ -437,3 +437,15 @@ create index
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a commit to branch anomalies-feedback-findbyid
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 7fab639475863086fc9fb8b44249656c94701996
Author: Jihao Zhang
AuthorDate: Wed Aug 19
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a change to branch anomalies-feedback-findbyid
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 7fab639 [TE] anomalies page - fix feedback is not populated if
retrived by the method
jackjlli commented on a change in pull request #5875:
URL: https://github.com/apache/incubator-pinot/pull/5875#discussion_r473276920
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/recommender/rules/utils/QueryInvertedSortedIndexRecommender.java
##
@@
buchireddy commented on a change in pull request #5892:
URL: https://github.com/apache/incubator-pinot/pull/5892#discussion_r473392543
##
File path:
pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/BrokerResponse.java
##
@@ -24,11 +24,12 @@
/**
*
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch exact-distinct-count
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 8f2dff4 addressing review comments
discard e5fd3a0 Removing code from another PR
discard
npawar opened a new issue #5901:
URL: https://github.com/apache/incubator-pinot/issues/5901
It seems using the instance APIs is getting users into some erroneous state.
Check all the APIs
This is an automated message from
npawar opened a new pull request #5902:
URL: https://github.com/apache/incubator-pinot/pull/5902
## Description
New API for updating instance tags.
```
curl -i -X PUT
"localhost:9000/instances/Server_10.20.30.40_8000/updateTags?tags=server_OFFLINE,server_REALTIME"
HTTP/1.1 200
Jackie-Jiang commented on a change in pull request #5872:
URL: https://github.com/apache/incubator-pinot/pull/5872#discussion_r473453456
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/common/ObjectSerDeUtils.java
##
@@ -452,6 +476,139 @@ public IntSet
buchireddy commented on a change in pull request #5892:
URL: https://github.com/apache/incubator-pinot/pull/5892#discussion_r473389458
##
File path:
pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/PreparedStatement.java
##
@@ -72,6 +72,15 @@ public
Jackie-Jiang merged pull request #5898:
URL: https://github.com/apache/incubator-pinot/pull/5898
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
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/incubator-pinot.git.
from 3e794ca Fixing segment push uri ingestion jobs (#5897)
add 4fd70fe Add RANGE FilterKind to support
jasperjiaguo commented on a change in pull request #5875:
URL: https://github.com/apache/incubator-pinot/pull/5875#discussion_r473318904
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/recommender/rules/io/params/RecommenderConstants.java
##
@@ -32,7
fx19880617 merged pull request #5864:
URL: https://github.com/apache/incubator-pinot/pull/5864
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
Jackie-Jiang commented on a change in pull request #5902:
URL: https://github.com/apache/incubator-pinot/pull/5902#discussion_r473459274
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotInstanceRestletResource.java
##
@@ -196,4
mcvsubbu commented on issue #4484:
URL:
https://github.com/apache/incubator-pinot/issues/4484#issuecomment-676836660
Helix provides us a view of "good " vs "bad" instances. IMO if we are adding
algorithms on the broker, it should be based on response time of servers. If a
server is
codecov-commenter commented on pull request #5875:
URL: https://github.com/apache/incubator-pinot/pull/5875#issuecomment-676757756
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5875?src=pr=h1)
Report
> Merging
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 c5a8b02 Fixing the race condition
mayankshriv commented on pull request #5872:
URL: https://github.com/apache/incubator-pinot/pull/5872#issuecomment-676999527
> @mayankshriv Currently the `DistinctCount` is not having the expected
behavior of returning the exact distinct count because it is storing the
`hashCode()` of the
jackjlli commented on pull request #5873:
URL: https://github.com/apache/incubator-pinot/pull/5873#issuecomment-677030837
> Most of these are already tracked in transformers. Invalid columns etc
Correct, but it's encapsulated in RecordRecorder and it will dirty the
`RecordRecorder`
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch add-counter-for-detecting-schema-mismatch
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard d638096 Add SchemaValidator
add 1cf73d1 Add SchemaValidator
This
fx19880617 commented on a change in pull request #5872:
URL: https://github.com/apache/incubator-pinot/pull/5872#discussion_r473493705
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/common/ObjectSerDeUtils.java
##
@@ -111,6 +131,19 @@ public static ObjectType
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch add-counter-for-detecting-schema-mismatch
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 8b0b778 Add Hadoop counter for detecting schema mismatch
add 2cfaed3
jackjlli commented on a change in pull request #5873:
URL: https://github.com/apache/incubator-pinot/pull/5873#discussion_r473581080
##
File path:
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch add-counter-for-detecting-schema-mismatch
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 1cf73d1 Add SchemaValidator
add 0fb58c0 Add SchemaValidator
This
Jackie-Jiang commented on a change in pull request #5872:
URL: https://github.com/apache/incubator-pinot/pull/5872#discussion_r473548121
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/common/ObjectSerDeUtils.java
##
@@ -111,6 +131,19 @@ public static ObjectType
44 matches
Mail list logo