[GitHub] [incubator-pinot] buchireddy opened a new pull request #4432: #4401 Reuse a ThreadLocal byte[] when reading String elements from the variable length value reader.

buchireddy opened a new pull request #4432: #4401 Reuse a ThreadLocal byte[] when reading String elements from the variable length value reader. URL: https://github.com/apache/incubator-pinot/pull/4432 This is an automated m

[incubator-pinot] branch master updated: Add pinot community inviter (#4424)

This is an automated email from the ASF dual-hosted git repository. snlee 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 52f9b08 Add pinot community inviter (#4

[GitHub] [incubator-pinot] snleee merged pull request #4424: Add pinot community inviter

snleee merged pull request #4424: Add pinot community inviter URL: https://github.com/apache/incubator-pinot/pull/4424 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[incubator-pinot] branch document_configuration updated (7bea398 -> 322d4a9)

This is an automated email from the ASF dual-hosted git repository. xhsun pushed a change to branch document_configuration in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 7bea398 [TE] update documents on configuration add 322d4a9 [TE] update documents on co

[incubator-pinot] branch document_configuration updated (6b833b9 -> 7bea398)

This is an automated email from the ASF dual-hosted git repository. xhsun pushed a change to branch document_configuration in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 6b833b9 [TE] added ThirdEye configuration documents add 7bea398 [TE] update documents

[incubator-pinot] branch misc_fix created (now 9c4d9fa)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch misc_fix in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 9c4d9fa Misc fix for controller tests This branch includes the following new commits: new 9c4d9fa

[incubator-pinot] 01/01: Misc fix for controller tests

This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch misc_fix in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 9c4d9fa43adda13a96577e48dc37f44165253c34 Author: Jackie (Xiaotian) Jiang AuthorDate: Thu Jul 11 18:46:09 2019

[incubator-pinot] branch helix_setup created (now db6fdf8)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch helix_setup in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at db6fdf8 Cleanup un-necessary setups in HelixSetupUtils This branch includes the following new commits

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #4430: Cleanup un-necessary setups in HelixSetupUtils

Jackie-Jiang opened a new pull request #4430: Cleanup un-necessary setups in HelixSetupUtils URL: https://github.com/apache/incubator-pinot/pull/4430 - Do not enable leadControllerResource by default (no auto-rebalance) - Remove redundant empty brokerResource resource config - Remove

[incubator-pinot] 01/01: Cleanup un-necessary setups in HelixSetupUtils

This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch helix_setup in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit db6fdf8da0321d1a1ef283bdb3becd64ad6c4068 Author: Jackie (Xiaotian) Jiang AuthorDate: Thu Jul 11 18:20:42 2

[incubator-pinot] branch uri_encoding updated (6c4f76c -> 4ec26fc)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch uri_encoding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. omit 6c4f76c Add URIUtils class to handle URI/URL encoding/decoding add 4ec26fc Add URIUtils class t

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding

Jackie-Jiang commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding URL: https://github.com/apache/incubator-pinot/pull/4426#discussion_r302793095 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/Pin

[GitHub] [incubator-pinot] xiaohui-sun opened a new pull request #4429: [TE] added ThirdEye configuration documents

xiaohui-sun opened a new pull request #4429: [TE] added ThirdEye configuration documents URL: https://github.com/apache/incubator-pinot/pull/4429 This is an automated message from the Apache Git Service. To respond to the me

[incubator-pinot] 01/01: [TE] added ThirdEye configuration documents

This is an automated email from the ASF dual-hosted git repository. xhsun pushed a commit to branch document_configuration in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 6b833b95d7bd6aea3b8ec0b3c6ed86e4a29c2d13 Author: Xiaohui Sun AuthorDate: Thu Jul 11 17:41:18 201

[incubator-pinot] branch document_configuration created (now 6b833b9)

This is an automated email from the ASF dual-hosted git repository. xhsun pushed a change to branch document_configuration in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 6b833b9 [TE] added ThirdEye configuration documents This branch includes the following new

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation

siddharthteotia commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation URL: https://github.com/apache/incubator-pinot/pull/4368#discussion_r302791778 ## File path: pinot-core/src/test/java/org/apache/pinot/core/segm

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4428: [TE] detection health - coverage fix

jihaozh commented on a change in pull request #4428: [TE] detection health - coverage fix URL: https://github.com/apache/incubator-pinot/pull/4428#discussion_r302790505 ## File path: thirdeye/thirdeye-frontend/app/pods/components/detection-health/template.hbs ## @@ -26,7

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding

jackjlli commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding URL: https://github.com/apache/incubator-pinot/pull/4426#discussion_r302789354 ## File path: pinot-common/src/main/java/org/apache/pinot/common/utils/URIUtils.java

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding

jackjlli commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding URL: https://github.com/apache/incubator-pinot/pull/4426#discussion_r302789110 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/PinotUR

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding

jackjlli commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding URL: https://github.com/apache/incubator-pinot/pull/4426#discussion_r302790223 ## File path: pinot-common/src/main/java/org/apache/pinot/common/utils/URIUtils.java

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding

jackjlli commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding URL: https://github.com/apache/incubator-pinot/pull/4426#discussion_r302790156 ## File path: pinot-common/src/main/java/org/apache/pinot/common/utils/URIUtils.java

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding

jackjlli commented on a change in pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding URL: https://github.com/apache/incubator-pinot/pull/4426#discussion_r302788814 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSe

[incubator-pinot] branch uri_encoding updated (3cfe19e -> 6c4f76c)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch uri_encoding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 3cfe19e Add URIUtils class to handle URI/URL encoding/decoding add 6c4f76c Add URIUtils class t

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4428: [TE] detection health - coverage fix

xiaohui-sun commented on a change in pull request #4428: [TE] detection health - coverage fix URL: https://github.com/apache/incubator-pinot/pull/4428#discussion_r302786669 ## File path: thirdeye/thirdeye-frontend/app/pods/components/detection-health/template.hbs ## @@ -2

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4428: [TE] detection health - coverage fix

jihaozh opened a new pull request #4428: [TE] detection health - coverage fix URL: https://github.com/apache/incubator-pinot/pull/4428 - Fix the issue that anomaly coverage can be more than 100% - Update the logic to get the latest success task to finish time in backend

[incubator-pinot] branch uri_encoding updated (403f5b7 -> 3cfe19e)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch uri_encoding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 403f5b7 Add URIUtils class to handle URI/URL encoding/decoding add 3cfe19e Add URIUtils class t

[GitHub] [incubator-pinot] siddharthteotia opened a new issue #4427: Consider using epoll based event loop in Netty when Pinot runs on Linux systems

siddharthteotia opened a new issue #4427: Consider using epoll based event loop in Netty when Pinot runs on Linux systems URL: https://github.com/apache/incubator-pinot/issues/4427 Netty provides the asynchronous event based multiplexing over connections in 3 different flavors -- java nio

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302774250 ## File path: pinot-common/src/main/java/org/apache/pinot/common/utils

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation

jackjlli commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation URL: https://github.com/apache/incubator-pinot/pull/4368#discussion_r302773071 ## File path: pinot-core/src/test/java/org/apache/pinot/core/segment/ind

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302772372 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Closea

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation

siddharthteotia commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation URL: https://github.com/apache/incubator-pinot/pull/4368#discussion_r302770631 ## File path: pinot-core/src/test/java/org/apache/pinot/core/segm

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation

siddharthteotia commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation URL: https://github.com/apache/incubator-pinot/pull/4368#discussion_r302770571 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segm

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation

siddharthteotia commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation URL: https://github.com/apache/incubator-pinot/pull/4368#discussion_r302770596 ## File path: pinot-core/src/test/java/org/apache/pinot/core/data

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding

Jackie-Jiang opened a new pull request #4426: Add URIUtils class to handle URI/URL encoding/decoding URL: https://github.com/apache/incubator-pinot/pull/4426 Fix the double encoding bug of encoding URI without scheme "/segments/segment %" -> Without fix: "file:/segments/segment+%2525"

[incubator-pinot] branch uri_encoding created (now 403f5b7)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch uri_encoding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 403f5b7 Add URIUtils class to handle URI/URL encoding/decoding This branch includes the following ne

[incubator-pinot] 01/01: Add URIUtils class to handle URI/URL encoding/decoding

This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch uri_encoding in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 403f5b75bd45cd4f54a0cdcd04b639d1127a740d Author: Jackie (Xiaotian) Jiang AuthorDate: Thu Jul 11 15:19:33

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #4418: Fix potential resource leak in the way we close a collection of closeables

codecov-io edited a comment on issue #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#issuecomment-510276514 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4418?src=pr&el=h1) Report

[GitHub] [incubator-pinot] siddharthteotia commented on issue #4418: Fix potential resource leak in the way we close a collection of closeables

siddharthteotia commented on issue #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#issuecomment-510649382 Based on the above conversations, I have addressed the review comments in latest commit. ---

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302736411 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Clo

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302736329 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Clo

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302736254 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Clo

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302736284 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segmen

[GitHub] [incubator-pinot] npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302680527 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafka2

[GitHub] [incubator-pinot] npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302678609 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafka2

[GitHub] [incubator-pinot] npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302680915 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafka2

[GitHub] [incubator-pinot] npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302675363 ## File path: pinot-connectors/pinot-connector-kafka-2.0/pom.xml ## @@ -0,0 +1,103 @@ + + +http://maven.

[GitHub] [incubator-pinot] npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302713189 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafka2

[GitHub] [incubator-pinot] npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302688764 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafka2

[GitHub] [incubator-pinot] npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302682512 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafka2

[GitHub] [incubator-pinot] npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

npawar commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302674935 ## File path: pinot-connectors/pinot-connector-kafka-2.0/README.md ## @@ -0,0 +1,24 @@ + +# Pinot connec

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302714401 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Clo

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #4323: [Controller Separation] Add logic for lead controller resource

codecov-io edited a comment on issue #4323: [Controller Separation] Add logic for lead controller resource URL: https://github.com/apache/incubator-pinot/pull/4323#issuecomment-502327426 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4323?src=pr&el=h1) Report > Merging

[GitHub] [incubator-pinot] ShubhamMMT1 opened a new issue #4425: Anomalies tab shows anomalies for only mock data source

ShubhamMMT1 opened a new issue #4425: Anomalies tab shows anomalies for only mock data source URL: https://github.com/apache/incubator-pinot/issues/4425 1. Ingested data into pinot. Had set up alerts for different metrics. The anomalies appear for different metrics in the home's tab as wel

[incubator-pinot] branch cluster_change_mediator deleted (was b47866e)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch cluster_change_mediator in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was b47866e In ClusterChangeMediator, stop enqueue/process changes if already stopped The re

[incubator-pinot] branch master updated: In ClusterChangeMediator, stop enqueue/process changes if already stopped (#4422)

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/incubator-pinot.git The following commit(s) were added to refs/heads/master by this push: new aadcd36 In ClusterChangeMediator, stop

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #4422: In ClusterChangeMediator, stop enqueue/process changes if already stopped

Jackie-Jiang merged pull request #4422: In ClusterChangeMediator, stop enqueue/process changes if already stopped URL: https://github.com/apache/incubator-pinot/pull/4422 This is an automated message from the Apache Git Serv

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302685987 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Closea

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302685376 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Closea

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4423: [TE] add ml auto config into YAML template

jihaozh commented on a change in pull request #4423: [TE] add ml auto config into YAML template URL: https://github.com/apache/incubator-pinot/pull/4423#discussion_r302684568 ## File path: thirdeye/thirdeye-frontend/app/utils/constants.js ## @@ -27,6 +27,7 @@ rules:

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302684714 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Clo

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302684337 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/c

[GitHub] [incubator-pinot] cyy0714 commented on a change in pull request #4423: [TE] add ml auto config into YAML template

cyy0714 commented on a change in pull request #4423: [TE] add ml auto config into YAML template URL: https://github.com/apache/incubator-pinot/pull/4423#discussion_r302684105 ## File path: thirdeye/thirdeye-frontend/app/utils/constants.js ## @@ -27,6 +27,7 @@ rules:

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302683406 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Clo

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

siddharthteotia commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302683406 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Clo

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

Jackie-Jiang commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302682935 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Closea

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4423: [TE] add ml auto config into YAML template

akshayrai commented on a change in pull request #4423: [TE] add ml auto config into YAML template URL: https://github.com/apache/incubator-pinot/pull/4423#discussion_r302682042 ## File path: thirdeye/thirdeye-frontend/app/utils/constants.js ## @@ -27,6 +27,7 @@ rules:

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4421: [TE] detection health UI adjustments

xiaohui-sun commented on a change in pull request #4421: [TE] detection health UI adjustments URL: https://github.com/apache/incubator-pinot/pull/4421#discussion_r302682047 ## File path: thirdeye/thirdeye-frontend/app/pods/components/detection-health/template.hbs ## @@ -7

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4421: [TE] detection health UI adjustments

xiaohui-sun commented on a change in pull request #4421: [TE] detection health UI adjustments URL: https://github.com/apache/incubator-pinot/pull/4421#discussion_r302681153 ## File path: thirdeye/thirdeye-frontend/app/pods/components/detection-health/template.hbs ## @@ -1

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4421: [TE] detection health UI adjustments

xiaohui-sun commented on a change in pull request #4421: [TE] detection health UI adjustments URL: https://github.com/apache/incubator-pinot/pull/4421#discussion_r302681493 ## File path: thirdeye/thirdeye-frontend/app/pods/components/detection-health/template.hbs ## @@ -7

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4423: [TE] add ml auto config into YAML template

akshayrai commented on a change in pull request #4423: [TE] add ml auto config into YAML template URL: https://github.com/apache/incubator-pinot/pull/4423#discussion_r302682042 ## File path: thirdeye/thirdeye-frontend/app/utils/constants.js ## @@ -27,6 +27,7 @@ rules:

[incubator-pinot] branch add-logic-for-lead-controller-resource updated (480f49d -> f7132cd)

This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch add-logic-for-lead-controller-resource in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. omit 480f49d Address PR comments add f7132cd Address PR comments This updat

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

mcvsubbu commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302644058 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafk

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

mcvsubbu commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302638730 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafk

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

mcvsubbu commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302673645 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafk

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

mcvsubbu commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302645727 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafk

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer

mcvsubbu commented on a change in pull request #4397: Adding Support for Kafka 2.0 Consumer URL: https://github.com/apache/incubator-pinot/pull/4397#discussion_r302679686 ## File path: pinot-connectors/pinot-connector-kafka-2.0/src/main/java/org/apache/pinot/core/realtime/impl/kafk

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend

jihaozh commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend URL: https://github.com/apache/incubator-pinot/pull/4419#discussion_r302677026 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/ap

[GitHub] [incubator-pinot] akshayrai merged pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend

akshayrai merged pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend URL: https://github.com/apache/incubator-pinot/pull/4419 This is an automated message from the

[incubator-pinot] branch master updated: [TE] Fix exception handling - Propagate and display the error message/exception on frontind (#4419)

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 0993974 [TE] Fix exception handli

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend

jihaozh commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend URL: https://github.com/apache/incubator-pinot/pull/4419#discussion_r302676262 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/ap

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation

jackjlli commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation URL: https://github.com/apache/incubator-pinot/pull/4368#discussion_r302673886 ## File path: pinot-core/src/test/java/org/apache/pinot/core/segment/ind

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation

jackjlli commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation URL: https://github.com/apache/incubator-pinot/pull/4368#discussion_r302671179 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/cre

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation

jackjlli commented on a change in pull request #4368: 3891: Check for validity of segment start/end time during segment generation URL: https://github.com/apache/incubator-pinot/pull/4368#discussion_r302671587 ## File path: pinot-core/src/test/java/org/apache/pinot/core/data/reader

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend

akshayrai commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend URL: https://github.com/apache/incubator-pinot/pull/4419#discussion_r302672546 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/

[GitHub] [incubator-pinot] ananthdurai opened a new pull request #4424: Add pinot community inviter

ananthdurai opened a new pull request #4424: Add pinot community inviter URL: https://github.com/apache/incubator-pinot/pull/4424 The current process to onboard new users into the Pinot & ThirdEye Slack community requires a manual invite. Community inviter is a free service that can automa

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend

akshayrai commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend URL: https://github.com/apache/incubator-pinot/pull/4419#discussion_r302670565 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/

[incubator-pinot] branch master updated: make kafka version number controlled by config number (#4396)

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 bca6756 make kafka version number c

[GitHub] [incubator-pinot] npawar merged pull request #4396: make kafka version number controlled by config number

npawar merged pull request #4396: make kafka version number controlled by config number URL: https://github.com/apache/incubator-pinot/pull/4396 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4423: [TE] add ml auto config into YAML template

jihaozh opened a new pull request #4423: [TE] add ml auto config into YAML template URL: https://github.com/apache/incubator-pinot/pull/4423 Add the ml auto-config into the default YAML template. This is an automated message

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend

jihaozh commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend URL: https://github.com/apache/incubator-pinot/pull/4419#discussion_r302654765 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/ap

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend

jihaozh commented on a change in pull request #4419: [TE] Fix exception handling - Propagate and display the error message/exception on frontend URL: https://github.com/apache/incubator-pinot/pull/4419#discussion_r302654507 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/ap

[GitHub] [incubator-pinot] jihaozh merged pull request #4421: [TE] detection health UI adjustments

jihaozh merged pull request #4421: [TE] detection health UI adjustments URL: https://github.com/apache/incubator-pinot/pull/4421 This is an automated message from the Apache Git Service. To respond to the message, please log

[incubator-pinot] branch master updated: [TE] detection health ui adjustments (#4421)

This is an automated email from the ASF dual-hosted git repository. jihao 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 b6fe4e6 [TE] detection health ui adjust

[incubator-pinot] branch master updated: prompt on fail schema (#4395)

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 dac9ae2 prompt on fail schema (#4395) da

[GitHub] [incubator-pinot] jackjlli merged pull request #4395: Prompt when get table schema failed on query-console

jackjlli merged pull request #4395: Prompt when get table schema failed on query-console URL: https://github.com/apache/incubator-pinot/pull/4395 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

mcvsubbu commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302626793 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Closeables

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables

mcvsubbu commented on a change in pull request #4418: Fix potential resource leak in the way we close a collection of closeables URL: https://github.com/apache/incubator-pinot/pull/4418#discussion_r302629209 ## File path: pinot-common/src/main/java/org/apache/pinot/common/Closeables

[GitHub] [incubator-pinot] buchireddy closed issue #4317: Support variable length Offline Dictionary Indexes for bytes, strings and maps to save on storage

buchireddy closed issue #4317: Support variable length Offline Dictionary Indexes for bytes, strings and maps to save on storage URL: https://github.com/apache/incubator-pinot/issues/4317 This is an automated message from th

[incubator-pinot] branch master updated: [TE] Inject the port from config into Commons mail SSL Smtp port (#4420)

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 6bec451 [TE] Inject the port from

[GitHub] [incubator-pinot] akshayrai merged pull request #4420: [TE] Inject the port from config into Commons mail SSL Smtp port

akshayrai merged pull request #4420: [TE] Inject the port from config into Commons mail SSL Smtp port URL: https://github.com/apache/incubator-pinot/pull/4420 This is an automated message from the Apache Git Service. To resp

  1   2   >