[GitHub] [incubator-pinot] ShubhamMMT1 commented on issue #4334: Pinot datasets not showing in thirdye UI

ShubhamMMT1 commented on issue #4334: Pinot datasets not showing in thirdye UI URL: https://github.com/apache/incubator-pinot/issues/4334#issuecomment-503432908 ERROR [2019-06-19 06:36:26,339] org.apache.pinot.thirdeye.datasource.DataSourcesLoader: Exception in creatingthirdeyedatasourceo

[GitHub] [incubator-pinot] haibow closed pull request #4335: Refresh schema when reloading segments

haibow closed pull request #4335: Refresh schema when reloading segments URL: https://github.com/apache/incubator-pinot/pull/4335 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-pinot] haibow opened a new pull request #4335: Refresh schema when reloading segments

haibow opened a new pull request #4335: Refresh schema when reloading segments URL: https://github.com/apache/incubator-pinot/pull/4335 Now when reloading segments, schema is re-fetched only for OFFLINE table. For realtime tables, it will still reload with the old schema, ignoring new colu

[GitHub] [incubator-pinot] haibow opened a new pull request #4335: Refresh schema when reloading segments

haibow opened a new pull request #4335: Refresh schema when reloading segments URL: https://github.com/apache/incubator-pinot/pull/4335 Now when reloading segments, schema is re-fetched only for OFFLINE table. For realtime tables, it will still reload with the old schema, ignoring new colu

[GitHub] [incubator-pinot] haibow closed pull request #4335: Refresh schema when reloading segments

haibow closed pull request #4335: Refresh schema when reloading segments URL: https://github.com/apache/incubator-pinot/pull/4335 This is an automated message from the Apache Git Service. To respond to the message, please log

[incubator-pinot] branch always_download_realtime_segments deleted (was 0990f83)

This is an automated email from the ASF dual-hosted git repository. nehapawar pushed a change to branch always_download_realtime_segments in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 0990f83 Always download realtime segments on non-winners This change permane

[GitHub] [incubator-pinot] jackjlli edited a comment on issue #4339: Only set Helix cluster properties when creating the cluster

jackjlli edited a comment on issue #4339: Only set Helix cluster properties when creating the cluster URL: https://github.com/apache/incubator-pinot/pull/4339#issuecomment-503415677 @Jackie-Jiang The existing code will check whether auto-join is enabled or not. It will set it **only when**

[GitHub] [incubator-pinot] jackjlli commented on issue #4339: Only set Helix cluster properties when creating the cluster

jackjlli commented on issue #4339: Only set Helix cluster properties when creating the cluster URL: https://github.com/apache/incubator-pinot/pull/4339#issuecomment-503415677 @Jackie-Jiang The existing code will check whether auto-join is enabled or not. It will set it **only when** auto-j

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing

codecov-io edited a comment on issue #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#issuecomment-503389816 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4338

[incubator-pinot] branch download_realtime_hotfix updated: Remove repositories

This is an automated email from the ASF dual-hosted git repository. nehapawar pushed a commit to branch download_realtime_hotfix in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/download_realtime_hotfix by this push: new

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing

jackjlli commented on a change in pull request #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#discussion_r295126848 ## File path: pinot-broker/src/main/java/org/apac

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing

jackjlli commented on a change in pull request #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#discussion_r295126430 ## File path: pinot-broker/src/main/java/org/apac

[incubator-pinot] 01/01: Hotfix for making realtime non-winner servers always download and never build

This is an automated email from the ASF dual-hosted git repository. nehapawar pushed a commit to branch download_realtime_hotfix in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 7fa32676d54f086e13c61d13feda6d741699c9c0 Author: Neha Pawar AuthorDate: Tue Jun 18 22:34:5

[incubator-pinot] branch download_realtime_hotfix created (now 7fa3267)

This is an automated email from the ASF dual-hosted git repository. nehapawar pushed a change to branch download_realtime_hotfix in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 7fa3267 Hotfix for making realtime non-winner servers always download and never build

[incubator-pinot] branch remove-jboss-chibi deleted (was c5ca7af)

