This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch
support_aggregation_function_name_with_underscore
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 6c5ca9d update codecov version to v1.0.5
add c1f5ec7 upd
vincentchenjl opened a new pull request #5799:
URL: https://github.com/apache/incubator-pinot/pull/5799
This PR is to fix the issue that RCA template is not loaded when there is
only anomalyId is specified in the RCA URL. The fix is to pull metric templates
after metric IDs are fetched bas
mayankshriv opened a new pull request #5798:
URL: https://github.com/apache/incubator-pinot/pull/5798
…sketches and unions.
* In a case with large number of predicates in the
post-aggregation-expression (with OR's), we tend
to end up with a lot of empty sketches (and unions) whe
This is an automated email from the ASF dual-hosted git repository.
kishoreg 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 d1b4586 New endpoint to get routing
kishoreg merged pull request #5791:
URL: https://github.com/apache/incubator-pinot/pull/5791
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
pradeepgv42 commented on issue #5616:
URL:
https://github.com/apache/incubator-pinot/issues/5616#issuecomment-668310406
this was fixed in https://github.com/apache/incubator-pinot/pull/5670
This is an automated message from
pradeepgv42 closed issue #5616:
URL: https://github.com/apache/incubator-pinot/issues/5616
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
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch
support_aggregation_function_name_with_underscore
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 51a27b7 Support aggregation function name with underscore insi
mayankshriv merged pull request #5789:
URL: https://github.com/apache/incubator-pinot/pull/5789
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
This is an automated email from the ASF dual-hosted git repository.
mayanks 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 81504cd Bug fix: PinotSegmentToAvroCo
pradeepgv42 opened a new issue #5797:
URL: https://github.com/apache/incubator-pinot/issues/5797
select *
from table
where timestamp > X and not regexp_like(column, '')
throws following exception
`pinot error: {'errorCode': 150, 'message': 'PQLParsingError:\n'
'java.lan
suvodeep-pyne commented on pull request #5792:
URL: https://github.com/apache/incubator-pinot/pull/5792#issuecomment-668283528
Thanks a lot @jihaozh! Yes! I'm planning to do that too!
This is an automated message from the Ap
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch
support_aggregation_function_name_with_underscore
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard c04e537 Support aggregation function name with underscore insi
jasperjiaguo opened a new pull request #5796:
URL: https://github.com/apache/incubator-pinot/pull/5796
## Description
Moved dependencies for config recommender to pinot core
This is an automated message from the Ap
fx19880617 commented on a change in pull request #5795:
URL: https://github.com/apache/incubator-pinot/pull/5795#discussion_r464699619
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/function/AggregationFunctionType.java
##
@@ -79,7 +79,7 @@ public boolean isO
Jackie-Jiang merged pull request #5786:
URL: https://github.com/apache/incubator-pinot/pull/5786
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
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 657e245 Add SegmentPartitionedDistinct
Jackie-Jiang commented on a change in pull request #5795:
URL: https://github.com/apache/incubator-pinot/pull/5795#discussion_r464692098
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/function/AggregationFunctionType.java
##
@@ -79,7 +79,7 @@ public boolean i
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 47623c0 [TE] Error handling refactor of EntityManagerResource (#5788)
add c2c623a Refactor TableCache
Jackie-Jiang merged pull request #5780:
URL: https://github.com/apache/incubator-pinot/pull/5780
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
npawar removed a comment on issue #5753:
URL:
https://github.com/apache/incubator-pinot/issues/5753#issuecomment-665126719
This will be covered as part of tiered storage:
https://github.com/apache/incubator-pinot/issues/5553
---
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch
support_aggregation_function_name_with_underscore
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 49e18e4 Support aggregation function name with underscore insi
Jackie-Jiang commented on a change in pull request #5786:
URL: https://github.com/apache/incubator-pinot/pull/5786#discussion_r464646026
##
File path:
pinot-common/src/main/java/org/apache/pinot/common/function/AggregationFunctionType.java
##
@@ -28,6 +28,7 @@
MINMAXRANGE("
Jackie-Jiang commented on a change in pull request #5786:
URL: https://github.com/apache/incubator-pinot/pull/5786#discussion_r464645873
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/PartitionedDistinctCountAggregationFunction.java
##
This is an automated email from the ASF dual-hosted git repository.
jihao pushed a change to branch anomalies-pagination
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from ee879b1 - new anomalies endpoint and unit tests
add 9586523 address comments
No new rev
fx19880617 opened a new pull request #5795:
URL: https://github.com/apache/incubator-pinot/pull/5795
## Description
Support aggregation function name with underscore inside
This is an automated message from the Apache
Jackie-Jiang commented on a change in pull request #5786:
URL: https://github.com/apache/incubator-pinot/pull/5786#discussion_r464642766
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/AggregationFunctionFactory.java
##
@@ -123,6 +123,8
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch
support_aggregation_function_name_with_underscore
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at 49e18e4 Support aggregation function name with underscore insi
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch
support_aggregation_function_name_with_underscore
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 49e18e4338819147993697266a8687174506dcc8
Author: Xiang Fu
AuthorDa
mangrrua removed a comment on pull request #5791:
URL: https://github.com/apache/incubator-pinot/pull/5791#issuecomment-668195859
> Can you please add an integration test case so that we dont break this
across releases?
Yes i can, but i could not found any integration tests about `ro
akshayrai opened a new pull request #5794:
URL: https://github.com/apache/incubator-pinot/pull/5794
* Added a version 2 of cube api that returns a map rather than a string.
`/summary/autoDimensionOrder/v2`
* Added a safeguard in the notification pipeline that doesn't fail the
anomaly no
npawar opened a new pull request #5793:
URL: https://github.com/apache/incubator-pinot/pull/5793
## Description
Issue: https://github.com/apache/incubator-pinot/issues/5553
Tiered storage support in Pinot - Phase 1.
This phase supports default tag based instance assignments only, wh
mangrrua commented on pull request #5791:
URL: https://github.com/apache/incubator-pinot/pull/5791#issuecomment-668195859
> Can you please add an integration test case so that we dont break this
across releases?
Yes i can, but i could not found any integration tests about `routing ta
This is an automated email from the ASF dual-hosted git repository.
akshayrai09 pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 0e95a67 Do not apply shade plugin on parent modules (#5761)
add 47623c0 [TE] Error handling refa
akshayrai merged pull request #5788:
URL: https://github.com/apache/incubator-pinot/pull/5788
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
suvodeep-pyne opened a new pull request #5792:
URL: https://github.com/apache/incubator-pinot/pull/5792
##Description
This refactor further extends the use of Guice beyond the datalayer by
wrapping all resource classes and dependencies with Guice.
Changes:
- replaces the top
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464543147
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/connector/PinotServerDataFetcher.scala
##
@
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464543147
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/connector/PinotServerDataFetcher.scala
##
@
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464544165
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/connector/PinotServerDataFetcher.scala
##
@
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464522910
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/connector/PinotClusterClient.scala
##
@@ -0
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464543147
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/connector/PinotServerDataFetcher.scala
##
@
mangrrua opened a new pull request #5791:
URL: https://github.com/apache/incubator-pinot/pull/5791
`PinotBrokerDebug` API contains endpoint that get routing table for PQL
query. This new endpoint gets routing table for SQL query.
What is the required for?
- Pinot is moving f
pradeepgv42 commented on pull request #5758:
URL: https://github.com/apache/incubator-pinot/pull/5758#issuecomment-668125822
@elonazoulay could you review this when you get a chance?
This is an automated message from the Apa
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464526782
##
File path: pinot-connectors/pinot-spark-connector/documentation/read_model.md
##
@@ -0,0 +1,145 @@
+
+# Read Model
+
+Connector can scan offli
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464524102
##
File path: pinot-connectors/pinot-spark-connector/documentation/read_model.md
##
@@ -0,0 +1,145 @@
+
+# Read Model
+
+Connector can scan offli
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464522910
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/connector/PinotClusterClient.scala
##
@@ -0
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464513043
##
File path:
pinot-connectors/pinot-spark-connector/src/test/resources/schema/pinot-schema.json
##
@@ -0,0 +1,57 @@
+{
Review comment:
mcvsubbu commented on issue #4001:
URL:
https://github.com/apache/incubator-pinot/issues/4001#issuecomment-668102031
Looks like this is a duplicate of #4854 so we can track discussions there.
Closing this one
This is an aut
mcvsubbu closed issue #4001:
URL: https://github.com/apache/incubator-pinot/issues/4001
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
mcvsubbu edited a comment on issue #4854:
URL:
https://github.com/apache/incubator-pinot/issues/4854#issuecomment-668098131
Related Issue #5490 #4001
This is an automated message from the Apache Git Service.
To respond to t
mcvsubbu commented on issue #5490:
URL:
https://github.com/apache/incubator-pinot/issues/5490#issuecomment-668101022
Here is the design doc for compatibility testing
https://docs.google.com/document/d/1yNlvnLKDNUuyRWOKYYF01FWW9weYMGoaLRtU-CueciM/edit?ts=5e38bdd0#heading=h.ngrw7r8gso5w
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464504242
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/datasource/PinotDataSourceReader.scala
##
@
mcvsubbu commented on issue #4854:
URL:
https://github.com/apache/incubator-pinot/issues/4854#issuecomment-668098131
Related Issue #5490
This is an automated message from the Apache Git Service.
To respond to the message, p
mayankshriv commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464474155
##
File path:
pinot-connectors/pinot-spark-connector/src/test/resources/schema/pinot-schema.json
##
@@ -0,0 +1,57 @@
+{
Review comment:
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464326792
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/connector/PinotServerDataFetcher.scala
##
@
mangrrua edited a comment on pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#issuecomment-667951838
> This is great work! Looking forward to streaming API for read and writeAPI
:)
>
> One question maybe related to this, shall we create a PinotRDD?
mangrrua commented on pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#issuecomment-667951838
> This is great work! Looking forward to streaming API for read and writeAPI
:)
>
> One question maybe related to this, shall we create a PinotRDD?
Thank y
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464326792
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/connector/PinotServerDataFetcher.scala
##
@
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464324725
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/connector/query/SQLSelectionQueryGenerator.scala
##
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464317075
##
File path:
pinot-connectors/pinot-spark-connector/src/main/scala/org/apache/pinot/connector/spark/connector/PinotSplitter.scala
##
@@ -0,0 +1
mangrrua commented on a change in pull request #5787:
URL: https://github.com/apache/incubator-pinot/pull/5787#discussion_r464311850
##
File path: config/.scalafmt.conf
##
@@ -0,0 +1,17 @@
+version = "2.4.0"
Review comment:
Defined rules to format scala code
---
fx19880617 merged pull request #5761:
URL: https://github.com/apache/incubator-pinot/pull/5761
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
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 0e95a67 Do not apply shade plugin on
63 matches
Mail list logo
