[GitHub] [incubator-pinot] ly923976094 commented on issue #3978: The segments are stored in memory

ly923976094 commented on issue #3978: The segments are stored in memory URL: https://github.com/apache/incubator-pinot/issues/3978#issuecomment-475469250 Do you mean the loading segment as shown below?

[GitHub] [incubator-pinot] harleyjj commented on a change in pull request #4004: [TE] frontend - harleyjj/yaml-editor - increases toast timeout from 4…

harleyjj commented on a change in pull request #4004: [TE] frontend - harleyjj/yaml-editor - increases toast timeout from 4… URL: https://github.com/apache/incubator-pinot/pull/4004#discussion_r268003885 ## File path:

[GitHub] [incubator-pinot] codecov-io commented on issue #4002: Added documentation regarding design docs

codecov-io commented on issue #4002: Added documentation regarding design docs URL: https://github.com/apache/incubator-pinot/pull/4002#issuecomment-475449546 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4002?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] harleyjj commented on a change in pull request #4004: [TE] frontend - harleyjj/yaml-editor - increases toast timeout from 4…

harleyjj commented on a change in pull request #4004: [TE] frontend - harleyjj/yaml-editor - increases toast timeout from 4… URL: https://github.com/apache/incubator-pinot/pull/4004#discussion_r268001507 ## File path:

[GitHub] [incubator-pinot] akshayrai commented on a change in pull request #4004: [TE] frontend - harleyjj/yaml-editor - increases toast timeout from 4…

akshayrai commented on a change in pull request #4004: [TE] frontend - harleyjj/yaml-editor - increases toast timeout from 4… URL: https://github.com/apache/incubator-pinot/pull/4004#discussion_r268000502 ## File path:

