[pinot] branch master updated: Add Support for Schema Registry in Protobuf Decoder (#9220)

2022-08-17 Thread kharekartik
This is an automated email from the ASF dual-hosted git repository. kharekartik pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git The following commit(s) were added to refs/heads/master by this push: new 42a2fc70ff Add Support for Schema Registry i

[GitHub] [pinot] KKcorps merged pull request #9220: Add Support for Schema Registry in Protobuf Decoder

2022-08-17 Thread GitBox
KKcorps merged PR #9220: URL: https://github.com/apache/pinot/pull/9220 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache

[pinot] branch add-table-auth-cleanup updated (379766a080 -> 42161ca9e3)

2022-08-17 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch add-table-auth-cleanup in repository https://gitbox.apache.org/repos/asf/pinot.git from 379766a080 cleanup add 42161ca9e3 review fixes No new revisions were added by this update. Summ

[GitHub] [pinot] apucher commented on a diff in pull request #9228: add-table refactor for clean auth

2022-08-17 Thread GitBox
apucher commented on code in PR #9228: URL: https://github.com/apache/pinot/pull/9228#discussion_r948685613 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/access/AccessControlUtils.java: ## @@ -100,11 +91,15 @@ public void validatePermission(Optional tableNam

[GitHub] [pinot] codecov-commenter commented on pull request #9239: refactor query and option compilation

2022-08-17 Thread GitBox
codecov-commenter commented on PR #9239: URL: https://github.com/apache/pinot/pull/9239#issuecomment-1218920372 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9239?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] walterddr commented on a diff in pull request #9239: refactor query and option compilation

2022-08-17 Thread GitBox
walterddr commented on code in PR #9239: URL: https://github.com/apache/pinot/pull/9239#discussion_r948585699 ## pinot-common/src/main/java/org/apache/pinot/sql/parsers/SqlNodeAndOptions.java: ## @@ -23,17 +23,25 @@ public class SqlNodeAndOptions { + private String _sql;

[GitHub] [pinot] walterddr commented on a diff in pull request #9239: refactor query and option compilation

2022-08-17 Thread GitBox
walterddr commented on code in PR #9239: URL: https://github.com/apache/pinot/pull/9239#discussion_r948585699 ## pinot-common/src/main/java/org/apache/pinot/sql/parsers/SqlNodeAndOptions.java: ## @@ -23,17 +23,25 @@ public class SqlNodeAndOptions { + private String _sql;

[GitHub] [pinot] walterddr opened a new pull request, #9239: refactor sql compiler

2022-08-17 Thread GitBox
walterddr opened a new pull request, #9239: URL: https://github.com/apache/pinot/pull/9239 This is part2 follow up of #9191 - move option parsing and query compile to common utils - support option string parsing as well from controller rest api -- This is an automated me

[GitHub] [pinot] walterddr commented on a diff in pull request #9237: move back brokerrequesthandler interface change

2022-08-17 Thread GitBox
walterddr commented on code in PR #9237: URL: https://github.com/apache/pinot/pull/9237#discussion_r948579952 ## pinot-common/src/main/java/org/apache/pinot/common/response/BrokerResponse.java: ## @@ -137,6 +138,18 @@ String toJsonString() */ int getExceptionsSize(); +

[pinot] branch master updated: move back interface change (#9237)

2022-08-17 Thread rongr
This is an automated email from the ASF dual-hosted git repository. rongr pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git The following commit(s) were added to refs/heads/master by this push: new 3cd195213e move back interface change (#9237) 3cd1

[GitHub] [pinot] walterddr merged pull request #9237: move back brokerrequesthandler interface change

2022-08-17 Thread GitBox
walterddr merged PR #9237: URL: https://github.com/apache/pinot/pull/9237 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.apac

[GitHub] [pinot] codecov-commenter commented on pull request #9238: Add minMaxInvalid flag to avoid unnecessary needPreprocess

2022-08-17 Thread GitBox
codecov-commenter commented on PR #9238: URL: https://github.com/apache/pinot/pull/9238#issuecomment-1218809276 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9238?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] codecov-commenter commented on pull request #9237: move back brokerrequesthandler interface change

2022-08-17 Thread GitBox
codecov-commenter commented on PR #9237: URL: https://github.com/apache/pinot/pull/9237#issuecomment-1218780256 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9237?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] Jackie-Jiang commented on a diff in pull request #9237: move back brokerrequesthandler interface change

2022-08-17 Thread GitBox
Jackie-Jiang commented on code in PR #9237: URL: https://github.com/apache/pinot/pull/9237#discussion_r948546528 ## pinot-common/src/main/java/org/apache/pinot/common/response/BrokerResponse.java: ## @@ -137,6 +138,18 @@ String toJsonString() */ int getExceptionsSize();

[GitHub] [pinot] codecov-commenter commented on pull request #9224: Extract record keys, headers and metadata from Stream sources

2022-08-17 Thread GitBox
codecov-commenter commented on PR #9224: URL: https://github.com/apache/pinot/pull/9224#issuecomment-1218743695 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9224?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] npawar opened a new pull request, #9238: Add minMaxInvalid flag to avoid unnecessary needPreprocess

2022-08-17 Thread GitBox
npawar opened a new pull request, #9238: URL: https://github.com/apache/pinot/pull/9238 Instructions: 1. The PR has to be tagged with at least one of the following labels (*): 1. `feature` 2. `bugfix` 3. `performance` 4. `ui` 5. `backward-incompat` 6. `r

[GitHub] [pinot] walterddr closed pull request #9191: Support using query options to choose v2 engine

2022-08-17 Thread GitBox
walterddr closed pull request #9191: Support using query options to choose v2 engine URL: https://github.com/apache/pinot/pull/9191 -- 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 comme

[GitHub] [pinot] walterddr commented on pull request #9191: Support using query options to choose v2 engine

2022-08-17 Thread GitBox
walterddr commented on PR #9191: URL: https://github.com/apache/pinot/pull/9191#issuecomment-1218720139 +1. creating #9237 to first move the API back will create another one for option refactoring and support controller side query option -- This is an automated message from the Apache

[GitHub] [pinot] walterddr opened a new pull request, #9237: move back brokerrequesthandler interface change

2022-08-17 Thread GitBox
walterddr opened a new pull request, #9237: URL: https://github.com/apache/pinot/pull/9237 #9202 changed broker request handler interface. changing it back by adding result table getter/setter to broker response. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [pinot] Jackie-Jiang merged pull request #9205: Max number of multi values should be set in DefaultNullValueProvider

2022-08-17 Thread GitBox
Jackie-Jiang merged PR #9205: URL: https://github.com/apache/pinot/pull/9205 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.a

[pinot] branch master updated: Max number of multi values should be set in DefaultNullValueProvider (#9205)

2022-08-17 Thread jackie
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/pinot.git The following commit(s) were added to refs/heads/master by this push: new cc92e4cf68 Max number of multi values should be s

[GitHub] [pinot] Jackie-Jiang commented on a diff in pull request #9191: Support using query options to choose v2 engine

2022-08-17 Thread GitBox
Jackie-Jiang commented on code in PR #9191: URL: https://github.com/apache/pinot/pull/9191#discussion_r948529540 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotQueryResource.java: ## @@ -121,9 +121,17 @@ public String handleGetSql(@QueryParam("s

[GitHub] [pinot] SabrinaZhaozyf opened a new pull request, #9236: [WIP] Add Support for Covariance Function

2022-08-17 Thread GitBox
SabrinaZhaozyf opened a new pull request, #9236: URL: https://github.com/apache/pinot/pull/9236 Label: `feature` This PR adds support for COV_POP function to pinot. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [pinot] sajjad-moradi commented on a diff in pull request #9228: add-table refactor for clean auth

2022-08-17 Thread GitBox
sajjad-moradi commented on code in PR #9228: URL: https://github.com/apache/pinot/pull/9228#discussion_r948461730 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/access/AccessControlUtils.java: ## @@ -100,11 +91,15 @@ public void validatePermission(Optional ta

[GitHub] [pinot] walterddr commented on a diff in pull request #9191: Support using query options to choose v2 engine

2022-08-17 Thread GitBox
walterddr commented on code in PR #9191: URL: https://github.com/apache/pinot/pull/9191#discussion_r948457895 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -202,24 +208,21 @@ private BrokerResponseNative handleRequest(l

[GitHub] [pinot] walterddr commented on a diff in pull request #9191: Support using query options to choose v2 engine

2022-08-17 Thread GitBox
walterddr commented on code in PR #9191: URL: https://github.com/apache/pinot/pull/9191#discussion_r948457277 ## pinot-common/src/main/java/org/apache/pinot/sql/parsers/SqlNodeAndOptions.java: ## @@ -23,17 +23,25 @@ public class SqlNodeAndOptions { + private String _sql;

[GitHub] [pinot] Jackie-Jiang commented on a diff in pull request #9184: [WIP] Add support for EXTRACT syntax and converts it to appropriate Pinot expression

2022-08-17 Thread GitBox
Jackie-Jiang commented on code in PR #9184: URL: https://github.com/apache/pinot/pull/9184#discussion_r948437161 ## pinot-common/src/thrift/query.thrift: ## @@ -70,4 +70,4 @@ union Literal { struct Function { 1: required string operator; 2: optional list operands; -} +}

[pinot] branch master updated (766769253b -> 8f495aefe8)

2022-08-17 Thread siddteotia
This is an automated email from the ASF dual-hosted git repository. siddteotia pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from 766769253b [multistage] fixing count not resolving as SUM0 in intermediate stage (#9233) add 8f495aefe8 Bug f

[GitHub] [pinot] siddharthteotia merged pull request #9235: Bug fix: Nested arrays and map not handled correctly for complex types

2022-08-17 Thread GitBox
siddharthteotia merged PR #9235: URL: https://github.com/apache/pinot/pull/9235 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pino

[pinot] branch master updated (3a655d2d0e -> 766769253b)

2022-08-17 Thread siddteotia
This is an automated email from the ASF dual-hosted git repository. siddteotia pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from 3a655d2d0e support to show running queries and cancel query by id (#9171) add 766769253b [multistage] fixing

[GitHub] [pinot] siddharthteotia closed issue #9190: [multistage] Aggregation Query with Join Fails with Classcast Exception

2022-08-17 Thread GitBox
siddharthteotia closed issue #9190: [multistage] Aggregation Query with Join Fails with Classcast Exception URL: https://github.com/apache/pinot/issues/9190 -- 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] [pinot] siddharthteotia merged pull request #9233: [multistage] fixing count not resolving as SUM0 in intermediate stage

2022-08-17 Thread GitBox
siddharthteotia merged PR #9233: URL: https://github.com/apache/pinot/pull/9233 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pino

[GitHub] [pinot] siddharthteotia commented on a diff in pull request #9191: Support using query options to choose v2 engine

2022-08-17 Thread GitBox
siddharthteotia commented on code in PR #9191: URL: https://github.com/apache/pinot/pull/9191#discussion_r948397953 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -289,8 +292,9 @@ private BrokerResponseNative handleReque

[GitHub] [pinot] siddharthteotia commented on a diff in pull request #9191: Support using query options to choose v2 engine

2022-08-17 Thread GitBox
siddharthteotia commented on code in PR #9191: URL: https://github.com/apache/pinot/pull/9191#discussion_r948393596 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -202,24 +208,21 @@ private BrokerResponseNative handleReq

[GitHub] [pinot] Jackie-Jiang commented on issue #7213: Support generic sliding window aggregations

2022-08-17 Thread GitBox
Jackie-Jiang commented on issue #7213: URL: https://github.com/apache/pinot/issues/7213#issuecomment-1218471084 @amrishlal I don't see that in the design doc. If you have created a new one, please share it here. IMO, it is not really possible to compute the window function on the server

[GitHub] [pinot] siddharthteotia commented on a diff in pull request #9191: Support using query options to choose v2 engine

2022-08-17 Thread GitBox
siddharthteotia commented on code in PR #9191: URL: https://github.com/apache/pinot/pull/9191#discussion_r948393596 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -202,24 +208,21 @@ private BrokerResponseNative handleReq

[GitHub] [pinot] amrishlal commented on issue #7213: Support generic sliding window aggregations

2022-08-17 Thread GitBox
amrishlal commented on issue #7213: URL: https://github.com/apache/pinot/issues/7213#issuecomment-1218443261 @Jackie-Jiang only the merge will be done on Broker side, the computation of window functions is on the server side. I think @siddharthteotia is going to be setting up time with you

[pinot] branch add-table-auth-cleanup updated (e3bd48115d -> 379766a080)

2022-08-17 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch add-table-auth-cleanup in repository https://gitbox.apache.org/repos/asf/pinot.git discard e3bd48115d cleanup discard 3613589df5 linter fix discard a70cac6703 add-table refactor for clean aut

[GitHub] [pinot] joshigaurava commented on a diff in pull request #9161: Show most recent scheduling errors

2022-08-17 Thread GitBox
joshigaurava commented on code in PR #9161: URL: https://github.com/apache/pinot/pull/9161#discussion_r948338666 ## pinot-controller/src/main/resources/app/pages/TaskQueueTable.tsx: ## @@ -197,7 +202,9 @@ const TaskQueueTable = (props) => { headerTitle="Scheduling

[GitHub] [pinot] navina commented on a diff in pull request #9205: Max number of multi values should be set in DefaultNullValueProvider

2022-08-17 Thread GitBox
navina commented on code in PR #9205: URL: https://github.com/apache/pinot/pull/9205#discussion_r948321018 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/readers/constant/ConstantMVForwardIndexReader.java: ## @@ -45,6 +45,9 @@ public DataType g

[GitHub] [pinot] navina commented on issue #8819: Updating schema with new MV column causes index failures and stops consumption

2022-08-17 Thread GitBox
navina commented on issue #8819: URL: https://github.com/apache/pinot/issues/8819#issuecomment-1218392411 This issue can be reproduced by adding the following test query in `BaseClusterIntegrationTestSet#testReload` : `SELECT SUMMV(NewIntMVDimension) FROM " + rawTableName` The pro

[GitHub] [pinot] codecov-commenter commented on pull request #9235: Bug fix: Nested arrays and map not handled correctly for complex types

2022-08-17 Thread GitBox
codecov-commenter commented on PR #9235: URL: https://github.com/apache/pinot/pull/9235#issuecomment-1218386047 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9235?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] tanmesh commented on pull request #9184: [WIP] Add support for EXTRACT syntax and converts it to appropriate Pinot expression

2022-08-17 Thread GitBox
tanmesh commented on PR #9184: URL: https://github.com/apache/pinot/pull/9184#issuecomment-1218380979 Hey @Jackie-Jiang , I have completed part 1 of the task. For Part2: Can you please help me point to where ‘DateTimeTransformFunction’ (or other query time transformers defined [h

[GitHub] [pinot] Jackie-Jiang commented on a diff in pull request #9191: Support using query options to choose v2 engine

2022-08-17 Thread GitBox
Jackie-Jiang commented on code in PR #9191: URL: https://github.com/apache/pinot/pull/9191#discussion_r948298984 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -168,6 +167,13 @@ private String getDefaultBrokerId() {

[GitHub] [pinot] KKcorps commented on a diff in pull request #9220: Add Support for Schema Registry in Protobuf Decoder

2022-08-17 Thread GitBox
KKcorps commented on code in PR #9220: URL: https://github.com/apache/pinot/pull/9220#discussion_r948291083 ## pinot-plugins/pinot-input-format/pinot-protobuf/src/main/java/org/apache/pinot/plugin/inputformat/protobuf/KafkaConfluentSchemaRegistryProtoBufMessageDecoder.java: ## @

[GitHub] [pinot] KKcorps commented on a diff in pull request #9220: Add Support for Schema Registry in Protobuf Decoder

2022-08-17 Thread GitBox
KKcorps commented on code in PR #9220: URL: https://github.com/apache/pinot/pull/9220#discussion_r948290553 ## pinot-plugins/pinot-input-format/pinot-protobuf/src/main/java/org/apache/pinot/plugin/inputformat/protobuf/KafkaConfluentSchemaRegistryProtoBufMessageDecoder.java: ## @

[GitHub] [pinot] Jackie-Jiang merged pull request #9171: support to show running queries and cancel query by id

2022-08-17 Thread GitBox
Jackie-Jiang merged PR #9171: URL: https://github.com/apache/pinot/pull/9171 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.a

[GitHub] [pinot] Jackie-Jiang commented on pull request #9171: support to show running queries and cancel query by id

2022-08-17 Thread GitBox
Jackie-Jiang commented on PR #9171: URL: https://github.com/apache/pinot/pull/9171#issuecomment-1218358454 Let's add this new feature into the documentation -- 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

[pinot] branch master updated: support to show running queries and cancel query by id (#9171)

2022-08-17 Thread jackie
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/pinot.git The following commit(s) were added to refs/heads/master by this push: new 3a655d2d0e support to show running queries and ca

[pinot] branch add-table-auth-cleanup updated (3613589df5 -> e3bd48115d)

2022-08-17 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch add-table-auth-cleanup in repository https://gitbox.apache.org/repos/asf/pinot.git from 3613589df5 linter fix add e3bd48115d cleanup No new revisions were added by this update. Summar

[GitHub] [pinot] KKcorps commented on a diff in pull request #9220: Add Support for Schema Registry in Protobuf Decoder

2022-08-17 Thread GitBox
KKcorps commented on code in PR #9220: URL: https://github.com/apache/pinot/pull/9220#discussion_r948284879 ## pinot-plugins/pinot-input-format/pinot-protobuf/src/main/java/org/apache/pinot/plugin/inputformat/protobuf/KafkaConfluentSchemaRegistryProtoBufMessageDecoder.java: ## @

[pinot] branch master updated: Refactor stream quickstart (#9227)

2022-08-17 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/pinot.git The following commit(s) were added to refs/heads/master by this push: new d41ec0bfd4 Refactor stream quickstart (#9227) d4

[GitHub] [pinot] xiangfu0 merged pull request #9227: [Refactor] Refactor stream quickstart

2022-08-17 Thread GitBox
xiangfu0 merged PR #9227: URL: https://github.com/apache/pinot/pull/9227 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.apach

[GitHub] [pinot] walterddr closed pull request #9234: green stuff to commit

2022-08-17 Thread GitBox
walterddr closed pull request #9234: green stuff to commit URL: https://github.com/apache/pinot/pull/9234 -- 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. To unsubscribe, e-mail

[GitHub] [pinot] KKcorps opened a new pull request, #9235: Bug fix: Nested arrays and map not handled correctly for complex types

2022-08-17 Thread GitBox
KKcorps opened a new pull request, #9235: URL: https://github.com/apache/pinot/pull/9235 When an input is encountered that has the following tree structure [array -> map -> array -> map] , the complex type transformer doesn't unnest the values properly. This is because we append the child n

[GitHub] [pinot] Jackie-Jiang commented on pull request #9193: Adding kafka offset support for period and timestamp

2022-08-17 Thread GitBox
Jackie-Jiang commented on PR #9193: URL: https://github.com/apache/pinot/pull/9193#issuecomment-1218345654 @ravishankar15 Thanks for contributing the feature! Can you help also update the Pinot documentation to include the new config options? The documentation is backed by this repo: https:

[GitHub] [pinot] Jackie-Jiang closed issue #7250: Configure Kafka consumption from a specific timestamp

2022-08-17 Thread GitBox
Jackie-Jiang closed issue #7250: Configure Kafka consumption from a specific timestamp URL: https://github.com/apache/pinot/issues/7250 -- 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 c

[pinot] branch master updated (fe90978939 -> c1bdd10e6b)

2022-08-17 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/pinot.git from fe90978939 Bump zstd-jni from 1.4.9-5 to 1.5.2-3 (#9232) add c1bdd10e6b Adding kafka offset support for period and

[GitHub] [pinot] Jackie-Jiang merged pull request #9193: Adding kafka offset support for period and timestamp

2022-08-17 Thread GitBox
Jackie-Jiang merged PR #9193: URL: https://github.com/apache/pinot/pull/9193 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.a

[GitHub] [pinot] walterddr opened a new pull request, #9234: green stuff to commit

2022-08-17 Thread GitBox
walterddr opened a new pull request, #9234: URL: https://github.com/apache/pinot/pull/9234 Instructions: 1. The PR has to be tagged with at least one of the following labels (*): 1. `feature` 2. `bugfix` 3. `performance` 4. `ui` 5. `backward-incompat` 6.

[GitHub] [pinot] siddharthteotia commented on a diff in pull request #9191: Support using query options to choose v2 engine

2022-08-17 Thread GitBox
siddharthteotia commented on code in PR #9191: URL: https://github.com/apache/pinot/pull/9191#discussion_r948240959 ## pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/BaseBrokerStarter.java: ## @@ -256,7 +257,7 @@ public void start() // Create Broker request

[GitHub] [pinot] siddharthteotia commented on pull request #9225: Allow explicit partition for real-time segment assignment

2022-08-17 Thread GitBox
siddharthteotia commented on PR #9225: URL: https://github.com/apache/pinot/pull/9225#issuecomment-1218300473 Reviewing this today -- 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 com

[GitHub] [pinot] codecov-commenter commented on pull request #9233: [multistage] fixing count not resolving as SUM0 in intermediate stage

2022-08-17 Thread GitBox
codecov-commenter commented on PR #9233: URL: https://github.com/apache/pinot/pull/9233#issuecomment-1218275876 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9233?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] Jackie-Jiang closed pull request #9231: Bump scala-reflect from 2.12.15 to 2.13.8

2022-08-17 Thread GitBox
Jackie-Jiang closed pull request #9231: Bump scala-reflect from 2.12.15 to 2.13.8 URL: https://github.com/apache/pinot/pull/9231 -- 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.

[GitHub] [pinot] dependabot[bot] commented on pull request #9231: Bump scala-reflect from 2.12.15 to 2.13.8

2022-08-17 Thread GitBox
dependabot[bot] commented on PR #9231: URL: https://github.com/apache/pinot/pull/9231#issuecomment-1218272362 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let m

[GitHub] [pinot] Jackie-Jiang commented on pull request #9231: Bump scala-reflect from 2.12.15 to 2.13.8

2022-08-17 Thread GitBox
Jackie-Jiang commented on PR #9231: URL: https://github.com/apache/pinot/pull/9231#issuecomment-1218272331 Close this one because this requires higher scalar library version -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[pinot] branch master updated (807f84392c -> fe90978939)

2022-08-17 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/pinot.git from 807f84392c fix empty data block not returning schema (#9222) add fe90978939 Bump zstd-jni from 1.4.9-5 to 1.5.2-3

[GitHub] [pinot] Jackie-Jiang merged pull request #9232: Bump zstd-jni from 1.4.9-5 to 1.5.2-3

2022-08-17 Thread GitBox
Jackie-Jiang merged PR #9232: URL: https://github.com/apache/pinot/pull/9232 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.a

[GitHub] [pinot] KKcorps commented on a diff in pull request #9220: Add Support for Schema Registry in Protobuf Decoder

2022-08-17 Thread GitBox
KKcorps commented on code in PR #9220: URL: https://github.com/apache/pinot/pull/9220#discussion_r948149651 ## pinot-plugins/pinot-input-format/pinot-protobuf/src/test/java/org/apache/pinot/plugin/inputformat/protobuf/kafka/schemaregistry/SchemaRegistryStarter.java: ## @@ -0,0 +

[GitHub] [pinot] walterddr commented on issue #9189: [multistage] Query Doesn't Return Schema When Result Set is Empty

2022-08-17 Thread GitBox
walterddr commented on issue #9189: URL: https://github.com/apache/pinot/issues/9189#issuecomment-1218216765 close with #9222 -- 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.

[GitHub] [pinot] walterddr closed issue #9189: [multistage] Query Doesn't Return Schema When Result Set is Empty

2022-08-17 Thread GitBox
walterddr closed issue #9189: [multistage] Query Doesn't Return Schema When Result Set is Empty URL: https://github.com/apache/pinot/issues/9189 -- 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 s

[GitHub] [pinot] walterddr opened a new pull request, #9233: fixing count not resolving as SUM0 in intermediate stage

2022-08-17 Thread GitBox
walterddr opened a new pull request, #9233: URL: https://github.com/apache/pinot/pull/9233 this fixes: #9190 -- 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. To unsubscribe,

[pinot] branch master updated: fix empty data block not returning schema (#9222)

2022-08-17 Thread siddteotia
This is an automated email from the ASF dual-hosted git repository. siddteotia pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git The following commit(s) were added to refs/heads/master by this push: new 807f84392c fix empty data block not returning

[GitHub] [pinot] siddharthteotia merged pull request #9222: [multistage] fix empty data block not returning schema

2022-08-17 Thread GitBox
siddharthteotia merged PR #9222: URL: https://github.com/apache/pinot/pull/9222 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pino

[GitHub] [pinot] walterddr commented on a diff in pull request #9220: Add Support for Schema Registry in Protobuf Decoder

2022-08-17 Thread GitBox
walterddr commented on code in PR #9220: URL: https://github.com/apache/pinot/pull/9220#discussion_r948034126 ## pinot-plugins/pinot-input-format/pinot-protobuf/src/test/java/org/apache/pinot/plugin/inputformat/protobuf/kafka/schemaregistry/SchemaRegistryStarter.java: ## @@ -0,0

[GitHub] [pinot] walterddr commented on a diff in pull request #9220: Add Support for Schema Registry in Protobuf Decoder

2022-08-17 Thread GitBox
walterddr commented on code in PR #9220: URL: https://github.com/apache/pinot/pull/9220#discussion_r948047491 ## pinot-plugins/pinot-input-format/pinot-protobuf/src/main/java/org/apache/pinot/plugin/inputformat/protobuf/KafkaConfluentSchemaRegistryProtoBufMessageDecoder.java: ##

[GitHub] [pinot] walterddr commented on a diff in pull request #9220: Add Support for Schema Registry in Protobuf Decoder

2022-08-17 Thread GitBox
walterddr commented on code in PR #9220: URL: https://github.com/apache/pinot/pull/9220#discussion_r948013279 ## pinot-plugins/pinot-input-format/pinot-protobuf/pom.xml: ## @@ -35,9 +35,16 @@ https://pinot.apache.org/ ${basedir}/../../.. -3.11.4 +2.8.1 Revie

[GitHub] [pinot] codecov-commenter commented on pull request #9232: Bump zstd-jni from 1.4.9-5 to 1.5.2-3

2022-08-17 Thread GitBox
codecov-commenter commented on PR #9232: URL: https://github.com/apache/pinot/pull/9232#issuecomment-1217917507 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9232?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] dependabot[bot] opened a new pull request, #9232: Bump zstd-jni from 1.4.9-5 to 1.5.2-3

2022-08-17 Thread GitBox
dependabot[bot] opened a new pull request, #9232: URL: https://github.com/apache/pinot/pull/9232 Bumps [zstd-jni](https://github.com/luben/zstd-jni) from 1.4.9-5 to 1.5.2-3. Commits https://github.com/luben/zstd-jni/commit/c983ae3e086b63a40e1bb430cb2ebf95ecc52c71";>c983ae3 Adju

[pinot] branch dependabot/maven/com.github.luben-zstd-jni-1.5.2-3 created (now e144f49a2b)

2022-08-17 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/com.github.luben-zstd-jni-1.5.2-3 in repository https://gitbox.apache.org/repos/asf/pinot.git at e144f49a2b Bump zstd-jni from 1.4.9-5 to 1.5.2-3 No new revisions wer

[GitHub] [pinot] dependabot[bot] opened a new pull request, #9231: Bump scala-reflect from 2.12.15 to 2.13.8

2022-08-17 Thread GitBox
dependabot[bot] opened a new pull request, #9231: URL: https://github.com/apache/pinot/pull/9231 Bumps [scala-reflect](https://github.com/scala/scala) from 2.12.15 to 2.13.8. Release notes Sourced from https://github.com/scala/scala/releases";>scala-reflect's releases. Scala

[pinot] branch dependabot/maven/org.scala-lang-scala-reflect-2.13.8 created (now 5132c9dca4)

2022-08-17 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/org.scala-lang-scala-reflect-2.13.8 in repository https://gitbox.apache.org/repos/asf/pinot.git at 5132c9dca4 Bump scala-reflect from 2.12.15 to 2.13.8 No new revisio

[GitHub] [pinot] codecov-commenter commented on pull request #9230: Add support for Auth in controller requests in java query client

2022-08-17 Thread GitBox
codecov-commenter commented on PR #9230: URL: https://github.com/apache/pinot/pull/9230#issuecomment-1217855504 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9230?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[pinot] branch master updated: Add close method to upsert interfaces (#9212)

2022-08-17 Thread kharekartik
This is an automated email from the ASF dual-hosted git repository. kharekartik pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git The following commit(s) were added to refs/heads/master by this push: new 5c703695d2 Add close method to upsert interf

[GitHub] [pinot] KKcorps merged pull request #9212: Add close method to upsert interfaces

2022-08-17 Thread GitBox
KKcorps merged PR #9212: URL: https://github.com/apache/pinot/pull/9212 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache

[GitHub] [pinot] KKcorps opened a new pull request, #9230: Add support for Auth in controller requests in java query client

2022-08-17 Thread GitBox
KKcorps opened a new pull request, #9230: URL: https://github.com/apache/pinot/pull/9230 The PR adds the support for auth and https when using `fromController` of java client. The PR also does refactor some interfaces while maintaining backwards compatibility * The `JsonAsyncHttp

[pinot] branch add-table-auth-cleanup updated (81812ea006 -> 3613589df5)

2022-08-17 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch add-table-auth-cleanup in repository https://gitbox.apache.org/repos/asf/pinot.git omit 81812ea006 linter fix omit 1bf9931a53 add-table refactor for clean auth add 2aa2165367 fix UT

[GitHub] [pinot] xiangfu0 merged pull request #9229: Adding pinot base docker image

2022-08-17 Thread GitBox
xiangfu0 merged PR #9229: URL: https://github.com/apache/pinot/pull/9229 -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.apach

[pinot] branch master updated: Adding pinot base docker image to reduce build time (#9229)

2022-08-17 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/pinot.git The following commit(s) were added to refs/heads/master by this push: new 661a7d6a5d Adding pinot base docker image to red