[GitHub] [incubator-pinot] kishoreg commented on issue #5944: Move to natural pom versions

kishoreg commented on issue #5944: URL: https://github.com/apache/incubator-pinot/issues/5944#issuecomment-683245902 +100 for this. @jackjlli can you please explain why this sha was added? This is an automated message from t

[GitHub] [incubator-pinot] adriancole opened a new issue #5944: Move to natural pom versions

adriancole opened a new issue #5944: URL: https://github.com/apache/incubator-pinot/issues/5944 Currently, `${revision}${sha1}` is coded into maven pom files, rendering tools such as jitpack helpless. It would be nice to have normal versions in these files so we have the same chances to us

[GitHub] [incubator-pinot] mr-agrwal closed issue #5893: Support for segmentPartitionedDistinctCount in Star Tree Index Pre aggregation Functions.

mr-agrwal closed issue #5893: URL: https://github.com/apache/incubator-pinot/issues/5893 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 g

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5910: 5804: Prints file name of invalid json file when parsing

Jackie-Jiang commented on a change in pull request #5910: URL: https://github.com/apache/incubator-pinot/pull/5910#discussion_r479596007 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/CreateSegmentCommand.java ## @@ -252,7 +252,8 @@ public boolean

[GitHub] [incubator-pinot] harleyjj opened a new pull request #5943: Protect against Infinity and NaN values in RCA time series response

harleyjj opened a new pull request #5943: URL: https://github.com/apache/incubator-pinot/pull/5943 ## Description * Strips non-finite values from time series values before passing them to c3.js. * Fixes one annoying console error from chart div being renamed in new version. -

[GitHub] [incubator-pinot] npawar commented on issue #5942: Better Table config validation

npawar commented on issue #5942: URL: https://github.com/apache/incubator-pinot/issues/5942#issuecomment-683191930 One more: https://github.com/apache/incubator-pinot/issues/5730 This is an automated message from the Apache

