[GitHub] [incubator-pinot] fx19880617 merged pull request #5210: fixing presto-coordinator k8s yaml file

2020-04-06 Thread GitBox
fx19880617 merged pull request #5210: fixing presto-coordinator k8s yaml file URL: https://github.com/apache/incubator-pinot/pull/5210 This is an automated message from the Apache Git Service. To respond to the message,

[incubator-pinot] branch master updated (8278cbc -> 20afda5)

2020-04-06 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 8278cbc Add gcs nio support (#5106) add 20afda5 fixing presto-coordinator k8s yaml file (#5210) No

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5176: Make text index query cache a configurable option

2020-04-06 Thread GitBox
siddharthteotia commented on a change in pull request #5176: Make text index query cache a configurable option URL: https://github.com/apache/incubator-pinot/pull/5176#discussion_r404282218 ## File path:

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5176: Make text index query cache a configurable option

2020-04-06 Thread GitBox
siddharthteotia commented on a change in pull request #5176: Make text index query cache a configurable option URL: https://github.com/apache/incubator-pinot/pull/5176#discussion_r404282234 ## File path:

[incubator-pinot] branch master updated: [TE] Retain the original last success task execution time if we can't… (#5207)

2020-04-06 Thread xhsun
This is an automated email from the ASF dual-hosted git repository. xhsun 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 610350a [TE] Retain the original last

[GitHub] [incubator-pinot] xiaohui-sun merged pull request #5207: [TE] Retain the original last success task execution time if we can't…

2020-04-06 Thread GitBox
xiaohui-sun merged pull request #5207: [TE] Retain the original last success task execution time if we can't… URL: https://github.com/apache/incubator-pinot/pull/5207 This is an automated message from the Apache Git

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #5176: Make text index query cache a configurable option

2020-04-06 Thread GitBox
codecov-io edited a comment on issue #5176: Make text index query cache a configurable option URL: https://github.com/apache/incubator-pinot/pull/5176#issuecomment-603045784 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5176?src=pr=h1) Report > Merging

[incubator-pinot] branch show_messages_as_error created (now 6e3abce)

2020-04-06 Thread xhsun
This is an automated email from the ASF dual-hosted git repository. xhsun pushed a change to branch show_messages_as_error in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 6e3abce [TE] Show error messages to the new UI This branch includes the following new

[incubator-pinot] 01/01: [TE] Show error messages to the new UI

2020-04-06 Thread xhsun
This is an automated email from the ASF dual-hosted git repository. xhsun pushed a commit to branch show_messages_as_error in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 6e3abcef8d1ec4b2484873500020b1d0cf632152 Author: Xiaohui Sun AuthorDate: Mon Apr 6 11:27:40

[GitHub] [incubator-pinot] xiaohui-sun opened a new pull request #5211: [TE] Show error messages to the new UI

2020-04-06 Thread GitBox
xiaohui-sun opened a new pull request #5211: [TE] Show error messages to the new UI URL: https://github.com/apache/incubator-pinot/pull/5211 Adjust the return error messages to UI so it could be shown in the new error panel.

[GitHub] [incubator-pinot] siddharthteotia merged pull request #5176: Make text index query cache a configurable option

2020-04-06 Thread GitBox
siddharthteotia merged pull request #5176: Make text index query cache a configurable option URL: https://github.com/apache/incubator-pinot/pull/5176 This is an automated message from the Apache Git Service. To respond to

[incubator-pinot] branch master updated (610350a -> 0d5b5e2)

2020-04-06 Thread siddteotia
This is an automated email from the ASF dual-hosted git repository. siddteotia pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 610350a [TE] Retain the original last success task execution time if we can't… (#5207) add

[GitHub] [incubator-pinot] chenboat commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion

2020-04-06 Thread GitBox
chenboat commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion URL: https://github.com/apache/incubator-pinot/pull/4914#issuecomment-610066213 > In phase-2, we will do: > > 1. Intoduce a table config (inside stream config? or maybe in

[GitHub] [incubator-pinot] harleyjj opened a new pull request #5212: [TE] endpoint - harleyjj/metricBreakdownPipeline - add flag for ignor…

2020-04-06 Thread GitBox
harleyjj opened a new pull request #5212: [TE] endpoint - harleyjj/metricBreakdownPipeline - add flag for ignor… URL: https://github.com/apache/incubator-pinot/pull/5212 …ing score when adding dimensions to response This is

[GitHub] [incubator-pinot] akshayrai commented on issue #5211: [TE] Show error messages to the new UI

2020-04-06 Thread GitBox
akshayrai commented on issue #5211: [TE] Show error messages to the new UI URL: https://github.com/apache/incubator-pinot/pull/5211#issuecomment-610051294 Is the UI change expecting this new field `error`? This is an

[GitHub] [incubator-pinot] akshayrai edited a comment on issue #5211: [TE] Show error messages to the new UI

2020-04-06 Thread GitBox
akshayrai edited a comment on issue #5211: [TE] Show error messages to the new UI URL: https://github.com/apache/incubator-pinot/pull/5211#issuecomment-610051294 Is the UI expecting this new field `error`? This is an

[GitHub] [incubator-pinot] akshayrai edited a comment on issue #5211: [TE] Show error messages to the new UI

2020-04-06 Thread GitBox
akshayrai edited a comment on issue #5211: [TE] Show error messages to the new UI URL: https://github.com/apache/incubator-pinot/pull/5211#issuecomment-610051294 Is the UI expecting this new field `error` in the response? Has that change been made?

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #5213: Support order-by on BYTES column

2020-04-06 Thread GitBox
Jackie-Jiang opened a new pull request #5213: Support order-by on BYTES column URL: https://github.com/apache/incubator-pinot/pull/5213 In order to support order-by on BYTES column everywhere, inside the system we should always use ByteArray (comparable) to store the BYTES value.

[GitHub] [incubator-pinot] mcvsubbu edited a comment on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion

2020-04-06 Thread GitBox
mcvsubbu edited a comment on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion URL: https://github.com/apache/incubator-pinot/pull/4914#issuecomment-610074062 The config change in step 5 is because we want the segment split-committer to look like this:

[GitHub] [incubator-pinot] mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion

2020-04-06 Thread GitBox
mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion URL: https://github.com/apache/incubator-pinot/pull/4914#issuecomment-610097015 @chenboat I believe @Jackie-Jiang suggested we don't go with another class, but with a table config

[GitHub] [incubator-pinot] snleee opened a new pull request #5214: Change the logo for navbar on the website

2020-04-06 Thread GitBox
snleee opened a new pull request #5214: Change the logo for navbar on the website URL: https://github.com/apache/incubator-pinot/pull/5214 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-pinot] mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion

2020-04-06 Thread GitBox
mcvsubbu commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion URL: https://github.com/apache/incubator-pinot/pull/4914#issuecomment-610074062 The config change in step 5 is because we want the segment split-committer to look like this: `

[GitHub] [incubator-pinot] mcvsubbu edited a comment on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion

2020-04-06 Thread GitBox
mcvsubbu edited a comment on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion URL: https://github.com/apache/incubator-pinot/pull/4914#issuecomment-610074062 The config change in step 5 is because we want the segment split-committer to look like this:

[GitHub] [incubator-pinot] chenboat commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion

2020-04-06 Thread GitBox
chenboat commented on issue #4914: [POC] By-passing deep-store requirement for Realtime segment completion URL: https://github.com/apache/incubator-pinot/pull/4914#issuecomment-610080673 Make sense. I thought you were talking about controller side change. I think the second option (i.e.,

[incubator-pinot] branch pr/5214 updated (79aab29 -> 33c680d)

2020-04-06 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch pr/5214 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 79aab29 Update package.json add 4ee35c3 Change the logo for navbar on the website (#5214) add

[incubator-pinot] branch pr/5214 updated (d45013a -> 249e48f)

2020-04-06 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch pr/5214 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard d45013a Update package.json add 249e48f Update package.json to resolve sharp version to fix the

[incubator-pinot] 01/01: Update package.json

2020-04-06 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch pr/5214 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 79aab295a7bbfff95a4727e022fb33f2d7a916e4 Author: Xiang Fu AuthorDate: Mon Apr 6 19:36:10 2020 -0700

[incubator-pinot] branch pr/5214 created (now 79aab29)

2020-04-06 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch pr/5214 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 79aab29 Update package.json This branch includes the following new commits: new 79aab29 Update

[GitHub] [incubator-pinot] fx19880617 opened a new pull request #5215: Fix Pinot website build script

2020-04-06 Thread GitBox
fx19880617 opened a new pull request #5215: Fix Pinot website build script URL: https://github.com/apache/incubator-pinot/pull/5215 By specifying the lib version of `sharp` to `^0.25.2` This is an automated message from the

[incubator-pinot] branch master updated: Change the logo for navbar on the website (#5214)

2020-04-06 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu 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 4ee35c3 Change the logo for navbar

[GitHub] [incubator-pinot] fx19880617 merged pull request #5214: Change the logo for navbar on the website

2020-04-06 Thread GitBox
fx19880617 merged pull request #5214: Change the logo for navbar on the website URL: https://github.com/apache/incubator-pinot/pull/5214 This is an automated message from the Apache Git Service. To respond to the message,

[incubator-pinot] branch pr/5214 updated (33c680d -> d45013a)

2020-04-06 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch pr/5214 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 33c680d Merge branch 'pr/5214' of https://github.com/apache/incubator-pinot into pr/5214 discard