[GitHub] [incubator-pinot] pradeepgv42 commented on pull request #5755: Pradeep/s3 credential chain fix

pradeepgv42 commented on pull request #5755: URL: https://github.com/apache/incubator-pinot/pull/5755#issuecomment-663818187 So, initially I just fixed headObjectRequest (https://github.com/apache/incubator-pinot/pull/5755/commits/e359dc109a786e55637ddc8dc8fc644e7d87760f) `testDele

[GitHub] [incubator-pinot] shahsank3t opened a new pull request #5757: Cluster Manger UI: adding instance details page showing instance config and tables in that particular instance

shahsank3t opened a new pull request #5757: URL: https://github.com/apache/incubator-pinot/pull/5757 ## Description This PR is an incremental PR for #5649 Added Instance Details pages showing Instance configuration & Tables that particular instance has. **Broker:** ![image](ht

[incubator-pinot] branch fix-multi-value-column-from-avro updated (be4be9a -> 297d5c5)

This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch fix-multi-value-column-from-avro in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard be4be9a Address PR comments add 297d5c5 Address PR comments This update adde

[GitHub] [incubator-pinot] jackjlli opened a new pull request #5756: Transform and extract map values to array

jackjlli opened a new pull request #5756: URL: https://github.com/apache/incubator-pinot/pull/5756 ## Description This PR fixes the issue introduced from this PR: #5238 The returned type from the GenericRow could be map, whereas there is no logic to handle map type in DataTypeTran

[incubator-pinot] 01/01: Transform and extract map values to array

This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch override-methods-in-object-array in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 77c3158e5d511737105c8bb124201a95ae7db7df Author: Jack Li(Analytics Engineering) AuthorD

[incubator-pinot] branch override-methods-in-object-array created (now 77c3158)

This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch override-methods-in-object-array in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 77c3158 Transform and extract map values to array This branch includes the followi

[GitHub] [incubator-pinot] KKcorps commented on pull request #5755: Pradeep/s3 credential chain fix

KKcorps commented on pull request #5755: URL: https://github.com/apache/incubator-pinot/pull/5755#issuecomment-663813201 Also, can you mentioned what's the bug in S3Mock? This is an automated message from the Apache Git Servi

[GitHub] [incubator-pinot] KKcorps edited a comment on pull request #5755: Pradeep/s3 credential chain fix

KKcorps edited a comment on pull request #5755: URL: https://github.com/apache/incubator-pinot/pull/5755#issuecomment-663813201 Also, can you mention what's the bug in S3Mock? This is an automated message from the Apache Git

[GitHub] [incubator-pinot] KKcorps commented on pull request #5755: Pradeep/s3 credential chain fix

KKcorps commented on pull request #5755: URL: https://github.com/apache/incubator-pinot/pull/5755#issuecomment-663813055 LGTM! The motivation behind headObjectRequest was to check the folders quickly since it just retrieves the metadata. It obviously fails when there is no folder with the

[incubator-pinot] branch fix-multi-value-column-from-avro updated (48a85f2 -> be4be9a)

This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch fix-multi-value-column-from-avro in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 48a85f2 Handle Map in DataTypeTransformer class add be4be9a Address PR commen

