[GitHub] [incubator-pinot] sunithabeeram commented on issue #4380: Define SLO config for tables

2019-08-15 Thread GitBox
sunithabeeram commented on issue #4380: Define SLO config for tables URL: https://github.com/apache/incubator-pinot/pull/4380#issuecomment-521829184 Closing this as this has been moved to higher layers. This is an automated

[GitHub] [incubator-pinot] sunithabeeram commented on issue #4380: Define SLO config for tables

2019-07-14 Thread GitBox
sunithabeeram commented on issue #4380: Define SLO config for tables URL: https://github.com/apache/incubator-pinot/pull/4380#issuecomment-511248076 @kishoreg sorry about the delay in responses on my end as I am traveling. Can you elaborate on what you mean by guaranteeing the

[GitHub] [incubator-pinot] sunithabeeram commented on issue #4380: Define SLO config for tables

2019-07-09 Thread GitBox
sunithabeeram commented on issue #4380: Define SLO config for tables URL: https://github.com/apache/incubator-pinot/pull/4380#issuecomment-509900678 @kishoreg - do you have further thoughts/comments on this? This is an

[GitHub] [incubator-pinot] sunithabeeram commented on issue #4380: Define SLO config for tables

2019-07-05 Thread GitBox
sunithabeeram commented on issue #4380: Define SLO config for tables URL: https://github.com/apache/incubator-pinot/pull/4380#issuecomment-508666033 @kishoreg these are required in the table config as they are per table parameters. The broker emits metrics that track against these numbers.