[incubator-pinot] branch h3-index updated (28f59ef -> 83bc217)

2020-12-30 Thread kishoreg
This is an automated email from the ASF dual-hosted git repository. kishoreg pushed a change to branch h3-index in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 28f59ef Added H3IndexFilterOperator (#6391) add 83bc217 Cleaned up H3Index Reader/Creator (#6397)

[GitHub] [incubator-pinot] kishoreg merged pull request #6397: Clean up H3Index Reader/Creator

2020-12-30 Thread GitBox
kishoreg merged pull request #6397: URL: https://github.com/apache/incubator-pinot/pull/6397 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

[GitHub] [incubator-pinot] kishoreg commented on a change in pull request #6397: Clean up H3Index Reader/Creator

2020-12-30 Thread GitBox
kishoreg commented on a change in pull request #6397: URL: https://github.com/apache/incubator-pinot/pull/6397#discussion_r550418324 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/creator/impl/geospatial/H3IndexCreator.java ## @@ -303,7 +339,6 @@ public s

[GitHub] [incubator-pinot] amarnathkarthik commented on issue #6298: Support Describe Pinot table and schema

2020-12-30 Thread GitBox
amarnathkarthik commented on issue #6298: URL: https://github.com/apache/incubator-pinot/issues/6298#issuecomment-752866538 Thanks for the clarification! Yes, parsing `SHOW ` should be straight forward implementation. Here is the list of SHOW SQL Statements in the scope (Please

[incubator-pinot] branch master updated: Segment reset API (#6336)

2020-12-30 Thread nehapawar
This is an automated email from the ASF dual-hosted git repository. nehapawar 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 37f2e28 Segment reset API (#6336) 3

[GitHub] [incubator-pinot] npawar merged pull request #6336: Segment reset API

2020-12-30 Thread GitBox
npawar merged pull request #6336: URL: https://github.com/apache/incubator-pinot/pull/6336 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

[GitHub] [incubator-pinot] jackjlli merged pull request #6384: Make required interfaces or classes serializable for spark

2020-12-30 Thread GitBox
jackjlli merged pull request #6384: URL: https://github.com/apache/incubator-pinot/pull/6384 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

[incubator-pinot] branch make-objects-serializable updated (322676c -> fc1e400)

2020-12-30 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch make-objects-serializable in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 322676c Make required interfaces or classes serializable for spark add fc1e400 Make

[GitHub] [incubator-pinot] yupeng9 opened a new pull request #6397: Cleaned up H3Index Reader/Creator

2020-12-30 Thread GitBox
yupeng9 opened a new pull request #6397: URL: https://github.com/apache/incubator-pinot/pull/6397 Cleaned up H3Index Reader/Creator, added some docs. Also, changed the index config to include resolutions. This is an automate

[GitHub] [incubator-pinot] jackjlli commented on pull request #6384: Make required interfaces or classes serializable for spark

2020-12-30 Thread GitBox
jackjlli commented on pull request #6384: URL: https://github.com/apache/incubator-pinot/pull/6384#issuecomment-752734967 > > lgtm, not sure if v0_deprecatd is a good place to put this test, but we can always move it if needed. > > Maybe move it into `pinot-connectors/pinot-spark-con

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6336: Segment reset API

2020-12-30 Thread GitBox
codecov-io edited a comment on pull request #6336: URL: https://github.com/apache/incubator-pinot/pull/6336#issuecomment-741130249 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6336?src=pr&el=h1) Report > Merging [#6336](https://codecov.io/gh/apache/incubator-pinot/pull

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6383: Introduce 'LOOKUP' Transform Function

2020-12-30 Thread GitBox
Jackie-Jiang commented on a change in pull request #6383: URL: https://github.com/apache/incubator-pinot/pull/6383#discussion_r550291479 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/LookupTransformFunction.java ## @@ -0,0 +1,332 @@ +

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6336: Segment reset API

2020-12-30 Thread GitBox
codecov-io edited a comment on pull request #6336: URL: https://github.com/apache/incubator-pinot/pull/6336#issuecomment-741130249 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6336?src=pr&el=h1) Report > Merging [#6336](https://codecov.io/gh/apache/incubator-pinot/pull

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6383: Introduce 'LOOKUP' Transform Function

2020-12-30 Thread GitBox
Jackie-Jiang commented on a change in pull request #6383: URL: https://github.com/apache/incubator-pinot/pull/6383#discussion_r550291479 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/LookupTransformFunction.java ## @@ -0,0 +1,332 @@ +

[GitHub] [incubator-pinot] kishoreg commented on issue #6378: Create a dashboard for most expensive queries per table

2020-12-30 Thread GitBox
kishoreg commented on issue #6378: URL: https://github.com/apache/incubator-pinot/issues/6378#issuecomment-752684947 why not write the query response back to Pinot. Have a logger that logs the query response to kafka topic and have that ingested into Pinot. We can build multiple visualiza

[incubator-pinot] branch master updated (13bd443 -> 93a4515)

2020-12-30 Thread xiangfu
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/incubator-pinot.git. from 13bd443 Fix remaining links to outdated helm chart repos. (#6394) add 93a4515 Adding Pinot minion se

[GitHub] [incubator-pinot] fx19880617 merged pull request #6340: Adding Pinot minion segment generation and push task.

2020-12-30 Thread GitBox
fx19880617 merged pull request #6340: URL: https://github.com/apache/incubator-pinot/pull/6340 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

[GitHub] [incubator-pinot] fx19880617 commented on pull request #6384: Make required interfaces or classes serializable for spark

2020-12-30 Thread GitBox
fx19880617 commented on pull request #6384: URL: https://github.com/apache/incubator-pinot/pull/6384#issuecomment-752397181 > lgtm, not sure if v0_deprecatd is a good place to put this test, but we can always move it if needed. Maybe move it into `pinot-connectors/pinot-spark-connect

[GitHub] [incubator-pinot] fx19880617 edited a comment on issue #6298: Support Describe Pinot table and schema

2020-12-30 Thread GitBox
fx19880617 edited a comment on issue #6298: URL: https://github.com/apache/incubator-pinot/issues/6298#issuecomment-752394089 > @fx19880617 Here is my initial analysis, correct me if I'm wrong. > Calcite does not support `SHOW (databases | tables)` kind of SQL, below are the available i

[GitHub] [incubator-pinot] fx19880617 commented on issue #6298: Support Describe Pinot table and schema

2020-12-30 Thread GitBox
fx19880617 commented on issue #6298: URL: https://github.com/apache/incubator-pinot/issues/6298#issuecomment-752394089 > @fx19880617 Here is my initial analysis, correct me if I'm wrong. > Calcite does not support `SHOW (databases | tables)` kind of SQL, below are the available implemen

[GitHub] [incubator-pinot] amarnathkarthik edited a comment on issue #6298: Support Describe Pinot table and schema

2020-12-30 Thread GitBox
amarnathkarthik edited a comment on issue #6298: URL: https://github.com/apache/incubator-pinot/issues/6298#issuecomment-752363299 @fx19880617 Here is my initial analysis, correct me if I'm wrong. Calcite does not support `SHOW (databases | tables)` kind of SQL, below are the available

[GitHub] [incubator-pinot] KKcorps commented on issue #5362: Add UDFs for String Transformation

2020-12-30 Thread GitBox
KKcorps commented on issue #5362: URL: https://github.com/apache/incubator-pinot/issues/5362#issuecomment-752383154 @SandishKumarHN Functions for 1 and 3rd already exist. You can implement the rest in the class provided by @fx19880617

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6396: Adding ImportData sub command in pinot admin

2020-12-30 Thread GitBox
codecov-io edited a comment on pull request #6396: URL: https://github.com/apache/incubator-pinot/pull/6396#issuecomment-752361098 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6396?src=pr&el=h1) Report > Merging [#6396](https://codecov.io/gh/apache/incubator-pinot/pull

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6396: Adding ImportData sub command in pinot admin

2020-12-30 Thread GitBox
codecov-io edited a comment on pull request #6396: URL: https://github.com/apache/incubator-pinot/pull/6396#issuecomment-752361098 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6396?src=pr&el=h1) Report > Merging [#6396](https://codecov.io/gh/apache/incubator-pinot/pull