[GitHub] [incubator-pinot] icefury71 opened a new pull request #6585: Fixing schema and table config for githubEvents demo

2021-02-16 Thread GitBox
icefury71 opened a new pull request #6585: URL: https://github.com/apache/incubator-pinot/pull/6585 ## Description The current schema and table config creates some issues when using the Cluster Manager UI. This PR fixes that issue. ## Upgrade Notes Does this PR prevent a ze

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6583: Implement QueryOp class.

2021-02-16 Thread GitBox
Jackie-Jiang commented on a change in pull request #6583: URL: https://github.com/apache/incubator-pinot/pull/6583#discussion_r577304465 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/compat/tests/CompatibilityOpsRunner.java ## @@ -36,8 +36,7 @@ private C

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #6466: Handle creation of segments with 0 rows

2021-02-16 Thread GitBox
Jackie-Jiang commented on a change in pull request #6466: URL: https://github.com/apache/incubator-pinot/pull/6466#discussion_r577303043 ## File path: pinot-core/src/main/java/org/apache/pinot/core/indexsegment/immutable/ImmutableSegmentLoader.java ## @@ -132,7 +135,7 @@ publi

[GitHub] [incubator-pinot] laxmanchekka commented on issue #6362: Support replacing real-time committed segment

2021-02-16 Thread GitBox
laxmanchekka commented on issue #6362: URL: https://github.com/apache/incubator-pinot/issues/6362#issuecomment-780273153 @Jackie-Jiang : is this duplicate of https://github.com/apache/incubator-pinot/pull/6567 This is an au

[GitHub] [incubator-pinot] codecov-io edited a comment on pull request #6546: Improve Real Time Provisioning Helper tool

2021-02-16 Thread GitBox
codecov-io edited a comment on pull request #6546: URL: https://github.com/apache/incubator-pinot/pull/6546#issuecomment-773727645 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6546?src=pr&el=h1) Report > Merging [#6546](https://codecov.io/gh/apache/incubator-pinot/pull

[GitHub] [incubator-pinot] sajjad-moradi commented on a change in pull request #6546: Improve Real Time Provisioning Helper tool

2021-02-16 Thread GitBox
sajjad-moradi commented on a change in pull request #6546: URL: https://github.com/apache/incubator-pinot/pull/6546#discussion_r577270646 ## File path: pinot-tools/src/test/resources/memory_estimation/schema.json ## @@ -0,0 +1,36 @@ +{ + "schemaName": "testTable", + "dimensio

[GitHub] [incubator-pinot] sajjad-moradi commented on a change in pull request #6546: Improve Real Time Provisioning Helper tool

2021-02-16 Thread GitBox
sajjad-moradi commented on a change in pull request #6546: URL: https://github.com/apache/incubator-pinot/pull/6546#discussion_r577268997 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/data/generator/StringGenerator.java ## @@ -56,11 +65,18 @@ public void init(

[GitHub] [incubator-pinot] sajjad-moradi commented on a change in pull request #6546: Improve Real Time Provisioning Helper tool

2021-02-16 Thread GitBox
sajjad-moradi commented on a change in pull request #6546: URL: https://github.com/apache/incubator-pinot/pull/6546#discussion_r577251040 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/realtime/provisioning/MemoryEstimator.java ## @@ -107,6 +129,15 @@ public Me

[GitHub] [incubator-pinot] sajjad-moradi commented on a change in pull request #6546: Improve Real Time Provisioning Helper tool

2021-02-16 Thread GitBox
sajjad-moradi commented on a change in pull request #6546: URL: https://github.com/apache/incubator-pinot/pull/6546#discussion_r577250920 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/realtime/provisioning/MemoryEstimator.java ## @@ -387,4 +418,182 @@ private

[GitHub] [incubator-pinot] sajjad-moradi commented on a change in pull request #6546: Improve Real Time Provisioning Helper tool

2021-02-16 Thread GitBox
sajjad-moradi commented on a change in pull request #6546: URL: https://github.com/apache/incubator-pinot/pull/6546#discussion_r577250380 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/data/generator/DataGenerator.java ## @@ -163,26 +174,75 @@ private FieldSpec

[GitHub] [incubator-pinot] sajjad-moradi commented on a change in pull request #6546: Improve Real Time Provisioning Helper tool

