[GitHub] [incubator-pinot] codecov-io edited a comment on issue #4545: Add APIs to get leader for all tables or a given table

2019-09-10 Thread GitBox
codecov-io edited a comment on issue #4545: Add APIs to get leader for all tables or a given table URL: https://github.com/apache/incubator-pinot/pull/4545#issuecomment-523237858 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4545?src=pr=h1) Report > Merging

[incubator-pinot] branch master updated (0a32309 -> e70a781)

2019-09-10 Thread snlee
This is an automated email from the ASF dual-hosted git repository. snlee pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 0a32309 [TE] fix create alert error message NPE (#4594) add e70a781 Add ApacheCon Slide to README

[incubator-pinot] branch add-apachecon-slide deleted (was efe45f2)

2019-09-10 Thread snlee
This is an automated email from the ASF dual-hosted git repository. snlee pushed a change to branch add-apachecon-slide in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was efe45f2 Add ApacheCon Slide to README file The revisions that were on this branch are still

[GitHub] [incubator-pinot] snleee merged pull request #4604: Add ApacheCon Slide to README file

2019-09-10 Thread GitBox
snleee merged pull request #4604: Add ApacheCon Slide to README file URL: https://github.com/apache/incubator-pinot/pull/4604 This is an automated message from the Apache Git Service. To respond to the message, please log on

[incubator-pinot] 01/01: Add ApacheCon Slide to README file

2019-09-10 Thread snlee
This is an automated email from the ASF dual-hosted git repository. snlee pushed a commit to branch add-apachecon-slide in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit efe45f2c912188570ce5aae54a3af091cf77feda Author: Seunghyun Lee AuthorDate: Tue Sep 10 17:22:35

[GitHub] [incubator-pinot] snleee opened a new pull request #4604: Add ApacheCon Slide to README file

2019-09-10 Thread GitBox
snleee opened a new pull request #4604: Add ApacheCon Slide to README file URL: https://github.com/apache/incubator-pinot/pull/4604 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-pinot] branch add-apachecon-slide created (now efe45f2)

2019-09-10 Thread snlee
This is an automated email from the ASF dual-hosted git repository. snlee pushed a change to branch add-apachecon-slide in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at efe45f2 Add ApacheCon Slide to README file This branch includes the following new commits:

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters

2019-09-10 Thread GitBox
akshayrai commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters URL: https://github.com/apache/incubator-pinot/pull/4599#discussion_r322978657 ## File path:

[GitHub] [incubator-pinot] codecov-io commented on issue #4602: First pass of GROUP BY with ORDER BY support

2019-09-10 Thread GitBox
codecov-io commented on issue #4602: First pass of GROUP BY with ORDER BY support URL: https://github.com/apache/incubator-pinot/pull/4602#issuecomment-530129911 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4602?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] mcvsubbu commented on issue #4597: [WIP] Adding json_path function to extract field from json object

2019-09-10 Thread GitBox
mcvsubbu commented on issue #4597: [WIP] Adding json_path function to extract field from json object URL: https://github.com/apache/incubator-pinot/pull/4597#issuecomment-530110310 Can you also add documentation on how json columns are to be used? I suppose the json columns cannot be used

[GitHub] [incubator-pinot] xiaohui-sun commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters

2019-09-10 Thread GitBox
xiaohui-sun commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters URL: https://github.com/apache/incubator-pinot/pull/4599#discussion_r322943580 ## File path:

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #4597: [WIP] Adding json_path function to extract field from json object

2019-09-10 Thread GitBox
fx19880617 commented on a change in pull request #4597: [WIP] Adding json_path function to extract field from json object URL: https://github.com/apache/incubator-pinot/pull/4597#discussion_r322941099 ## File path:

[incubator-pinot] branch jsonpath_function updated (af7bdc1 -> 21e2810)

2019-09-10 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch jsonpath_function in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard af7bdc1 Adding json_path function to extract field from json object add 21e2810 Adding

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters

2019-09-10 Thread GitBox
akshayrai commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters URL: https://github.com/apache/incubator-pinot/pull/4599#discussion_r322912489 ## File path:

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters

2019-09-10 Thread GitBox
akshayrai commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters URL: https://github.com/apache/incubator-pinot/pull/4599#discussion_r322912504 ## File path:

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters

2019-09-10 Thread GitBox
akshayrai commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters URL: https://github.com/apache/incubator-pinot/pull/4599#discussion_r322912541 ## File path:

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters

2019-09-10 Thread GitBox
akshayrai commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters URL: https://github.com/apache/incubator-pinot/pull/4599#discussion_r322912486 ## File path:

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table

2019-09-10 Thread GitBox
jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table URL: https://github.com/apache/incubator-pinot/pull/4545#discussion_r322898604 ## File path:

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table

2019-09-10 Thread GitBox
jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table URL: https://github.com/apache/incubator-pinot/pull/4545#discussion_r322898513 ## File path:

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table

2019-09-10 Thread GitBox
jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table URL: https://github.com/apache/incubator-pinot/pull/4545#discussion_r322898462 ## File path:

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table

2019-09-10 Thread GitBox
jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table URL: https://github.com/apache/incubator-pinot/pull/4545#discussion_r322898204 ## File path:

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table

2019-09-10 Thread GitBox
jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table URL: https://github.com/apache/incubator-pinot/pull/4545#discussion_r322898170 ## File path:

[GitHub] [incubator-pinot] jihaozh merged pull request #4594: [TE] fix create alert error message null pointer exception

2019-09-10 Thread GitBox
jihaozh merged pull request #4594: [TE] fix create alert error message null pointer exception URL: https://github.com/apache/incubator-pinot/pull/4594 This is an automated message from the Apache Git Service. To respond to

[incubator-pinot] branch master updated (5e525d0 -> 0a32309)

2019-09-10 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 5e525d0 [TE] Exclude template/blacklist emails before sending out alert (#4601) add 0a32309 [TE] fix

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4597: [WIP] Adding json_path function to extract field from json object

2019-09-10 Thread GitBox
mcvsubbu commented on a change in pull request #4597: [WIP] Adding json_path function to extract field from json object URL: https://github.com/apache/incubator-pinot/pull/4597#discussion_r322853507 ## File path:

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters

2019-09-10 Thread GitBox
jihaozh commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters URL: https://github.com/apache/incubator-pinot/pull/4599#discussion_r322499005 ## File path:

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters

2019-09-10 Thread GitBox
jihaozh commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters URL: https://github.com/apache/incubator-pinot/pull/4599#discussion_r322873181 ## File path:

[GitHub] [incubator-pinot] jihaozh commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters

2019-09-10 Thread GitBox
jihaozh commented on a change in pull request #4599: [TE] Redesign to support multiple Alert Content formatters URL: https://github.com/apache/incubator-pinot/pull/4599#discussion_r322875117 ## File path:

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table

2019-09-10 Thread GitBox
jackjlli commented on a change in pull request #4545: Add APIs to get leader for all tables or a given table URL: https://github.com/apache/incubator-pinot/pull/4545#discussion_r322872808 ## File path:

[incubator-pinot] branch master updated: [TE] Exclude template/blacklist emails before sending out alert (#4601)

2019-09-10 Thread akshayrai09
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 5e525d0 [TE] Exclude

[GitHub] [incubator-pinot] akshayrai merged pull request #4601: [TE] Exclude template/blacklist emails before sending out alert

2019-09-10 Thread GitBox
akshayrai merged pull request #4601: [TE] Exclude template/blacklist emails before sending out alert URL: https://github.com/apache/incubator-pinot/pull/4601 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-pinot] chenboat edited a comment on issue #4525: Pinot server replacement in Cloud where we have no control over hostname

2019-09-10 Thread GitBox
chenboat edited a comment on issue #4525: Pinot server replacement in Cloud where we have no control over hostname URL: https://github.com/apache/incubator-pinot/issues/4525#issuecomment-529774810 Currently when a Pinot server starts and joins the Helix cluster, it calls HelixManager's

[GitHub] [incubator-pinot] chenboat edited a comment on issue #4525: Pinot server replacement in Cloud where we have no control over hostname

2019-09-10 Thread GitBox
chenboat edited a comment on issue #4525: Pinot server replacement in Cloud where we have no control over hostname URL: https://github.com/apache/incubator-pinot/issues/4525#issuecomment-529774810 Currently when a Pinot server starts and joins the Helix cluster, it calls HelixManager's