[incubator-pinot] branch master updated: [TE] polish the rca template in email (#5939)

This is an automated email from the ASF dual-hosted git repository. akshayrai09 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 c3446ba [TE] polish the rca templ

[GitHub] [incubator-pinot] codecov-commenter commented on pull request #5910: 5804: Prints file name of invalid json file when parsing

codecov-commenter commented on pull request #5910: URL: https://github.com/apache/incubator-pinot/pull/5910#issuecomment-683185554 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5910?src=pr&el=h1) Report > Merging [#5910](https://codecov.io/gh/apache/incubator-pinot/pull

[GitHub] [incubator-pinot] akshayrai merged pull request #5939: [TE] polish the rca template in email

akshayrai merged pull request #5939: URL: https://github.com/apache/incubator-pinot/pull/5939 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] icefury71 opened a new issue #5942: Better Table config validation

icefury71 opened a new issue #5942: URL: https://github.com/apache/incubator-pinot/issues/5942 As mentioned in #5915 and #5899 we need a better way to validate table config. This includes following checks (not exhaustive): * Validate column names : should match the ones in schema * Ti

[GitHub] [incubator-pinot] icefury71 edited a comment on issue #4484: Pinot query timeout due to the broker waiting for a single non-responsive server

icefury71 edited a comment on issue #4484: URL: https://github.com/apache/incubator-pinot/issues/4484#issuecomment-683171767 Thanks for the input @mcvsubbu . I've started a design here: https://docs.google.com/document/d/1PP_RaDuS7KGeF9RnAcRFJRCA8aCVxxVLTJn-c3hg9qQ/edit Plea

[GitHub] [incubator-pinot] icefury71 commented on issue #4484: Pinot query timeout due to the broker waiting for a single non-responsive server

icefury71 commented on issue #4484: URL: https://github.com/apache/incubator-pinot/issues/4484#issuecomment-683171767 Thanks for the input @mcvsubbu . I've started a design here (based on Kishore's recommendation): https://docs.pinot.apache.org/developers/design-documents/server-he

[GitHub] [incubator-pinot] buchireddy closed issue #5871: Expose response stats in Pinot java client's BrokerReponse

buchireddy closed issue #5871: URL: https://github.com/apache/incubator-pinot/issues/5871 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] buchireddy commented on issue #5871: Expose response stats in Pinot java client's BrokerReponse

buchireddy commented on issue #5871: URL: https://github.com/apache/incubator-pinot/issues/5871#issuecomment-683166530 Yes. The PR is merged. This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [incubator-pinot] jackjlli commented on pull request #5941: Bump up swagger-ui version and update related code

jackjlli commented on pull request #5941: URL: https://github.com/apache/incubator-pinot/pull/5941#issuecomment-683150005 @kishoreg yeah it's because the locations for those front-end related files like css, js files have changed in the newer swagger-ui version. --

[incubator-pinot] branch master updated (701ffcb -> 347eff1)

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 701ffcb Revert the commit 41dc432e775aeeb841f57bd25525832950698739 (#5938) add 347eff1 Fixing the i

[GitHub] [incubator-pinot] fx19880617 merged pull request #5940: Fixing the issue that transform UDFs are parsed as function name 'OTHER', not the real function names

fx19880617 merged pull request #5940: URL: https://github.com/apache/incubator-pinot/pull/5940 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 #5940: Fixing the issue that transform UDFs are parsed as function name 'OTHER', not the real function names

fx19880617 commented on pull request #5940: URL: https://github.com/apache/incubator-pinot/pull/5940#issuecomment-683145518 > LGTM. Wonder how Calcite differentiate `OTHER_FUNCTION` and `OTHER` :-P OTHER_FUNCTION is the functions Calcite recognize but not special functions like `COUN

[GitHub] [incubator-pinot] kishoreg commented on pull request #5941: Bump up swagger-ui version and update related code

kishoreg commented on pull request #5941: URL: https://github.com/apache/incubator-pinot/pull/5941#issuecomment-683144893 > what was the bug in the previous PR? Never mind looked at the diff and figured it out. Thanks for fixing it. --

[GitHub] [incubator-pinot] fx19880617 edited a comment on issue #5704: Support toWeek/toMonth/toYear in datetimeConvertUdf

fx19880617 edited a comment on issue #5704: URL: https://github.com/apache/incubator-pinot/issues/5704#issuecomment-683144571 Fixing an issue https://github.com/apache/incubator-pinot/pull/5940 which prevents using functions introduced in #5438 --

[GitHub] [incubator-pinot] fx19880617 commented on issue #5704: Support toWeek/toMonth/toYear in datetimeConvertUdf

fx19880617 commented on issue #5704: URL: https://github.com/apache/incubator-pinot/issues/5704#issuecomment-683144571 Fixing an issue which prevents using functions introduced in #5438 This is an automated message from the

[GitHub] [incubator-pinot] kishoreg commented on pull request #5941: Bump up swagger-ui version and update related code

kishoreg commented on pull request #5941: URL: https://github.com/apache/incubator-pinot/pull/5941#issuecomment-683144586 what was the bug in the previous PR? This is an automated message from the Apache Git Service. To respo

svn commit: r41178 - /dev/incubator/pinot/apache-pinot-incubating-0.5.0-rc1/

Author: tingchen Date: Fri Aug 28 20:54:26 2020 New Revision: 41178 Log: Update apache-pinot-incubating-0.5.0-rc1 Added: dev/incubator/pinot/apache-pinot-incubating-0.5.0-rc1/ dev/incubator/pinot/apache-pinot-incubating-0.5.0-rc1/apache-pinot-incubating-0.5.0-bin.tar.gz (with props)

[GitHub] [incubator-pinot] jackjlli opened a new pull request #5941: Bump up swagger-ui version and update related code

jackjlli opened a new pull request #5941: URL: https://github.com/apache/incubator-pinot/pull/5941 ## Description This PR bumps up swagger-ui version and updates related code. vulnerability: Swagger-ui before 3.18.0 is vulnerable to Reverse Tabnabbing. Setting target="_blank" on a

[incubator-pinot] branch bump-up-swagger-ui-version created (now 2b736b4)

This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch bump-up-swagger-ui-version in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 2b736b4 Bump up swagger-ui version and update related code This branch includes the foll

[incubator-pinot] 01/01: Bump up swagger-ui version and update related code

This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch bump-up-swagger-ui-version in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 2b736b417a0910552d18ed8a2755405e49ba452f Author: Jack Li(Analytics Engineering) AuthorDate: F

[GitHub] [incubator-pinot] guruguha commented on a change in pull request #5718: Feature/#5390 segment indexing reload status api

guruguha commented on a change in pull request #5718: URL: https://github.com/apache/incubator-pinot/pull/5718#discussion_r479518772 ## File path: pinot-common/src/main/java/org/apache/pinot/common/restlet/resources/SegmentStatus.java ## @@ -0,0 +1,53 @@ +/** + * Licensed to t

[GitHub] [incubator-pinot] mr-agrwal commented on issue #5937: Swagger UI not working.

mr-agrwal commented on issue #5937: URL: https://github.com/apache/incubator-pinot/issues/5937#issuecomment-683129155 @jackjlli @kishoreg The fix works :). Thanks!! This is an automated message from the Apache Git Service. T

[GitHub] [incubator-pinot] mr-agrwal closed issue #5937: Swagger UI not working.

mr-agrwal closed issue #5937: URL: https://github.com/apache/incubator-pinot/issues/5937 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 g

[GitHub] [incubator-pinot] fx19880617 opened a new pull request #5940: Fixing the issue that transform UDFs are parsed as function name 'OTHER', not the real function names

fx19880617 opened a new pull request #5940: URL: https://github.com/apache/incubator-pinot/pull/5940 ## Description This PR fixes the query like: ``` select Hour(event_time), group_city, sum(rsvp_count) from meetupRsvp group by Hour(event_time), group_city order by sum(rsvp_count)

[incubator-pinot] branch fixing_transform_udf_parsing_in_calcite_sql created (now 9a0f765)

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch fixing_transform_udf_parsing_in_calcite_sql in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 9a0f765 Fixing the issue that transform UDFs are parsed as function

[incubator-pinot] 01/01: Fixing the issue that transform UDFs are parsed as function name 'OTHER', not the real function names

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch fixing_transform_udf_parsing_in_calcite_sql in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 9a0f765ab6ea0e0f310718ba6434da5061ce55a3 Author: Xiang Fu AuthorDate: Fri

[incubator-pinot] annotated tag release-0.5.0-rc1 updated (3c40ea2 -> 3834863)

This is an automated email from the ASF dual-hosted git repository. tingchen pushed a change to annotated tag release-0.5.0-rc1 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. *** WARNING: tag release-0.5.0-rc1 was modified! *** from 3c40ea2 (commit) to 3834863

[incubator-pinot] branch release-0.5.0-rc updated: [maven-release-plugin] prepare release release-0.5.0-rc1

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 3c40ea2 [maven-r

[incubator-pinot] branch release-0.5.0-rc updated: [maven-release-plugin] prepare for next development iteration

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 af40e90 [maven-r

[GitHub] [incubator-pinot] fx19880617 commented on issue #5877: Standalone ingestion using SegmentUriPush to S3 does not work

fx19880617 commented on issue #5877: URL: https://github.com/apache/incubator-pinot/issues/5877#issuecomment-683071844 Here is also a doc for using s3 as deep store: https://docs.pinot.apache.org/users/tutorials/use-s3-as-deep-store-for-pinot -

[GitHub] [incubator-pinot] fx19880617 closed issue #5877: Standalone ingestion using SegmentUriPush to S3 does not work

fx19880617 closed issue #5877: URL: https://github.com/apache/incubator-pinot/issues/5877 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] kishoreg commented on issue #5877: Standalone ingestion using SegmentUriPush to S3 does not work

kishoreg commented on issue #5877: URL: https://github.com/apache/incubator-pinot/issues/5877#issuecomment-683041881 @fx19880617 can we close this This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [incubator-pinot] kishoreg commented on issue #5871: Expose response stats in Pinot java client's BrokerReponse

kishoreg commented on issue #5871: URL: https://github.com/apache/incubator-pinot/issues/5871#issuecomment-683042498 @buchireddy can we close this? This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5926: Introduce IdSet and add IdSetAggregationFunction

Jackie-Jiang commented on a change in pull request #5926: URL: https://github.com/apache/incubator-pinot/pull/5926#discussion_r479457266 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/IdSetAggregationFunction.java ## @@ -0,0 +1,339 @@ +

[incubator-pinot] branch release-0.5.0-rc updated: Fix CSV and JSON converter on BYTES column (#5931)

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 b0d8da0 Fix CSV

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5934: Segment processing framework

npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r479439110 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/framework/SegmentMapper.java ## @@ -0,0 +1,148 @@ +/** + * License

[incubator-pinot] branch master updated (d47e751 -> 701ffcb)

This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from d47e751 Add an explicit license check exclusion for github related templates. (#5936) add 701ffcb Reve

[GitHub] [incubator-pinot] jackjlli merged pull request #5938: Revert the commit 41dc432e775aeeb841f57bd25525832950698739

jackjlli merged pull request #5938: URL: https://github.com/apache/incubator-pinot/pull/5938 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] npawar commented on a change in pull request #5934: Segment processing framework

npawar commented on a change in pull request #5934: URL: https://github.com/apache/incubator-pinot/pull/5934#discussion_r479437446 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/processing/transformer/TransformFunctionRecordTransformer.java ## @@ -0,0 +1,

[GitHub] [incubator-pinot] codecov-commenter commented on pull request #5938: Revert the commit 41dc432e775aeeb841f57bd25525832950698739

codecov-commenter commented on pull request #5938: URL: https://github.com/apache/incubator-pinot/pull/5938#issuecomment-682953613 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5938?src=pr&el=h1) Report > Merging [#5938](https://codecov.io/gh/apache/incubator-pinot/pull

[GitHub] [incubator-pinot] akshayrai opened a new pull request #5939: [TE] polish the rca template in email

akshayrai opened a new pull request #5939: URL: https://github.com/apache/incubator-pinot/pull/5939 Changes: * polished the rca table with line separators and text wrapping * fixed the rca url in the template * removed deprecated fields from configuration -

[incubator-pinot] branch release-0.5.0-rc updated: Add an explicit license check exclusion for github related templates. (#5936)

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 f9814c9 Add an e

[incubator-pinot] branch revert-swagger-version-bump updated (b942df9 -> 915f23f)

This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch revert-swagger-version-bump in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard b942df9 Revert the commit 41dc432e775aeeb841f57bd25525832950698739 add 915f23f Rev

[GitHub] [incubator-pinot] jackjlli opened a new pull request #5938: Revert the commit 41dc432e775aeeb841f57bd25525832950698739

jackjlli opened a new pull request #5938: URL: https://github.com/apache/incubator-pinot/pull/5938 ## Description This is to fastly unblock https://github.com/apache/incubator-pinot/issues/5937. This is an automated messa

[incubator-pinot] 01/01: Revert the commit 41dc432e775aeeb841f57bd25525832950698739

This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch revert-swagger-version-bump in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit b942df9c7175d92a45d97da1191b85cd49dc7492 Author: Jack Li(Analytics Engineering) AuthorDate:

[incubator-pinot] branch revert-swagger-version-bump created (now b942df9)

This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch revert-swagger-version-bump in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at b942df9 Revert the commit 41dc432e775aeeb841f57bd25525832950698739 This branch includes

[GitHub] [incubator-pinot] chenboat merged pull request #5936: Add an explicit RAT license check exclusion for github related templates.

chenboat merged pull request #5936: URL: https://github.com/apache/incubator-pinot/pull/5936 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 (6b78dcc -> d47e751)

This is an automated email from the ASF dual-hosted git repository. tingchen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 6b78dcc Add max qps bucket count (#5922) add d47e751 Add an explicit license check exclusion for gi

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5911: Add compatibility verifier scripts (#4854)

mcvsubbu commented on a change in pull request #5911: URL: https://github.com/apache/incubator-pinot/pull/5911#discussion_r479399437 ## File path: compatibility-verifier/compCheck.sh ## @@ -0,0 +1,210 @@ +#!/bin/bash +# +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [incubator-pinot] kishoreg commented on issue #5904: Support for a fixed precision Decimal column type

kishoreg commented on issue #5904: URL: https://github.com/apache/incubator-pinot/issues/5904#issuecomment-682745569 What operators will you need on top of DECIMAL. is sum the only one? This is an automated message from the

[GitHub] [incubator-pinot] kishoreg commented on issue #5928: Add connector for Pravega

kishoreg commented on issue #5928: URL: https://github.com/apache/incubator-pinot/issues/5928#issuecomment-682738316 @mcvsubbu @npawar @KKcorps thoughts? This is an automated message from the Apache Git Service. To respond t

[GitHub] [incubator-pinot] kishoreg commented on issue #5937: Swagger UI not working.

kishoreg commented on issue #5937: URL: https://github.com/apache/incubator-pinot/issues/5937#issuecomment-682728526 @jackjlli looks like you change broker swagger git checkout 87b5b77264c1f05ac6073ff77e1728667dce1e1b works git checkout e9ad1e017bf06ac579014357462c86861a31cf7d

[GitHub] [incubator-pinot] mr-agrwal opened a new issue #5937: Swagger UI not working.

mr-agrwal opened a new issue #5937: URL: https://github.com/apache/incubator-pinot/issues/5937 I am not able to access the Swagger UI. All APIs for `swaggerui-dist/lib/*` and `swaggerui-dist/css/*` are failing with 404. https://user-images.githubusercontent.com/24796847/91584939-0143

[GitHub] [incubator-pinot] saileshnankani edited a comment on pull request #5911: Add compatibility verifier scripts (#4854)

saileshnankani edited a comment on pull request #5911: URL: https://github.com/apache/incubator-pinot/pull/5911#issuecomment-682280899 I have implemented most of the changes you've requested. There's an additional change I made: - Since providing a working directory is optional, I ad

[GitHub] [incubator-pinot] saileshnankani commented on a change in pull request #5911: Add compatibility verifier scripts (#4854)

saileshnankani commented on a change in pull request #5911: URL: https://github.com/apache/incubator-pinot/pull/5911#discussion_r479319009 ## File path: compatibility-verifier/comp-verifier.sh ## @@ -0,0 +1,145 @@ +#!/bin/bash +# +# Licensed to the Apache Software Foundation (A

[GitHub] [incubator-pinot] jackjlli merged pull request #5922: Add max qps bucket count

jackjlli merged pull request #5922: URL: https://github.com/apache/incubator-pinot/pull/5922 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: Add max qps bucket count (#5922)

This is an automated email from the ASF dual-hosted git repository. jlli 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 6b78dcc Add max qps bucket count (#5922)