[incubator-pinot] branch master updated: Improvements to RealtimeProvisioningHelper command (#5737)

This is an automated email from the ASF dual-hosted git repository. mcvsubbu 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 540853d Improvements to RealtimeProv

[GitHub] [incubator-pinot] mcvsubbu merged pull request #5737: Improvements to RealtimeProvisioningHelper command

mcvsubbu merged pull request #5737: URL: https://github.com/apache/incubator-pinot/pull/5737 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: GROOVY transform function UDF for queries (#5748)

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 6911172 GROOVY transform function U

[GitHub] [incubator-pinot] npawar merged pull request #5748: GROOVY transform function UDF for queries

npawar merged pull request #5748: URL: https://github.com/apache/incubator-pinot/pull/5748 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] pradeepgv42 opened a new pull request #5755: Pradeep/s3 credential chain fix

pradeepgv42 opened a new pull request #5755: URL: https://github.com/apache/incubator-pinot/pull/5755 ## Description For S3PinotFs Use DefaultCredentialProvider which chains common ways to provider credentials For example: Credentials provider file and Instance profile based c

[GitHub] [incubator-pinot] codecov-commenter commented on pull request #5748: GROOVY transform function UDF for queries

codecov-commenter commented on pull request #5748: URL: https://github.com/apache/incubator-pinot/pull/5748#issuecomment-663788640 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5748?src=pr&el=h1) Report > :exclamation: No coverage uploaded for pull request base (`master

[incubator-pinot] branch fixing_case_of_record_readers updated (4ca53ef -> 0f19cde)

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch fixing_case_of_record_readers in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. omit 4ca53ef Fixing the case of Orc and Json record reader class name add 0f19cde

[GitHub] [incubator-pinot] fx19880617 opened a new pull request #5754: Fixing the case of Orc and Json record reader class name

fx19880617 opened a new pull request #5754: URL: https://github.com/apache/incubator-pinot/pull/5754 ## Description Fixing the class name to avoid any further confusion for the users. This is an automated message from the

[incubator-pinot] branch fixing_case_of_record_readers created (now 4ca53ef)

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch fixing_case_of_record_readers in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 4ca53ef Fixing the case of Orc and Json record reader class name This branch inclu

[incubator-pinot] 01/01: Fixing the case of Orc and Json record reader class name

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch fixing_case_of_record_readers in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 4ca53efbea2d0f56243e518ed20b3009b524ad4b Author: Xiang Fu AuthorDate: Fri Jul 24 18:05:

[GitHub] [incubator-pinot] mcvsubbu commented on pull request #5737: Improvements to RealtimeProvisioningHelper command

mcvsubbu commented on pull request #5737: URL: https://github.com/apache/incubator-pinot/pull/5737#issuecomment-663784663 @npawar please take another look, thanks This is an automated message from the Apache Git Service. To

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5737: Improvements to RealtimeProvisioningHelper command

mcvsubbu commented on a change in pull request #5737: URL: https://github.com/apache/incubator-pinot/pull/5737#discussion_r460340851 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/RealtimeProvisioningHelperCommand.java ## @@ -158,17 +173,36 @@ pub

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5737: Improvements to RealtimeProvisioningHelper command

mcvsubbu commented on a change in pull request #5737: URL: https://github.com/apache/incubator-pinot/pull/5737#discussion_r460339934 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/RealtimeProvisioningHelperCommand.java ## @@ -115,17 +116,17 @@ pub

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5737: Improvements to RealtimeProvisioningHelper command

mcvsubbu commented on a change in pull request #5737: URL: https://github.com/apache/incubator-pinot/pull/5737#discussion_r460339757 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/RealtimeProvisioningHelperCommand.java ## @@ -90,8 +91,8 @@ public

[GitHub] [incubator-pinot] harleyjj commented on a change in pull request #5752: [TE] frontend - harleyjj/rca - Phase 1 of custom baseline selector

harleyjj commented on a change in pull request #5752: URL: https://github.com/apache/incubator-pinot/pull/5752#discussion_r460339293 ## File path: thirdeye/thirdeye-frontend/app/pods/components/rootcause-select-comparison-range/component.js ## @@ -48,23 +50,15 @@ export defaul

[incubator-pinot] 03/04: request log

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 956cd269025143a3934cba726c7cbfb1aa0879a3 Author: Alexander Pucher AuthorDate: Thu Jul 23 20:30:41 2020

[incubator-pinot] 04/04: [Deepstore by-passing]Introduce a subclasses SplitSegmentCommitter which will proceeds to commit even if the segment upload fails. (#5700)

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 387f0e0641a17d24ed07170192e7fc31feb70494 Author: Ting Chen AuthorDate: Thu Jul 23 17:13:34 2020 -0700

[incubator-pinot] branch thirdeye-temp updated (41ee120 -> 387f0e0)

This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 41ee120 [Deepstore by-passing]Introduce a subclasses SplitSegmentCommitter which will proceeds to

[incubator-pinot] 02/04: external logger settings in docker configs

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit a1a93bc095c1414828184241887b8dd968e68faf Author: Alexander Pucher AuthorDate: Thu Jul 23 17:13:20 2020

[incubator-pinot] 01/04: add log conf to Dockerfile launch script

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 62583607ee820980e860ef3b69a2562df34f74c7 Author: Alexander Pucher AuthorDate: Thu Jul 23 16:25:20 2020

[incubator-pinot] branch support-multiple-datasources-per-type updated (ed9ace8 -> 6310468)

This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch support-multiple-datasources-per-type in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from ed9ace8 unfix predicate add 6310468 test fix No new revisions were a

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5737: Improvements to RealtimeProvisioningHelper command

mcvsubbu commented on a change in pull request #5737: URL: https://github.com/apache/incubator-pinot/pull/5737#discussion_r460333836 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/RealtimeProvisioningHelperCommand.java ## @@ -158,17 +173,36 @@ pub

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5737: Improvements to RealtimeProvisioningHelper command

mcvsubbu commented on a change in pull request #5737: URL: https://github.com/apache/incubator-pinot/pull/5737#discussion_r460333739 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/RealtimeProvisioningHelperCommand.java ## @@ -177,25 +211,26 @@ pub

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5737: Improvements to RealtimeProvisioningHelper command

mcvsubbu commented on a change in pull request #5737: URL: https://github.com/apache/incubator-pinot/pull/5737#discussion_r460333583 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/RealtimeProvisioningHelperCommand.java ## @@ -177,25 +211,26 @@ pub

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5737: Improvements to RealtimeProvisioningHelper command

mcvsubbu commented on a change in pull request #5737: URL: https://github.com/apache/incubator-pinot/pull/5737#discussion_r460333511 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/RealtimeProvisioningHelperCommand.java ## @@ -145,6 +146,20 @@ publ

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5737: Improvements to RealtimeProvisioningHelper command

npawar commented on a change in pull request #5737: URL: https://github.com/apache/incubator-pinot/pull/5737#discussion_r460240388 ## File path: pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/RealtimeProvisioningHelperCommand.java ## @@ -115,17 +116,17 @@ publi

[GitHub] [incubator-pinot] fx19880617 opened a new issue #5753: Built-in jobs to move segments of hybrid tables from Realtime Servers to Offline Servers

fx19880617 opened a new issue #5753: URL: https://github.com/apache/incubator-pinot/issues/5753 We should try to have some built-in scheduled jobs to move segments from realtime servers to offline servers. So that users don't need to setup external jobs to push segments from external data

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5746: Fix code to correctly extract value of multi-value column from avro file

siddharthteotia commented on a change in pull request #5746: URL: https://github.com/apache/incubator-pinot/pull/5746#discussion_r460321430 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/recordtransformer/PinotDataType.java ## @@ -470,6 +474,43 @@ public Str

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5746: Fix code to correctly extract value of multi-value column from avro file

siddharthteotia commented on a change in pull request #5746: URL: https://github.com/apache/incubator-pinot/pull/5746#discussion_r460317334 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/recordtransformer/DataTypeTransformer.java ## @@ -87,6 +90,8 @@ public

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5746: Fix code to correctly extract value of multi-value column from avro file

siddharthteotia commented on a change in pull request #5746: URL: https://github.com/apache/incubator-pinot/pull/5746#discussion_r460322646 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/recordtransformer/DataTypeTransformer.java ## @@ -87,6 +90,8 @@ public

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5746: Fix code to correctly extract value of multi-value column from avro file

siddharthteotia commented on a change in pull request #5746: URL: https://github.com/apache/incubator-pinot/pull/5746#discussion_r460321729 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/recordtransformer/PinotDataType.java ## @@ -632,4 +713,22 @@ public sta

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5746: Fix code to correctly extract value of multi-value column from avro file

siddharthteotia commented on a change in pull request #5746: URL: https://github.com/apache/incubator-pinot/pull/5746#discussion_r460321430 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/recordtransformer/PinotDataType.java ## @@ -470,6 +474,43 @@ public Str

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #5740: [TE] Added a backfill start date for Anomaly Detection

akshayrai commented on a change in pull request #5740: URL: https://github.com/apache/incubator-pinot/pull/5740#discussion_r460318552 ## File path: thirdeye/thirdeye-frontend/app/utils/yaml-tools.js ## @@ -29,6 +28,20 @@ rules: type: DATA_SLA # Alert if data i

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #5740: [TE] Added a backfill start date for Anomaly Detection

akshayrai commented on a change in pull request #5740: URL: https://github.com/apache/incubator-pinot/pull/5740#discussion_r460319108 ## File path: thirdeye/thirdeye-frontend/app/utils/yaml-tools.js ## @@ -29,6 +28,20 @@ rules: type: DATA_SLA # Alert if data i

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #5740: [TE] Added a backfill start date for Anomaly Detection

akshayrai commented on a change in pull request #5740: URL: https://github.com/apache/incubator-pinot/pull/5740#discussion_r460318786 ## File path: thirdeye/thirdeye-frontend/app/utils/yaml-tools.js ## @@ -12,12 +12,11 @@ dataset: dataset_to_which_this_metric_belongs rules: -

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #5740: [TE] Added a backfill start date for Anomaly Detection

akshayrai commented on a change in pull request #5740: URL: https://github.com/apache/incubator-pinot/pull/5740#discussion_r460318552 ## File path: thirdeye/thirdeye-frontend/app/utils/yaml-tools.js ## @@ -29,6 +28,20 @@ rules: type: DATA_SLA # Alert if data i

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #5740: [TE] Added a backfill start date for Anomaly Detection

akshayrai commented on a change in pull request #5740: URL: https://github.com/apache/incubator-pinot/pull/5740#discussion_r460318266 ## File path: thirdeye/thirdeye-frontend/app/utils/yaml-tools.js ## @@ -29,6 +28,20 @@ rules: type: DATA_SLA # Alert if data i

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5746: Fix code to correctly extract value of multi-value column from avro file

siddharthteotia commented on a change in pull request #5746: URL: https://github.com/apache/incubator-pinot/pull/5746#discussion_r460317334 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/recordtransformer/DataTypeTransformer.java ## @@ -87,6 +90,8 @@ public

[incubator-pinot] branch fix-multi-value-column-from-avro updated (9469cec -> 48a85f2)

This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch fix-multi-value-column-from-avro in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 9469cec Fix code to correctly extract value of multi-value column from avro file

[incubator-pinot] branch master updated: [TE] Thirdeye docker log crash fix (#5749)

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 405b4ef [TE] Thirdeye docker log

[GitHub] [incubator-pinot] akshayrai merged pull request #5749: Thirdeye docker log crash fix

akshayrai merged pull request #5749: URL: https://github.com/apache/incubator-pinot/pull/5749 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: [TE] pinot - harleyjj/detectionutils - return current when predicted baseline cannot be trained (#5736)

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 2af8d8e [TE] pinot - harleyjj/det

[GitHub] [incubator-pinot] akshayrai merged pull request #5736: [TE] pinot - harleyjj/detectionutils - return current when predicted …

akshayrai merged pull request #5736: URL: https://github.com/apache/incubator-pinot/pull/5736 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] akshayrai merged pull request #5713: [TE] Setup a rest client pipeline for comms between ThirdEye services; added RCA highlights API as an example

akshayrai merged pull request #5713: URL: https://github.com/apache/incubator-pinot/pull/5713 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: [TE] Setup a rest client pipeline for comms between ThirdEye services; added RCA highlights API as an example (#5713)

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 f5bb36b [TE] Setup a rest client

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #5752: [TE] frontend - harleyjj/rca - Phase 1 of custom baseline selector

akshayrai commented on a change in pull request #5752: URL: https://github.com/apache/incubator-pinot/pull/5752#discussion_r460234402 ## File path: thirdeye/thirdeye-frontend/app/pods/components/rootcause-select-comparison-range/component.js ## @@ -48,23 +50,15 @@ export defau

[incubator-pinot] 02/04: external logger settings in docker configs

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit d471f82924c67763aaa4df7b0f166415ac7f7f50 Author: Alexander Pucher AuthorDate: Thu Jul 23 17:13:20 2020

[incubator-pinot] 01/04: add log conf to Dockerfile launch script

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit c6a5cf615da477138b67d631dbade6d50a6ac771 Author: Alexander Pucher AuthorDate: Thu Jul 23 16:25:20 2020

[incubator-pinot] 03/04: request log

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit b0c4194aaa7eff47a292f4316d78050694725ab2 Author: Alexander Pucher AuthorDate: Thu Jul 23 20:30:41 2020

[incubator-pinot] branch support-multiple-datasources-per-type updated (0caec1f -> ed9ace8)

This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch support-multiple-datasources-per-type in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 0caec1f remove dependencies of PinotThirdEyeDataSource.DATA_SOURCE_NAME

[incubator-pinot] 04/04: [Deepstore by-passing]Introduce a subclasses SplitSegmentCommitter which will proceeds to commit even if the segment upload fails. (#5700)

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 41ee120d9a84318214600d48ed264541a13cbd73 Author: Ting Chen AuthorDate: Thu Jul 23 17:13:34 2020 -0700

[incubator-pinot] branch thirdeye-temp updated (abd7de7 -> 41ee120)

This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard abd7de7 [Deepstore by-passing]Introduce a subclasses SplitSegmentCommitter which will proceeds to

[incubator-pinot] 02/04: external logger settings in docker configs

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 069003987628ca77b42f811213afea798ed83fc9 Author: Alexander Pucher AuthorDate: Thu Jul 23 17:13:20 2020

[incubator-pinot] 01/04: add log conf to Dockerfile launch script

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit be30a9f7ddf9933231bb02eac6e05ee59ee7bfac Author: Alexander Pucher AuthorDate: Thu Jul 23 16:25:20 2020

[incubator-pinot] 04/04: [Deepstore by-passing]Introduce a subclasses SplitSegmentCommitter which will proceeds to commit even if the segment upload fails. (#5700)

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit abd7de7cb414bb8899bb4fe1be469ad6b97d8703 Author: Ting Chen AuthorDate: Thu Jul 23 17:13:34 2020 -0700

[incubator-pinot] branch thirdeye-temp updated (69b9bc4 -> abd7de7)

This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 69b9bc4 [Deepstore by-passing]Introduce a subclasses SplitSegmentCommitter which will proceeds to

[incubator-pinot] 03/04: request log

This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch thirdeye-temp in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 6b579867f50470fbbf438730d314f5125a96b52f Author: Alexander Pucher AuthorDate: Thu Jul 23 20:30:41 2020

[incubator-pinot] branch support-multiple-datasources-per-type updated (02d7583 -> 0caec1f)

This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch support-multiple-datasources-per-type in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 02d7583 support optional 'name' field in data source config proeprties

[GitHub] [incubator-pinot] harleyjj opened a new pull request #5752: [TE] frontend - harleyjj/rca - Phase 1 of custom baseline selector

harleyjj opened a new pull request #5752: URL: https://github.com/apache/incubator-pinot/pull/5752 ## Description * This PR allows the UI to lever the versatility of the RCA times series endpoint to provide a broad range of baselines * Future addition to this modal will be a tab for

[GitHub] [incubator-pinot] mcvsubbu commented on issue #5751: Support Serving Latest Offline Segments Immediately

mcvsubbu commented on issue #5751: URL: https://github.com/apache/incubator-pinot/issues/5751#issuecomment-663679922 Here is my suggested approach. 1. Create a config in table that indicates that for a table we need the time boundary to be next unit up. (i.e. if daily push, next day, if

[GitHub] [incubator-pinot] ianvkoeppe closed pull request #5745: Allow serving of offline segments immediately/inclusively.

ianvkoeppe closed pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745 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] ianvkoeppe commented on pull request #5745: Allow serving of offline segments immediately/inclusively.

ianvkoeppe commented on pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745#issuecomment-663674214 @kishoreg and @mcvsubbu I've created this issue: https://github.com/apache/incubator-pinot/issues/5751. I've described the problem statement and a couple potential ap

[GitHub] [incubator-pinot] ianvkoeppe opened a new issue #5751: Support Serving Latest Offline Segments Immediately

ianvkoeppe opened a new issue #5751: URL: https://github.com/apache/incubator-pinot/issues/5751 ## Overview For hybrid setups, Pinot splits/filters broker queries to both the offline and realtime tables. It does so based on a `time value` which represents the latest available offlin

[GitHub] [incubator-pinot] mcvsubbu commented on pull request #5745: Allow serving of offline segments immediately/inclusively.

mcvsubbu commented on pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745#issuecomment-663672482 @ianvkoeppe can u please create an issue first? It is better to agree on an implementation strategy in an issue. In case of external API changes, let us define the APIs

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5745: Allow serving of offline segments immediately/inclusively.

mcvsubbu commented on a change in pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745#discussion_r460214241 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/HelixBrokerStarter.java ## @@ -235,7 +234,7 @@ public void start()

[GitHub] [incubator-pinot] kishoreg commented on a change in pull request #5748: GROOVY transform function UDF for queries

kishoreg commented on a change in pull request #5748: URL: https://github.com/apache/incubator-pinot/pull/5748#discussion_r460145074 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/function/GroovyFunctionEvaluator.java ## @@ -55,6 +55,7 @@ private static f

[GitHub] [incubator-pinot] ianvkoeppe commented on a change in pull request #5745: Allow serving of offline segments immediately/inclusively.

ianvkoeppe commented on a change in pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745#discussion_r460209486 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java ## @@ -1087,6 +1084,22 @@ privat

[GitHub] [incubator-pinot] kishoreg commented on pull request #5745: Allow serving of offline segments immediately/inclusively.

kishoreg commented on pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745#issuecomment-663664728 @ianvkoeppe this is definitely a good feature to have. However, the solution is too restrictive and we generally prefer table level config (for multi-tenancy). Can

[GitHub] [incubator-pinot] yupeng9 commented on issue #5747: Check overlapping offline segments

yupeng9 commented on issue #5747: URL: https://github.com/apache/incubator-pinot/issues/5747#issuecomment-663662162 Right. The generated segment name includes three parts: time column's min value, time columns' max value, sequence id. e.g. `transcript_OFFLINE_157086360_157241880_0

[GitHub] [incubator-pinot] Sreemanth commented on issue #5559: Table truncate command

Sreemanth commented on issue #5559: URL: https://github.com/apache/incubator-pinot/issues/5559#issuecomment-663655592 @npawar Ensure table setup logic added. This is an automated message from the Apache Git Service. To

[GitHub] [incubator-pinot] ianvkoeppe commented on a change in pull request #5745: Allow serving of offline segments immediately/inclusively.

ianvkoeppe commented on a change in pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745#discussion_r460197217 ## File path: pinot-spi/src/main/java/org/apache/pinot/spi/config/table/QueryConfig.java ## @@ -37,15 +37,23 @@ // If the server times out,

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5745: Allow serving of offline segments immediately/inclusively.

mcvsubbu commented on a change in pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745#discussion_r460169975 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/HelixBrokerStarter.java ## @@ -235,7 +234,7 @@ public void start()

[incubator-pinot] branch master updated: [TE] Make few internal documentationi and mocks publicwq (#5743)

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 cb149be [TE] Make few internal do

[GitHub] [incubator-pinot] akshayrai merged pull request #5743: [TE] Make few internal documentation and mocks public

akshayrai merged pull request #5743: URL: https://github.com/apache/incubator-pinot/pull/5743 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] ianvkoeppe commented on a change in pull request #5745: Allow serving of offline segments immediately/inclusively.

ianvkoeppe commented on a change in pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745#discussion_r460132216 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java ## @@ -122,19 +122,16 @@ pu

[GitHub] [incubator-pinot] cbrentharris commented on a change in pull request #5745: Allow serving of offline segments immediately/inclusively.

cbrentharris commented on a change in pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745#discussion_r460111885 ## File path: pinot-spi/src/main/java/org/apache/pinot/spi/config/table/QueryConfig.java ## @@ -37,15 +37,23 @@ // If the server times ou

[incubator-pinot] branch pinot_client_controller_constructor updated (0b39c5e -> 1130bf1)

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch pinot_client_controller_constructor in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 0b39c5e Adding pinot java client constructor only by pinot controller ad

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5719: Adding support to construct pinot java client with pinot controller url

fx19880617 commented on a change in pull request #5719: URL: https://github.com/apache/incubator-pinot/pull/5719#discussion_r459883768 ## File path: pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/ControllerBasedBrokerSelector.java ## @@ -0,0 +1,140 @@ +/

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5719: Adding support to construct pinot java client with pinot controller url

fx19880617 commented on a change in pull request #5719: URL: https://github.com/apache/incubator-pinot/pull/5719#discussion_r459883507 ## File path: pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/ControllerBasedBrokerSelector.java ## @@ -0,0 +1,140 @@ +/

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5719: Adding support to construct pinot java client with pinot controller url

fx19880617 commented on a change in pull request #5719: URL: https://github.com/apache/incubator-pinot/pull/5719#discussion_r459883456 ## File path: pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/ControllerBasedBrokerSelector.java ## @@ -0,0 +1,140 @@ +/

[incubator-pinot] branch pinot_client_controller_constructor updated (cbeecde -> 0b39c5e)

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch pinot_client_controller_constructor in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard cbeecde Adding pinot java client constructor only by pinot controller ad

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5719: Adding support to construct pinot java client with pinot controller url

fx19880617 commented on a change in pull request #5719: URL: https://github.com/apache/incubator-pinot/pull/5719#discussion_r459882804 ## File path: pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/ControllerBasedBrokerSelector.java ## @@ -0,0 +1,140 @@ +/