[incubator-pinot] branch master updated: [TE] Fix alert owner; use consistent filter convention (#4017)

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 d000501 [TE] Fix alert owner;

[GitHub] [incubator-pinot] akshayrai merged pull request #4017: [TE] Fix alert owner; use consistent filter convention

akshayrai merged pull request #4017: [TE] Fix alert owner; use consistent filter convention URL: https://github.com/apache/incubator-pinot/pull/4017 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4010: Add docs for record reader

Jackie-Jiang commented on a change in pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#discussion_r268872716 ## File path: docs/record_reader.rst ## @@ -0,0 +1,133 @@ +.. +.. Licensed to the Apache Software Foundation

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4010: Add docs for record reader

Jackie-Jiang commented on a change in pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#discussion_r268872701 ## File path: docs/record_reader.rst ## @@ -0,0 +1,133 @@ +.. +.. Licensed to the Apache Software Foundation

[incubator-pinot] branch record_reader_doc updated (27dad76 -> 2711d3d)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch record_reader_doc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 27dad76 Add docs for record reader new d58f8bc [TE] Fix ArrayIndexOutOfBoundsException;

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support

sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#discussion_r268866807 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ##

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support

sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#discussion_r268870590 ## File path: pinot-orc/src/test/java/org/apache/pinot/orc/data/readers/ORCRecordReaderTest.java

[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #4010: Add docs for record reader

Jackie-Jiang commented on issue #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#issuecomment-476400182 @sunithabeeram Addressed the comments. The travis failure is from a timeout which is caused by too less CPU cores running the tests.

[incubator-pinot] branch master updated: [TE] detection - current & baseline provider lazy evaluation (#4014)

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 1870c36 [TE] detection - current &

[GitHub] [incubator-pinot] jihaozh merged pull request #4014: [TE] detection - current & baseline provider lazy evaluation

jihaozh merged pull request #4014: [TE] detection - current & baseline provider lazy evaluation URL: https://github.com/apache/incubator-pinot/pull/4014 This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4018: [TE] detection - zero divide fix

jihaozh opened a new pull request #4018: [TE] detection - zero divide fix URL: https://github.com/apache/incubator-pinot/pull/4018 - Fix 0.0 divide - Refactor anomaly baseline & current value filling This is an automated

[GitHub] [incubator-pinot] ly923976094 commented on issue #4006: Controller web cannot create and update schemas

ly923976094 commented on issue #4006: Controller web cannot create and update schemas URL: https://github.com/apache/incubator-pinot/pull/4006#issuecomment-476460134 > A couple of general notes: please always add unit and integration tests to ensure: > a) Your code works as expected.

[GitHub] [incubator-pinot] ly923976094 edited a comment on issue #4006: Controller web cannot create and update schemas

ly923976094 edited a comment on issue #4006: Controller web cannot create and update schemas URL: https://github.com/apache/incubator-pinot/pull/4006#issuecomment-476460134 > A couple of general notes: please always add unit and integration tests to ensure: > a) Your code works as

[incubator-pinot] branch orcMV updated: Addressing comments

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

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4010: Add docs for record reader

Jackie-Jiang commented on a change in pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#discussion_r268874720 ## File path: docs/record_reader.rst ## @@ -0,0 +1,133 @@ +.. +.. Licensed to the Apache Software Foundation

[incubator-pinot] branch master updated: Add docs for record reader (#4010)

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 a4a3571 Add docs for record reader

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #4010: Add docs for record reader

Jackie-Jiang commented on a change in pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#discussion_r268876190 ## File path: docs/record_reader.rst ## @@ -0,0 +1,133 @@ +.. +.. Licensed to the Apache Software Foundation

[GitHub] [incubator-pinot] ly923976094 commented on a change in pull request #4006: Controller web cannot create and update schemas

ly923976094 commented on a change in pull request #4006: Controller web cannot create and update schemas URL: https://github.com/apache/incubator-pinot/pull/4006#discussion_r268930480 ## File path:

[GitHub] [incubator-pinot] ly923976094 commented on a change in pull request #4006: Controller web cannot create and update schemas

ly923976094 commented on a change in pull request #4006: Controller web cannot create and update schemas URL: https://github.com/apache/incubator-pinot/pull/4006#discussion_r268930480 ## File path:

[incubator-pinot] branch master updated: Added a percent threshold to consider startup of services (#4011)

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 a0011df Added a percent threshold

[GitHub] [incubator-pinot] mcvsubbu merged pull request #4011: Added a percent threshold to consider startup of services

mcvsubbu merged pull request #4011: Added a percent threshold to consider startup of services URL: https://github.com/apache/incubator-pinot/pull/4011 This is an automated message from the Apache Git Service. To respond to

[incubator-pinot] branch master updated: [TE] frontend - harleyjj/yaml-editor - set subscription group by default in edit mode (#4016)

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 1248705 [TE] frontend -

[GitHub] [incubator-pinot] akshayrai merged pull request #4016: [TE] frontend - harleyjj/yaml-editor - set subscription group by defa…

akshayrai merged pull request #4016: [TE] frontend - harleyjj/yaml-editor - set subscription group by defa… URL: https://github.com/apache/incubator-pinot/pull/4016 This is an automated message from the Apache Git Service.

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #4010: Add docs for record reader

codecov-io edited a comment on issue #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#issuecomment-476359577 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4010?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-io commented on issue #4009: Adding ORC Multi-value Column Support

codecov-io commented on issue #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#issuecomment-476416964 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4009?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] akshayrai opened a new pull request #4017: [TE] Fix alert owner; use consistent filter convention

akshayrai opened a new pull request #4017: [TE] Fix alert owner; use consistent filter convention URL: https://github.com/apache/incubator-pinot/pull/4017 This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4010: Add docs for record reader

sunithabeeram commented on a change in pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#discussion_r268897305 ## File path: docs/record_reader.rst ## @@ -0,0 +1,104 @@ +.. +.. Licensed to the Apache Software

[incubator-pinot] branch orcMV deleted (was 58d1a2d)

This is an automated email from the ASF dual-hosted git repository. jenniferdai pushed a change to branch orcMV in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 58d1a2d Addressing comments The revisions that were on this branch are still contained in other

[GitHub] [incubator-pinot] ly923976094 commented on a change in pull request #4006: Controller web cannot create and update schemas

ly923976094 commented on a change in pull request #4006: Controller web cannot create and update schemas URL: https://github.com/apache/incubator-pinot/pull/4006#discussion_r268930480 ## File path:

[GitHub] [incubator-pinot] harleyjj opened a new pull request #4016: [TE] frontend - harleyjj/yaml-editor - set subscription group by defa…

harleyjj opened a new pull request #4016: [TE] frontend - harleyjj/yaml-editor - set subscription group by defa… URL: https://github.com/apache/incubator-pinot/pull/4016 …ult in edit mode 1) Sets subscription group in yaml editor by default in edit mode (Edit Alert) 2) Adds some

[incubator-pinot] branch orcMV updated (d257661 -> 58d1a2d)

This is an automated email from the ASF dual-hosted git repository. jenniferdai pushed a change to branch orcMV in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard d257661 Addressing comments new 58d1a2d Addressing comments This update added new revisions

[incubator-pinot] branch master updated: Adding ORC Multi-value Column Support (#4009)

This is an automated email from the ASF dual-hosted git repository. jenniferdai 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 55f68ce Adding ORC Multi-value

[GitHub] [incubator-pinot] jenniferdai merged pull request #4009: Adding ORC Multi-value Column Support

jenniferdai merged pull request #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-pinot] akshayrai opened a new pull request #4015: [TE] Set anomaly duration upper limit to 7 days

akshayrai opened a new pull request #4015: [TE] Set anomaly duration upper limit to 7 days URL: https://github.com/apache/incubator-pinot/pull/4015 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-pinot] akshayrai merged pull request #4015: [TE] Set anomaly duration upper limit to 7 days

akshayrai merged pull request #4015: [TE] Set anomaly duration upper limit to 7 days URL: https://github.com/apache/incubator-pinot/pull/4015 This is an automated message from the Apache Git Service. To respond to the

[incubator-pinot] branch master updated: [TE] Set anomaly duration upper limit to 7 days (#4015)

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 ccb8d58 [TE] Set anomaly

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4010: Add docs for record reader

sunithabeeram commented on a change in pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#discussion_r268897511 ## File path: docs/record_reader.rst ## @@ -0,0 +1,104 @@ +.. +.. Licensed to the Apache Software

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #4010: Add docs for record reader

Jackie-Jiang merged pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010 This is an automated message from the Apache Git Service. To respond to the message, please log on

[incubator-pinot] branch record_reader_doc deleted (was 2711d3d)

This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch record_reader_doc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 2711d3d Address comments The revisions that were on this branch are still contained in other

[GitHub] [incubator-pinot] sunithabeeram opened a new pull request #4012: Update contribution guidelines

sunithabeeram opened a new pull request #4012: Update contribution guidelines URL: https://github.com/apache/incubator-pinot/pull/4012 Adds more info around testing done for contributions. This is an automated message from

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support

sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#discussion_r268640918 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ##

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support

sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#discussion_r268641325 ## File path: pinot-orc/src/test/java/org/apache/pinot/orc/data/readers/ORCRecordReaderTest.java

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support

sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#discussion_r268642597 ## File path: pinot-orc/src/test/java/org/apache/pinot/orc/data/readers/ORCRecordReaderTest.java

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support

sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#discussion_r268642057 ## File path: pinot-orc/src/test/java/org/apache/pinot/orc/data/readers/ORCRecordReaderTest.java

[GitHub] [incubator-pinot] sunithabeeram commented on issue #4009: Adding ORC Multi-value Column Support

sunithabeeram commented on issue #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#issuecomment-476204196 Can you please check on travis failure? Create a flaky test tracker/issue if needed.

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4011: Added a percent threshold to consider startup of services

sunithabeeram commented on a change in pull request #4011: Added a percent threshold to consider startup of services URL: https://github.com/apache/incubator-pinot/pull/4011#discussion_r268653547 ## File path:

[GitHub] [incubator-pinot] sunithabeeram commented on issue #4010: Add docs for record reader

sunithabeeram commented on issue #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#issuecomment-476203810 Given this is a doc change, travis shouldn't have failed. Can you please follow the failure and create a Flaky test issue/tracker if required?

[GitHub] [incubator-pinot] sunithabeeram commented on issue #4006: Controller web cannot create and update schemas

sunithabeeram commented on issue #4006: Controller web cannot create and update schemas URL: https://github.com/apache/incubator-pinot/pull/4006#issuecomment-476217198 A couple of general notes: please always add unit and integration tests to ensure: a) Your code works as expected.

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4006: Controller web cannot create and update schemas

sunithabeeram commented on a change in pull request #4006: Controller web cannot create and update schemas URL: https://github.com/apache/incubator-pinot/pull/4006#discussion_r268659691 ## File path:

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support

sunithabeeram commented on a change in pull request #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#discussion_r268641178 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ##

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4010: Add docs for record reader

sunithabeeram commented on a change in pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#discussion_r268644244 ## File path: docs/record_reader.rst ## @@ -0,0 +1,133 @@ +.. +.. Licensed to the Apache Software

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4010: Add docs for record reader

sunithabeeram commented on a change in pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#discussion_r268645511 ## File path: docs/record_reader.rst ## @@ -0,0 +1,133 @@ +.. +.. Licensed to the Apache Software

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4010: Add docs for record reader

sunithabeeram commented on a change in pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#discussion_r268646501 ## File path: docs/record_reader.rst ## @@ -0,0 +1,133 @@ +.. +.. Licensed to the Apache Software

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4010: Add docs for record reader

sunithabeeram commented on a change in pull request #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#discussion_r268645013 ## File path: docs/record_reader.rst ## @@ -0,0 +1,133 @@ +.. +.. Licensed to the Apache Software

[incubator-pinot] branch orcMV updated: Addressing comments

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

[GitHub] [incubator-pinot] codecov-io commented on issue #4012: Update contribution guidelines

codecov-io commented on issue #4012: Update contribution guidelines URL: https://github.com/apache/incubator-pinot/pull/4012#issuecomment-476245155 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4012?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #4011: Added a percent threshold to consider startup of services

sunithabeeram commented on a change in pull request #4011: Added a percent threshold to consider startup of services URL: https://github.com/apache/incubator-pinot/pull/4011#discussion_r268731776 ## File path:

[GitHub] [incubator-pinot] jenniferdai opened a new issue #4013: Flaky Test List from https://github.com/apache/incubator-pinot/pull/4009#issuecomment-476204196

jenniferdai opened a new issue #4013: Flaky Test List from https://github.com/apache/incubator-pinot/pull/4009#issuecomment-476204196 URL: https://github.com/apache/incubator-pinot/issues/4013 org.apache.pinot.integration.tests.FlakyConsumerRealtimeClusterIntegrationTest

[GitHub] [incubator-pinot] jenniferdai commented on a change in pull request #4009: Adding ORC Multi-value Column Support

jenniferdai commented on a change in pull request #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#discussion_r268741910 ## File path: pinot-orc/src/test/java/org/apache/pinot/orc/data/readers/ORCRecordReaderTest.java

[GitHub] [incubator-pinot] jenniferdai commented on a change in pull request #4009: Adding ORC Multi-value Column Support

jenniferdai commented on a change in pull request #4009: Adding ORC Multi-value Column Support URL: https://github.com/apache/incubator-pinot/pull/4009#discussion_r268737165 ## File path: pinot-orc/src/test/java/org/apache/pinot/orc/data/readers/ORCRecordReaderTest.java

[GitHub] [incubator-pinot] sunithabeeram commented on issue #4013: Flaky Test List from https://github.com/apache/incubator-pinot/pull/4009#issuecomment-476204196

sunithabeeram commented on issue #4013: Flaky Test List from https://github.com/apache/incubator-pinot/pull/4009#issuecomment-476204196 URL: https://github.com/apache/incubator-pinot/issues/4013#issuecomment-476286456 @jenniferdai did you encounter 2 failed tests? Do you have the trace

[GitHub] [incubator-pinot] codecov-io commented on issue #4011: Added a percent threshold to consider startup of services

codecov-io commented on issue #4011: Added a percent threshold to consider startup of services URL: https://github.com/apache/incubator-pinot/pull/4011#issuecomment-476316958 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4011?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4011: Added a percent threshold to consider startup of services

mcvsubbu commented on a change in pull request #4011: Added a percent threshold to consider startup of services URL: https://github.com/apache/incubator-pinot/pull/4011#discussion_r268761098 ## File path: pinot-common/src/main/java/org/apache/pinot/common/utils/ServiceStatus.java

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4011: Added a percent threshold to consider startup of services

mcvsubbu commented on a change in pull request #4011: Added a percent threshold to consider startup of services URL: https://github.com/apache/incubator-pinot/pull/4011#discussion_r268760987 ## File path: pinot-common/src/main/java/org/apache/pinot/common/utils/ServiceStatus.java

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4014: [TE] detection - current & baseline provider lazy evaluation

jihaozh opened a new pull request #4014: [TE] detection - current & baseline provider lazy evaluation URL: https://github.com/apache/incubator-pinot/pull/4014 - Update the anomaly current & baseline value only when the anomaly is merged. - Clean up current and baseline loader

[GitHub] [incubator-pinot] snleee commented on issue #3852: add support for parquet reader

snleee commented on issue #3852: add support for parquet reader URL: https://github.com/apache/incubator-pinot/pull/3852#issuecomment-476350541 @WangGuangxin Travis test is failing. Can you double check on this? Once that passes, I will merge the pr. Thank you for working on this!

[GitHub] [incubator-pinot] codecov-io commented on issue #4010: Add docs for record reader

codecov-io commented on issue #4010: Add docs for record reader URL: https://github.com/apache/incubator-pinot/pull/4010#issuecomment-476359577 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4010?src=pr=h1) Report > Merging