This is an automated email from the ASF dual-hosted git repository. snlee pushed a change to branch remove-jboss-chibi in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was c5ca7af Remove jboss and chibi repository from pom file Since we do not need jboss and chibi re

[GitHub] [incubator-pinot] snleee commented on issue #4329: Enable https over apache's maven repo

snleee commented on issue #4329: Enable https over apache's maven repo URL: https://github.com/apache/incubator-pinot/pull/4329#issuecomment-503411364 https://github.com/apache/incubator-pinot/pull/4340 This is an automated me

[GitHub] [incubator-pinot] snleee commented on issue #4330: Including https apache repo in repositories

snleee commented on issue #4330: Including https apache repo in repositories URL: https://github.com/apache/incubator-pinot/pull/4330#issuecomment-503411298 https://github.com/apache/incubator-pinot/pull/4340 This is an au

[GitHub] [incubator-pinot] snleee closed pull request #4330: Including https apache repo in repositories

snleee closed pull request #4330: Including https apache repo in repositories URL: https://github.com/apache/incubator-pinot/pull/4330 This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [incubator-pinot] snleee merged pull request #4340: Remove jboss and chibi repository from pom file

snleee merged pull request #4340: Remove jboss and chibi repository from pom file URL: https://github.com/apache/incubator-pinot/pull/4340 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-pinot] snleee closed pull request #4329: Enable https over apache's maven repo

snleee closed pull request #4329: Enable https over apache's maven repo URL: https://github.com/apache/incubator-pinot/pull/4329 This is an automated message from the Apache Git Service. To respond to the message, please log