[incubator-pinot] branch master updated: [TE] predicted baseline endpoint fix (#4005)

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 dac8eca [TE] predicted baseline

[GitHub] [incubator-pinot] jihaozh merged pull request #4005: [TE] predicted baseline endpoint fix

jihaozh merged pull request #4005: [TE] predicted baseline endpoint fix URL: https://github.com/apache/incubator-pinot/pull/4005 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4005: [TE] predicted baseline endpoint fix

jihaozh opened a new pull request #4005: [TE] predicted baseline endpoint fix URL: https://github.com/apache/incubator-pinot/pull/4005 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-pinot] harleyjj opened a new pull request #4004: [TE] frontend - harleyjj/yaml-editor - increases toast timeout from 4…

harleyjj opened a new pull request #4004: [TE] frontend - harleyjj/yaml-editor - increases toast timeout from 4… URL: https://github.com/apache/incubator-pinot/pull/4004 … seconds to 10 seconds This is an automated message

[GitHub] [incubator-pinot] akshayrai opened a new pull request #4003: [TE] Fix ArrayIndexOutOfBoundsException; Wrap replay endpoint and ret…

akshayrai opened a new pull request #4003: [TE] Fix ArrayIndexOutOfBoundsException; Wrap replay endpoint and ret… URL: https://github.com/apache/incubator-pinot/pull/4003 …urn meaningful message This is an automated message

[GitHub] [incubator-pinot] mcvsubbu opened a new pull request #4002: Added documentation regarding design docs

mcvsubbu opened a new pull request #4002: Added documentation regarding design docs URL: https://github.com/apache/incubator-pinot/pull/4002 Added documentation pointing to design docs. Also fixed some warnings thrown during compilation

[incubator-pinot] branch master updated: Adding ORC reader (#3994)

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 8074d0a Adding ORC reader

[GitHub] [incubator-pinot] jenniferdai merged pull request #3994: Adding ORC reader

jenniferdai merged pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[incubator-pinot] branch orc deleted (was 3a18621)

This is an automated email from the ASF dual-hosted git repository. jenniferdai pushed a change to branch orc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 3a18621 Changing orc pom to 0.2.0-SNAPSHOT version The revisions that were on this branch are still

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #3994: Adding ORC reader

codecov-io edited a comment on issue #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#issuecomment-475355375 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/3994?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[incubator-pinot] branch orc updated: Changing orc pom to 0.2.0-SNAPSHOT version

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

[incubator-pinot] branch orc updated (38792d8 -> 8ba814b)

This is an automated email from the ASF dual-hosted git repository. jenniferdai pushed a change to branch orc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 38792d8 Addressing comments discard 025159e Fixing dependency issues discard 7f6263b fixing unit

[incubator-pinot] branch master updated: Reserve version 0.2.0 for the next release (#3807)

This is an automated email from the ASF dual-hosted git repository. snlee 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 a57caf0 Reserve version 0.2.0 for the

[incubator-pinot] branch update-version deleted (was 56201d3)

This is an automated email from the ASF dual-hosted git repository. snlee pushed a change to branch update-version in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 56201d3 Reserve version 0.2.0 for the next release The revisions that were on this branch are

[GitHub] [incubator-pinot] snleee merged pull request #3807: Reserve version 0.2.0 for the next release

snleee merged pull request #3807: Reserve version 0.2.0 for the next release URL: https://github.com/apache/incubator-pinot/pull/3807 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-pinot] jenniferdai commented on a change in pull request #3994: Adding ORC reader

jenniferdai commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267959911 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -0,0 +1,207 @@

[incubator-pinot] branch orc updated: Addressing comments

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

[GitHub] [incubator-pinot] wagnermarkd commented on a change in pull request #3994: Adding ORC reader

wagnermarkd commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267959634 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -0,0 +1,207 @@

[GitHub] [incubator-pinot] wagnermarkd commented on a change in pull request #3994: Adding ORC reader

wagnermarkd commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267959254 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -0,0 +1,207 @@

[incubator-pinot] 01/01: Add mv support

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 commit 5d9e806959ba10ad7b588d2b56f4329370d681e1 Author: Jennifer Dai AuthorDate: Thu Mar 21 14:01:27 2019 -0700

[incubator-pinot] branch orcMV created (now 5d9e806)

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. at 5d9e806 Add mv support This branch includes the following new commits: new 5d9e806 Add mv

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #3942: Set processingException when all queried segments cannot be acquired

codecov-io edited a comment on issue #3942: Set processingException when all queried segments cannot be acquired URL: https://github.com/apache/incubator-pinot/pull/3942#issuecomment-471113711 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/3942?src=pr=h1) Report >

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267903230 ## File path: docs/in_production.rst ## @@ -40,11 +40,14 @@

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267905349 ## File path: docs/in_production.rst ## @@ -64,4 +67,63 @@

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267902633 ## File path: docs/in_production.rst ## @@ -40,11 +40,14 @@

[GitHub] [incubator-pinot] codecov-io commented on issue #3994: Adding ORC reader

codecov-io commented on issue #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#issuecomment-475355375 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/3994?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267904312 ## File path: docs/in_production.rst ## @@ -64,4 +67,63 @@

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267903566 ## File path: docs/in_production.rst ## @@ -64,4 +67,63 @@

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267892780 ## File path: docs/tableconfig_schema.rst ## @@ -0,0 +1,204 @@ +..

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267890561 ## File path: docs/tableconfig_schema.rst ## @@ -0,0 +1,204 @@ +..

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267891006 ## File path: docs/tableconfig_schema.rst ## @@ -0,0 +1,204 @@ +..

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267891543 ## File path: docs/tableconfig_schema.rst ## @@ -0,0 +1,204 @@ +..

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267888179 ## File path: docs/tableconfig_schema.rst ## @@ -0,0 +1,204 @@ +..

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267887625 ## File path: docs/tableconfig_schema.rst ## @@ -0,0 +1,204 @@ +..

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267899010 ## File path: docs/tableconfig_schema.rst ## @@ -0,0 +1,204 @@ +..

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment

mcvsubbu commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r267890343 ## File path: docs/tableconfig_schema.rst ## @@ -0,0 +1,204 @@ +..

[GitHub] [incubator-pinot] jenniferdai commented on issue #3994: Adding ORC reader

jenniferdai commented on issue #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#issuecomment-475337813 I don't believe we can use that particular next method. That next method keeps the row batch (of unknown number) within it, which means that we

[incubator-pinot] branch master updated: [TE] yaml - anomaly filter validation (#4000)

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 dfaaf11 [TE] yaml - anomaly filter

[incubator-pinot] branch orc updated: Fixing dependency issues

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

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #3942: Set processingException when all queried segments cannot be acquired

sunithabeeram commented on a change in pull request #3942: Set processingException when all queried segments cannot be acquired URL: https://github.com/apache/incubator-pinot/pull/3942#discussion_r267867758 ## File path:

[GitHub] [incubator-pinot] mcvsubbu opened a new issue #4001: Create test suite across pinot releases and services

mcvsubbu opened a new issue #4001: Create test suite across pinot releases and services URL: https://github.com/apache/incubator-pinot/issues/4001 We should write a test suite that verifies that pinot releases are compatible with each other in terms of segment format, zookeeper data

[incubator-pinot] branch master updated: Updated realtime provisioning helper to take in max memory available in a host (#3997)

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 dd595cf Updated realtime

[GitHub] [incubator-pinot] mcvsubbu merged pull request #3997: Added maxUsableHostMemory argument to realtime provision helper.

mcvsubbu merged pull request #3997: Added maxUsableHostMemory argument to realtime provision helper. URL: https://github.com/apache/incubator-pinot/pull/3997 This is an automated message from the Apache Git Service. To

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

kishoreg commented on issue #3998: Upgrade to use Kafka release 2.1.1 URL: https://github.com/apache/incubator-pinot/issues/3998#issuecomment-475286938 Yes, we can keep a default implementation but have the ability to configure something else. This will be similar to remoteFS - >

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

mcvsubbu commented on issue #3998: Upgrade to use Kafka release 2.1.1 URL: https://github.com/apache/incubator-pinot/issues/3998#issuecomment-475277536 And we need to keep kafka client out of the distribution I suppose?

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3942: Set processingException when all queried segments cannot be acquired

mcvsubbu commented on a change in pull request #3942: Set processingException when all queried segments cannot be acquired URL: https://github.com/apache/incubator-pinot/pull/3942#discussion_r267811436 ## File path:

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #3942: Set processingException when all queried segments cannot be acquired

mcvsubbu commented on a change in pull request #3942: Set processingException when all queried segments cannot be acquired URL: https://github.com/apache/incubator-pinot/pull/3942#discussion_r267808241 ## File path:

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

mcvsubbu commented on issue #3998: Upgrade to use Kafka release 2.1.1 URL: https://github.com/apache/incubator-pinot/issues/3998#issuecomment-475270400 Not sure how the realtime integration tests will run, though. This is

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #3942: Set processingException when all queried segments cannot be acquired

sunithabeeram commented on a change in pull request #3942: Set processingException when all queried segments cannot be acquired URL: https://github.com/apache/incubator-pinot/pull/3942#discussion_r267806504 ## File path:

[GitHub] [incubator-pinot] mcvsubbu commented on issue #3978: The segments are stored in memory

mcvsubbu commented on issue #3978: The segments are stored in memory URL: https://github.com/apache/incubator-pinot/issues/3978#issuecomment-475268337 You do not need to disable and re-enable table for doing this. You can change the table config from HEAP to MMAP and either reload all