[GitHub] [incubator-pinot] mayankshriv merged pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts.

2019-07-05 Thread GitBox
mayankshriv merged pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts. URL: https://github.com/apache/incubator-pinot/pull/4321 This is an automated message from

[incubator-pinot] branch master updated: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts. (#4321)

2019-07-05 Thread mayanks
This is an automated email from the ASF dual-hosted git repository. mayanks 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 49d8fa7 #4317 Feature/variable

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts.

2019-07-05 Thread GitBox
codecov-io edited a comment on issue #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts. URL: https://github.com/apache/incubator-pinot/pull/4321#issuecomment-507968394 #

[GitHub] [incubator-pinot] mcvsubbu commented on issue #3998: Upgrade to use Kafka release 2.1.1

2019-07-05 Thread GitBox
mcvsubbu commented on issue #3998: Upgrade to use Kafka release 2.1.1 URL: https://github.com/apache/incubator-pinot/issues/3998#issuecomment-508865713 PR #4397 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-pinot] buchireddy commented on a change in pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts.

2019-07-05 Thread GitBox
buchireddy commented on a change in pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts. URL: https://github.com/apache/incubator-pinot/pull/4321#discussion_r300798144 ## File path:

[GitHub] [incubator-pinot] buchireddy commented on a change in pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts.

2019-07-05 Thread GitBox
buchireddy commented on a change in pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts. URL: https://github.com/apache/incubator-pinot/pull/4321#discussion_r300774780 ## File path:

[GitHub] [incubator-pinot] buchireddy opened a new issue #4401: Avoid creating garbage in variable length bytes store for offline dictionary

2019-07-05 Thread GitBox
buchireddy opened a new issue #4401: Avoid creating garbage in variable length bytes store for offline dictionary URL: https://github.com/apache/incubator-pinot/issues/4401 This is coming from https://github.com/apache/incubator-pinot/pull/4321#discussion_r299724111 > Well, another

[GitHub] [incubator-pinot] buchireddy commented on a change in pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts.

2019-07-05 Thread GitBox
buchireddy commented on a change in pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts. URL: https://github.com/apache/incubator-pinot/pull/4321#discussion_r300765246 ## File path:

[GitHub] [incubator-pinot] buchireddy commented on a change in pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts.

2019-07-05 Thread GitBox
buchireddy commented on a change in pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts. URL: https://github.com/apache/incubator-pinot/pull/4321#discussion_r299708993 ## File path:

[GitHub] [incubator-pinot] buchireddy commented on a change in pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts.

2019-07-05 Thread GitBox
buchireddy commented on a change in pull request #4321: #4317 Feature/variable length bytes offline dictionary for indexing bytes and string dicts. URL: https://github.com/apache/incubator-pinot/pull/4321#discussion_r300764157 ## File path:

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #4388: Adding support for Map type fields

2019-07-05 Thread GitBox
mayankshriv commented on a change in pull request #4388: Adding support for Map type fields URL: https://github.com/apache/incubator-pinot/pull/4388#discussion_r300754240 ## File path:

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #4388: Adding support for Map type fields

2019-07-05 Thread GitBox
mayankshriv commented on a change in pull request #4388: Adding support for Map type fields URL: https://github.com/apache/incubator-pinot/pull/4388#discussion_r300756550 ## File path:

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #4388: Adding support for Map type fields

2019-07-05 Thread GitBox
mayankshriv commented on a change in pull request #4388: Adding support for Map type fields URL: https://github.com/apache/incubator-pinot/pull/4388#discussion_r300754468 ## File path:

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #4388: Adding support for Map type fields

2019-07-05 Thread GitBox
mayankshriv commented on a change in pull request #4388: Adding support for Map type fields URL: https://github.com/apache/incubator-pinot/pull/4388#discussion_r300755669 ## File path:

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #4388: Adding support for Map type fields

2019-07-05 Thread GitBox
mayankshriv commented on a change in pull request #4388: Adding support for Map type fields URL: https://github.com/apache/incubator-pinot/pull/4388#discussion_r300753959 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/readers/AvroRecordReader.java ##

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #4388: Adding support for Map type fields

2019-07-05 Thread GitBox
mayankshriv commented on a change in pull request #4388: Adding support for Map type fields URL: https://github.com/apache/incubator-pinot/pull/4388#discussion_r300755805 ## File path: pinot-core/src/main/java/org/apache/pinot/core/util/AvroUtils.java ## @@ -296,4 +302,33

[GitHub] [incubator-pinot] JackyYangPassion commented on issue #4400: what is the difference between Pinot and Druid

2019-07-05 Thread GitBox
JackyYangPassion commented on issue #4400: what is the difference between Pinot and Druid URL: https://github.com/apache/incubator-pinot/issues/4400#issuecomment-508684306 3Q This is an automated message from the Apache Git

[GitHub] [incubator-pinot] sunny209868 commented on issue #4400: what is the difference between Pinot and Druid

2019-07-05 Thread GitBox
sunny209868 commented on issue #4400: what is the difference between Pinot and Druid URL: https://github.com/apache/incubator-pinot/issues/4400#issuecomment-508670502 This article is very detailed.

[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.