[incubator-pinot] branch master updated: Remove jboss and chibi repository from pom file (#4340)

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 43c70c9 Remove jboss and chibi reposito

[incubator-pinot] branch remove-jboss-chibi updated (5794eec -> c5ca7af)

This is an automated email from the ASF dual-hosted git repository. snlee pushed a change to branch remove-jboss-chibi in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 5794eec Remove jboss and chibi repository from pom file new c5ca7af Remove jboss and chibi

[incubator-pinot] 01/01: Remove jboss and chibi repository from pom file Since we do not need jboss and chibi repositories for building Pinot, thir pr removes the unnecessary repositories.

This is an automated email from the ASF dual-hosted git repository. snlee pushed a commit to branch remove-jboss-chibi in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit c5ca7af137ef6379eebc111b5db7ab20226061a8 Author: Seunghyun Lee AuthorDate: Tue Jun 18 22:08:43 2019

[GitHub] [incubator-pinot] snleee opened a new pull request #4340: Remove jboss and chibi repository from pom file

snleee opened a new pull request #4340: Remove jboss and chibi repository from pom file URL: https://github.com/apache/incubator-pinot/pull/4340 Since we do not need jboss and chibi repositories for building Pinot, thir pr removes the unnecessary repositories. --

[incubator-pinot] branch remove-jboss-chibi created (now 5794eec)

This is an automated email from the ASF dual-hosted git repository. snlee pushed a change to branch remove-jboss-chibi in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 5794eec Remove jboss and chibi repository from pom file This branch includes the following new

[incubator-pinot] 01/01: Remove jboss and chibi repository from pom file

This is an automated email from the ASF dual-hosted git repository. snlee pushed a commit to branch remove-jboss-chibi in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 5794eec39a877532d65c11a279c39ae39d9be84d Author: Seunghyun Lee AuthorDate: Tue Jun 18 22:08:43 2019

[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing

Jackie-Jiang commented on issue #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#issuecomment-503406441 @snleee @mcvsubbu Added unit test -

[incubator-pinot] branch consuming_routing updated (194a969 -> f8833d1)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch consuming_routing in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 194a969 Within a partition, only allow querying the first CONSUMING segment for the real-time t

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

buchireddy commented on 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#issuecomment-503405306 @mcvsubbu Here is the PR for this issue. https://github.com/apache/incub

[GitHub] [incubator-pinot] codecov-io commented on issue #4337: Take OFFLINE segment into account for real-time rebalancer

codecov-io commented on issue #4337: Take OFFLINE segment into account for real-time rebalancer URL: https://github.com/apache/incubator-pinot/pull/4337#issuecomment-503403448 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4337?src=pr&el=h1) Report > :exclamation: No cov

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing

Jackie-Jiang commented on a change in pull request #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#discussion_r295117490 ## File path: pinot-broker/src/main/java/org/

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing

mcvsubbu commented on a change in pull request #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#discussion_r295116979 ## File path: pinot-broker/src/main/java/org/apac

[GitHub] [incubator-pinot] haibow opened a new pull request #4335: Refresh schema when reloading segments

haibow opened a new pull request #4335: Refresh schema when reloading segments URL: https://github.com/apache/incubator-pinot/pull/4335 Now when reloading segments, schema is re-fetched only for OFFLINE table. For realtime tables, it will still reload with the old schema, ignoring new colu

[GitHub] [incubator-pinot] haibow closed pull request #4335: Refresh schema when reloading segments

haibow closed pull request #4335: Refresh schema when reloading segments URL: https://github.com/apache/incubator-pinot/pull/4335 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing

Jackie-Jiang commented on issue #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#issuecomment-503400722 > Got it. Good idea. One concern with this is if one replica of a segmen

[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #4339: Only set Helix cluster properties when creating the cluster

Jackie-Jiang commented on issue #4339: Only set Helix cluster properties when creating the cluster URL: https://github.com/apache/incubator-pinot/pull/4339#issuecomment-503399728 @mcvsubbu The auto-join the number of threads should already be configured in the Helix cluster before Pinot co

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #4339: Only set Helix cluster properties when creating the cluster

Jackie-Jiang merged pull request #4339: Only set Helix cluster properties when creating the cluster URL: https://github.com/apache/incubator-pinot/pull/4339 This is an automated message from the Apache Git Service. To respon

[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #4339: Only set Helix cluster properties when creating the cluster

Jackie-Jiang commented on issue #4339: Only set Helix cluster properties when creating the cluster URL: https://github.com/apache/incubator-pinot/pull/4339#issuecomment-503398156 > Should we make `maxParallelStateTransition` configurable? > > @jackjlli Can you comment if there was a

[incubator-pinot] branch master updated: Only set Helix cluster properties when creating the cluster (#4339)

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 9c49b40 Only set Helix cluster propert

[incubator-pinot] branch fix_state_transition_max_threads deleted (was 35dad73)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch fix_state_transition_max_threads in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 35dad73 Only set Helix cluster properties when creating the cluster The revision

[GitHub] [incubator-pinot] kishoreg commented on issue #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing

kishoreg commented on issue #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#issuecomment-503397221 Got it. Good idea. One concern with this is if one replica of a segment sta

[incubator-pinot] branch llc_routing_hotfix created (now 77954ec)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch llc_routing_hotfix in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 77954ec Within a partition, only allow querying the first CONSUMING segment for the real-time

[incubator-pinot] 01/01: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing

This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch llc_routing_hotfix in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 77954eccad9e4e5086c7edaa41235ef823a91d1a Author: Jackie (Xiaotian) Jiang AuthorDate: Tue Jun 18 18:

[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing

Jackie-Jiang commented on issue #4338: Within a partition, only allow querying the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#issuecomment-503393812 > I can’t follow. Can you review your comment again and see if there is

[incubator-pinot] branch consuming_routing updated (fa2a039 -> 194a969)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch consuming_routing in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard fa2a039 Only allow the first CONSUMING segment for the real-time table routing add 194a969

[GitHub] [incubator-pinot] codecov-io commented on issue #4339: Only set Helix cluster properties when creating the cluster

codecov-io commented on issue #4339: Only set Helix cluster properties when creating the cluster URL: https://github.com/apache/incubator-pinot/pull/4339#issuecomment-503390342 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4339?src=pr&el=h1) Report > :exclamation: No co

[GitHub] [incubator-pinot] codecov-io commented on issue #4338: Only allow the first CONSUMING segment for the real-time table routing

codecov-io commented on issue #4338: Only allow the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#issuecomment-503389816 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4338?src=pr&el=h1) Report > :exclama

[incubator-pinot] branch consuming_routing updated (5e49764 -> fa2a039)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch consuming_routing in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 5e49764 Only allow the first CONSUMING segment for the real-time table routing add fa2a039

[GitHub] [incubator-pinot] kishoreg commented on issue #4338: Only allow the first CONSUMING segment for the real-time table routing

kishoreg commented on issue #4338: Only allow the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#issuecomment-503381307 I can’t follow. Can you review your comment again and see if there is any typo. --

[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #4338: Only allow the first CONSUMING segment for the real-time table routing

Jackie-Jiang commented on issue #4338: Only allow the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#issuecomment-503379408 > Not sure if I follow the description. If for a given partition, there are 'n' replicas that are al

[GitHub] [incubator-pinot] mayankshriv commented on issue #4338: Only allow the first CONSUMING segment for the real-time table routing

mayankshriv commented on issue #4338: Only allow the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338#issuecomment-503377579 Not sure if I follow the description. If for a given partition, there are 'n' replicas that are all i

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #4339: Only set Helix cluster properties when creating the cluster

Jackie-Jiang opened a new pull request #4339: Only set Helix cluster properties when creating the cluster URL: https://github.com/apache/incubator-pinot/pull/4339 If Helix cluster does not exist, create the cluster with auto-join enabled Do not set the STATE_TRANSITION.maxThreads when cr

[incubator-pinot] 01/01: Only set Helix cluster properties when creating the cluster

This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch fix_state_transition_max_threads in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 35dad739b0e6dd2ed1d9f00a959835a076a57dd4 Author: Jackie (Xiaotian) Jiang AuthorDate:

[incubator-pinot] branch fix_state_transition_max_threads updated (dbbf398 -> 35dad73)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch fix_state_transition_max_threads in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard dbbf398 Only set Helix cluster properties when creating the cluster new 35da

[incubator-pinot] branch fix_state_transition_max_threads created (now dbbf398)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch fix_state_transition_max_threads in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at dbbf398 Only set Helix cluster properties when creating the cluster This branch

[incubator-pinot] 01/01: Only set Helix cluster properties when creating the cluster

This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch fix_state_transition_max_threads in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit dbbf3984d8d7aa25e70eeeae42fed7d17d3e3afa Author: Jackie (Xiaotian) Jiang AuthorDate:

[incubator-pinot] branch consuming_routing created (now 5e49764)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch consuming_routing in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 5e49764 Only allow the first CONSUMING segment for the real-time table routing No new revision

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #4338: Only allow the first CONSUMING segment for the real-time table routing

Jackie-Jiang opened a new pull request #4338: Only allow the first CONSUMING segment for the real-time table routing URL: https://github.com/apache/incubator-pinot/pull/4338 We only allow the first CONSUMING segment (segment with instances in CONSUMING state) for the following reasons:

[incubator-pinot] branch rebalance_llc created (now 82cbaca)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch rebalance_llc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 82cbaca Take OFFLINE segment into account for real-time rebalancer This branch includes the followi

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #4337: Take OFFLINE segment into account for real-time rebalancer

Jackie-Jiang opened a new pull request #4337: Take OFFLINE segment into account for real-time rebalancer URL: https://github.com/apache/incubator-pinot/pull/4337 When the consuming segment encounters error, the ideal state can be turned into OFFLINE If the instance states for a segment

[incubator-pinot] 01/01: Take OFFLINE segment into account for real-time rebalancer

This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch rebalance_llc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 82cbaca278eea6571f4cd44a2eff694b3f8dc4f3 Author: Jackie (Xiaotian) Jiang AuthorDate: Tue Jun 18 16:53:58

[incubator-pinot] 01/01: Always download realtime segments on non-winners

This is an automated email from the ASF dual-hosted git repository. nehapawar pushed a commit to branch always_download_realtime_segments in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 0990f83c4d22fdea2d160921311d7bd584714450 Author: Neha Pawar AuthorDate: Tue Jun 1

[incubator-pinot] branch always_download_realtime_segments created (now 0990f83)

This is an automated email from the ASF dual-hosted git repository. nehapawar pushed a change to branch always_download_realtime_segments in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 0990f83 Always download realtime segments on non-winners This branch include

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4336: [TE] detection dimensional cache

jihaozh opened a new pull request #4336: [TE] detection dimensional cache URL: https://github.com/apache/incubator-pinot/pull/4336 For high dimensional metrics, currently, the detection will send a query for each dimension and then run the anomaly detection. This PR implements an optimizat

[GitHub] [incubator-pinot] haibow closed pull request #4335: Refresh schema when reloading segments

haibow closed pull request #4335: Refresh schema when reloading segments URL: https://github.com/apache/incubator-pinot/pull/4335 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-pinot] haibow opened a new pull request #4335: Refresh schema when reloading segments

haibow opened a new pull request #4335: Refresh schema when reloading segments URL: https://github.com/apache/incubator-pinot/pull/4335 Now when reloading segments, schema is re-fetched only for OFFLINE table. For realtime tables, it will still reload with the old schema, ignoring new colu

[GitHub] [incubator-pinot] akshayrai edited a comment on issue #4332: When using jdbc mysql connection, Root Cause Analysis breaks

akshayrai edited a comment on issue #4332: When using jdbc mysql connection, Root Cause Analysis breaks URL: https://github.com/apache/incubator-pinot/issues/4332#issuecomment-503332658 @vasilije90 appreciate your efforts in narrowing down the issue. Have you configured the time for

[GitHub] [incubator-pinot] akshayrai edited a comment on issue #4332: When using jdbc mysql connection, Root Cause Analysis breaks

akshayrai edited a comment on issue #4332: When using jdbc mysql connection, Root Cause Analysis breaks URL: https://github.com/apache/incubator-pinot/issues/4332#issuecomment-503332658 @vasilije90 appreciate your efforts in narrowing down the issue. Have you configured the time for

[GitHub] [incubator-pinot] akshayrai commented on issue #4332: When using jdbc mysql connection, Root Cause Analysis breaks

akshayrai commented on issue #4332: When using jdbc mysql connection, Root Cause Analysis breaks URL: https://github.com/apache/incubator-pinot/issues/4332#issuecomment-503332658 @vasilije90 appreciate your efforts in narrowing down the issue. Have you configured the time format cor

[GitHub] [incubator-pinot] akshayrai commented on issue #4334: Pinot datasets not showing in thirdye UI

akshayrai commented on issue #4334: Pinot datasets not showing in thirdye UI URL: https://github.com/apache/incubator-pinot/issues/4334#issuecomment-503322539 @ShubhamMMT1 can you share the events you see in the logs? Any error or warning messages? ---

[GitHub] [incubator-pinot] haibow opened a new pull request #4335: Refresh schema when reloading segments

haibow opened a new pull request #4335: Refresh schema when reloading segments URL: https://github.com/apache/incubator-pinot/pull/4335 Now when reloading segments, schema is re-fetched only for OFFLINE table. For realtime tables, it will still reload with the old schema, ignoring new colu

[GitHub] [incubator-pinot] ShubhamMMT1 opened a new issue #4334: Pinot datasets not showing in thirdye UI

ShubhamMMT1 opened a new issue #4334: Pinot datasets not showing in thirdye UI URL: https://github.com/apache/incubator-pinot/issues/4334 I have integrated pinot into thirdeye using following connector configurations- - className: com.linkedin.thirdeye.datasource.pinot.PinotThirdEyeDataS

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #4330: Including https apache repo in repositories

codecov-io edited a comment on issue #4330: Including https apache repo in repositories URL: https://github.com/apache/incubator-pinot/pull/4330#issuecomment-502911692 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4330?src=pr&el=h1) Report > Merging [#4330](https://cod

[GitHub] [incubator-pinot] akshayrai opened a new pull request #4333: [TE] Entity Yaml (Composite Alert) Translator

akshayrai opened a new pull request #4333: [TE] Entity Yaml (Composite Alert) Translator URL: https://github.com/apache/incubator-pinot/pull/4333 Changes: - Introduced 2 types of alerts - METRIC_ALERT and COMPOSITE_ALERT (refer yaml examples) - Modified and refactored the DetectionCo

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

kishoreg commented on 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#issuecomment-503277146 Take a look at the code within MutableOffHeapByteArrayStore.Buffer. That's

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4322: [TE] detection health status

jihaozh commented on a change in pull request #4322: [TE] detection health status URL: https://github.com/apache/incubator-pinot/pull/4322#discussion_r294971749 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/DetectionHealth.java ##

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4322: [TE] detection health status

jihaozh commented on a change in pull request #4322: [TE] detection health status URL: https://github.com/apache/incubator-pinot/pull/4322#discussion_r294971585 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/DetectionHealth.java ##

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4322: [TE] detection health status

jihaozh commented on a change in pull request #4322: [TE] detection health status URL: https://github.com/apache/incubator-pinot/pull/4322#discussion_r294971266 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/DetectionResource.java

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4322: [TE] detection health status

jihaozh commented on a change in pull request #4322: [TE] detection health status URL: https://github.com/apache/incubator-pinot/pull/4322#discussion_r294971159 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/DetectionHealth.java ##

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4322: [TE] detection health status

jihaozh commented on a change in pull request #4322: [TE] detection health status URL: https://github.com/apache/incubator-pinot/pull/4322#discussion_r294971103 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/DetectionHealth.java ##

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4322: [TE] detection health status

jihaozh commented on a change in pull request #4322: [TE] detection health status URL: https://github.com/apache/incubator-pinot/pull/4322#discussion_r294971013 ## File path: thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/DetectionHealth.java ##

[incubator-pinot] branch master updated: [TE] Use MapUtils instead of ConfigUtils - return default instead of exception (#4331)

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 dd73957 [TE] Use MapUtils instead

[GitHub] [incubator-pinot] akshayrai merged pull request #4331: [TE] Use MapUtils instead of ConfigUtils - return default instead of …

akshayrai merged pull request #4331: [TE] Use MapUtils instead of ConfigUtils - return default instead of … URL: https://github.com/apache/incubator-pinot/pull/4331 This is an automated message from the Apache Git Service. T

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

mcvsubbu commented on 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#issuecomment-503243421 I am sure I am missing something, but I don't see why we should try to com

[incubator-pinot] 01/01: Deleting extra segments after push

This is an automated email from the ASF dual-hosted git repository. jenniferdai pushed a commit to branch deletePush in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 618e475005abe42165e3260241193d87118a39a5 Author: Jennifer Dai AuthorDate: Tue Jun 18 10:54:07 2019 -07

[incubator-pinot] branch deletePush created (now 618e475)

This is an automated email from the ASF dual-hosted git repository. jenniferdai pushed a change to branch deletePush in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 618e475 Deleting extra segments after push This branch includes the following new commits:

[GitHub] [incubator-pinot] vasilije90 opened a new issue #4332: When using jdbc mysql connection, Root Cause Analysis breaks

vasilije90 opened a new issue #4332: When using jdbc mysql connection, Root Cause Analysis breaks URL: https://github.com/apache/incubator-pinot/issues/4332 I tried connecting via jdbc to mysql database and fetching some daily data to test the Root Cause Analysis dashboard. There i

[incubator-pinot] branch repo-testing updated (51a4a26 -> 0549dd8)

This is an automated email from the ASF dual-hosted git repository. mcvsubbu pushed a change to branch repo-testing in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 51a4a26 Including https apache repo in repositories add 0549dd8 comment out http repos No ne

[GitHub] [incubator-pinot] akshayrai opened a new pull request #4331: [TE] Use MapUtils instead of ConfigUtils - return default instead of …

akshayrai opened a new pull request #4331: [TE] Use MapUtils instead of ConfigUtils - return default instead of … URL: https://github.com/apache/incubator-pinot/pull/4331 …exception This is an automated message from the Apach