2021-02-16 Thread GitBox
sajjad-moradi commented on a change in pull request #6546: URL: https://github.com/apache/incubator-pinot/pull/6546#discussion_r577250274 ## File path: pinot-tools/src/test/java/org/apache/pinot/tools/realtime/provisioning/MemoryEstimatorTest.java ## @@ -0,0 +1,85 @@ +/** + *

[incubator-pinot] branch master updated (7a94181 -> 1a5a3e0)

2021-02-16 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 7a94181 Fix QueryRunner for multiple runs (#6582) add 1a5a3e0 [TE] Enable breakdown endpoint to handle

[GitHub] [incubator-pinot] jihaozh merged pull request #6578: [TE] Enable breakdown endpoint to handle forecast offset, repair metr…

2021-02-16 Thread GitBox
jihaozh merged pull request #6578: URL: https://github.com/apache/incubator-pinot/pull/6578 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 t

[GitHub] [incubator-pinot] jihaozh commented on pull request #6578: [TE] Enable breakdown endpoint to handle forecast offset, repair metr…

2021-02-16 Thread GitBox
jihaozh commented on pull request #6578: URL: https://github.com/apache/incubator-pinot/pull/6578#issuecomment-780175282 LGTM This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-pinot] fx19880617 commented on issue #1051: Date function supports in PQL

2021-02-16 Thread GitBox
fx19880617 commented on issue #1051: URL: https://github.com/apache/incubator-pinot/issues/1051#issuecomment-780129116 This is supported now, please refer to the datetime functions: https://docs.pinot.apache.org/users/user-guide-query/supported-transformations#datetime-functions -

[GitHub] [incubator-pinot] fx19880617 closed issue #1051: Date function supports in PQL

2021-02-16 Thread GitBox
fx19880617 closed issue #1051: URL: https://github.com/apache/incubator-pinot/issues/1051 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] codecov-io edited a comment on pull request #6403: [WIP] Allow upcasting and downcasting of numerical values in SQL predicates.

2021-02-16 Thread GitBox
codecov-io edited a comment on pull request #6403: URL: https://github.com/apache/incubator-pinot/pull/6403#issuecomment-759105793 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6403?src=pr&el=h1) Report > Merging [#6403](https://codecov.io/gh/apache/incubator-pinot/pull

[GitHub] [incubator-pinot] codecov-io commented on pull request #6583: Implement QueryOp class.

2021-02-16 Thread GitBox
codecov-io commented on pull request #6583: URL: https://github.com/apache/incubator-pinot/pull/6583#issuecomment-780110310 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/6583?src=pr&el=h1) Report > Merging [#6583](https://codecov.io/gh/apache/incubator-pinot/pull/6583?s

[GitHub] [incubator-pinot] npawar commented on a change in pull request #6466: Handle creation of segments with 0 rows

2021-02-16 Thread GitBox
npawar commented on a change in pull request #6466: URL: https://github.com/apache/incubator-pinot/pull/6466#discussion_r577120279 ## File path: pinot-core/src/main/java/org/apache/pinot/core/indexsegment/immutable/ImmutableSegmentImpl.java ## @@ -115,7 +116,11 @@ public DataS

[GitHub] [incubator-pinot] mqliang commented on pull request #6583: Implement QueryOp class.

2021-02-16 Thread GitBox
mqliang commented on pull request #6583: URL: https://github.com/apache/incubator-pinot/pull/6583#issuecomment-780087492 @mcvsubbu This is an automated message from the Apache Git Service. To respond to the message, plea

[incubator-pinot] branch master updated (a95c9b2 -> 7a94181)

2021-02-16 Thread jackie
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 a95c9b2 [TE] Enable autotuning API (#6584) add 7a94181 Fix QueryRunner for multiple runs (#6582) No

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #6582: Fix QueryRunner for multiple runs

2021-02-16 Thread GitBox
Jackie-Jiang merged pull request #6582: URL: https://github.com/apache/incubator-pinot/pull/6582 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

[incubator-pinot] branch master updated (b97cce5 -> a95c9b2)

2021-02-16 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from b97cce5 [TE]: fix eslint reported issue under the serializers, the models, and the adapters folders (#6547)

[GitHub] [incubator-pinot] jihaozh merged pull request #6584: [TE] Enable autotuning API

2021-02-16 Thread GitBox
jihaozh merged pull request #6584: URL: https://github.com/apache/incubator-pinot/pull/6584 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 t

[GitHub] [incubator-pinot] bryantachen opened a new pull request #6584: [TE] Enable autotuning API

2021-02-16 Thread GitBox
bryantachen opened a new pull request #6584: URL: https://github.com/apache/incubator-pinot/pull/6584 Added a new resource to do autotuning. This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-pinot] mqliang opened a new pull request #6583: Implement QueryOp class.

2021-02-16 Thread GitBox
mqliang opened a new pull request #6583: URL: https://github.com/apache/incubator-pinot/pull/6583 ## Description Implement QueryOp class, which take a file that has queries, and a file that has results, run the queries through the broker/server and compare the results with the result fi

[incubator-pinot] branch master updated (abf21ed -> b97cce5)

2021-02-16 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from abf21ed Compatibility test: streamOp (#6419) add b97cce5 [TE]: fix eslint reported issue under the ser

[GitHub] [incubator-pinot] jihaozh merged pull request #6547: [TE]: fix eslint reported issue under the serializers, the models, and the adapters folders

2021-02-16 Thread GitBox
jihaozh merged pull request #6547: URL: https://github.com/apache/incubator-pinot/pull/6547 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 t

[incubator-pinot] branch master updated (d0177f9 -> abf21ed)

2021-02-16 Thread mcvsubbu
This is an automated email from the ASF dual-hosted git repository. mcvsubbu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from d0177f9 [TE]: Fix all eslint reported errors and warnings under app/pods/components (#6568) add abf

[GitHub] [incubator-pinot] mcvsubbu merged pull request #6419: Compatibility test: streamOp

2021-02-16 Thread GitBox
mcvsubbu merged pull request #6419: URL: https://github.com/apache/incubator-pinot/pull/6419 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 master updated (a733317 -> d0177f9)

2021-02-16 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from a733317 [TE]: Fix all eslint warning/error under utils and shared folders (#6542) add d0177f9 [TE]: F

[GitHub] [incubator-pinot] jihaozh merged pull request #6568: [TE]: Fix all eslint reported errors and warnings under app/pods/components

2021-02-16 Thread GitBox
jihaozh merged pull request #6568: URL: https://github.com/apache/incubator-pinot/pull/6568 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 t

[incubator-pinot] branch master updated (50ac78c -> a733317)

2021-02-16 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 50ac78c URL encoded generated segment tar name (#6571) add a733317 [TE]: Fix all eslint warning/error

[GitHub] [incubator-pinot] jihaozh merged pull request #6542: [TE]: Fix all eslint warning/error under utils and shared folders

2021-02-16 Thread GitBox
jihaozh merged pull request #6542: URL: https://github.com/apache/incubator-pinot/pull/6542 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 t

[GitHub] [incubator-pinot] louisthomas commented on issue #6573: Report explicit error message for missing properties in S3PinotFS

2021-02-16 Thread GitBox
louisthomas commented on issue #6573: URL: https://github.com/apache/incubator-pinot/issues/6573#issuecomment-779810328 I would like to work on this issue, thanks! This is an automated message from the Apache Git Service. To

[incubator-pinot] branch master updated: URL encoded generated segment tar name (#6571)

2021-02-16 Thread xiangfu
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 50ac78c URL encoded generated segment

[GitHub] [incubator-pinot] fx19880617 merged pull request #6571: URL encoded generated segment name in ingestion jobs

2021-02-16 Thread GitBox
fx19880617 merged pull request #6571: URL: https://github.com/apache/incubator-pinot/pull/6571 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 a change in pull request #6581: Add support for getObject to Pinot JDBC driver

2021-02-16 Thread GitBox
fx19880617 commented on a change in pull request #6581: URL: https://github.com/apache/incubator-pinot/pull/6581#discussion_r576652602 ## File path: pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotResultSet.java ## @@ -297,6 +299,54 @@ public String

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #6581: Add support for getObject to Pinot JDBC driver

2021-02-16 Thread GitBox
fx19880617 commented on a change in pull request #6581: URL: https://github.com/apache/incubator-pinot/pull/6581#discussion_r576652602 ## File path: pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotResultSet.java ## @@ -297,6 +299,54 @@ public String