[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #3986: Modify documentation in managing Pinot page

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3986: Modify documentation in managing Pinot page URL: https://github.com/apache/incubator-pinot/pull/3986#discussion_r267344684 ## File path: docs/management_api.rst ## @@ -26,26 +26,74 @@ Pinot Management Console

[GitHub] [incubator-pinot] sunithabeeram merged pull request #3986: Modify documentation in managing Pinot page

2019-03-20 Thread GitBox
sunithabeeram merged pull request #3986: Modify documentation in managing Pinot page URL: https://github.com/apache/incubator-pinot/pull/3986 This is an automated message from the Apache Git Service. To respond to the messag

[incubator-pinot] branch add-docs deleted (was 9b50d89)

2019-03-20 Thread sunithabeeram
This is an automated email from the ASF dual-hosted git repository. sunithabeeram pushed a change to branch add-docs in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 9b50d89 Add documentation The revisions that were on this branch are still contained in other ref

[incubator-pinot] branch master updated: Add documentation (#3986)

2019-03-20 Thread sunithabeeram
This is an automated email from the ASF dual-hosted git repository. sunithabeeram 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 59fd4aa Add documentation (#398

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #3989: Add experiment section in getting started doc

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3989: Add experiment section in getting started doc URL: https://github.com/apache/incubator-pinot/pull/3989#discussion_r267346057 ## File path: docs/getting_started.rst ## @@ -94,3 +94,159 @@ show up in Pinot. To show

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

2019-03-20 Thread GitBox
sunithabeeram 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_r267351884 ## File path: docs/tableconfig_schema.rst ## @@ -0,0 +1,204 @@

[incubator-pinot] branch orc updated (eb5e528 -> 2f03bf8)

2019-03-20 Thread jenniferdai
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 eb5e528 Adding orc reader new 2f03bf8 Adding orc reader This update added new revisions after undo

[incubator-pinot] branch orc updated (2f03bf8 -> 8b60f6f)

2019-03-20 Thread jenniferdai
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 2f03bf8 Adding orc reader new 8b60f6f Adding orc reader This update added new revisions after undo

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

2019-03-20 Thread GitBox
mcvsubbu commented on issue #3978: The segments are stored in memory URL: https://github.com/apache/incubator-pinot/issues/3978#issuecomment-474932084 You can read up on MMAP here : https://en.wikipedia.org/wiki/Memory-mapped_file A segment is MMAPed as long as the server is alive and h

[incubator-pinot] branch orc updated (8b60f6f -> e51533f)

2019-03-20 Thread jenniferdai
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 8b60f6f Adding orc reader new e51533f Adding orc reader This update added new revisions after undo

[incubator-pinot] branch management-api-doc created (now 35adb1f)

2019-03-20 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch management-api-doc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 35adb1f Update managing pinot doc This branch includes the following new commits: new 35ad

[incubator-pinot] 01/01: Update managing pinot doc

2019-03-20 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch management-api-doc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 35adb1fe511731883a780d8070c9865abf9b6535 Author: jackjlli AuthorDate: Wed Mar 20 10:20:19 2019 -0700

[incubator-pinot] branch management-api-doc updated (35adb1f -> a340bdf)

2019-03-20 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch management-api-doc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 35adb1f Update managing pinot doc new a340bdf Update managing pinot doc This update added

[GitHub] [incubator-pinot] jackjlli opened a new pull request #3991: Update managing pinot documentation

2019-03-20 Thread GitBox
jackjlli opened a new pull request #3991: Update managing pinot documentation URL: https://github.com/apache/incubator-pinot/pull/3991 This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #3986: Modify documentation in managing Pinot page

2019-03-20 Thread GitBox
jackjlli commented on a change in pull request #3986: Modify documentation in managing Pinot page URL: https://github.com/apache/incubator-pinot/pull/3986#discussion_r267458317 ## File path: docs/management_api.rst ## @@ -26,26 +26,74 @@ Pinot Management Console -

[incubator-pinot] branch add-doc-for-experiment updated (3d0a897 -> f9fb9a5)

2019-03-20 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch add-doc-for-experiment in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 3d0a897 Add experiment section in getting started new f9fb9a5 Add experiment section in

[incubator-pinot] branch orc updated: Copying orc reader to pinot core for now so I don't have to edit the pinot script internally to publish jars

2019-03-20 Thread jenniferdai
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 41c6bf7 Copying orc reader to pinot cor

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #3989: Add experiment section in getting started doc

2019-03-20 Thread GitBox
jackjlli commented on a change in pull request #3989: Add experiment section in getting started doc URL: https://github.com/apache/incubator-pinot/pull/3989#discussion_r267461373 ## File path: docs/getting_started.rst ## @@ -94,3 +94,159 @@ show up in Pinot. To show new e

[GitHub] [incubator-pinot] fx19880617 opened a new pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence

2019-03-20 Thread GitBox
fx19880617 opened a new pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence URL: https://github.com/apache/incubator-pinot/pull/3992 This is an automated message from the A

[incubator-pinot] branch orc updated: Fixing maven enforcer dependency issuse

2019-03-20 Thread jenniferdai
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 63336cc Fixing maven enforcer dependenc

[incubator-pinot] 01/01: In TableConfig, add checks for mandatory fields

2019-03-20 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch table_config_mandatory_fields in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit a409684e5eacfe71b7293b13ca85c4d89df58a07 Author: Jackie (Xiaotian) Jiang AuthorDate: Wed

[incubator-pinot] branch table_config_mandatory_fields created (now a409684)

2019-03-20 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch table_config_mandatory_fields in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at a409684 In TableConfig, add checks for mandatory fields This branch includes the fo

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #3993: In TableConfig, add checks for mandatory fields

2019-03-20 Thread GitBox
Jackie-Jiang opened a new pull request #3993: In TableConfig, add checks for mandatory fields URL: https://github.com/apache/incubator-pinot/pull/3993 Add explicit checks for mandatory fields Without the explicit checks, it will throw NPE, which is not clear and hard to debug -

[GitHub] [incubator-pinot] sunithabeeram merged pull request #3989: Add experiment section in getting started doc

2019-03-20 Thread GitBox
sunithabeeram merged pull request #3989: Add experiment section in getting started doc URL: https://github.com/apache/incubator-pinot/pull/3989 This is an automated message from the Apache Git Service. To respond to the mess

[incubator-pinot] branch master updated: Add experiment section in getting started (#3989)

2019-03-20 Thread sunithabeeram
This is an automated email from the ASF dual-hosted git repository. sunithabeeram 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 98dcebc Add experiment section

[incubator-pinot] branch management-api-doc deleted (was a340bdf)

2019-03-20 Thread sunithabeeram
This is an automated email from the ASF dual-hosted git repository. sunithabeeram pushed a change to branch management-api-doc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was a340bdf Update managing pinot doc The revisions that were on this branch are still cont

[GitHub] [incubator-pinot] sunithabeeram merged pull request #3991: Update managing pinot documentation

2019-03-20 Thread GitBox
sunithabeeram merged pull request #3991: Update managing pinot documentation URL: https://github.com/apache/incubator-pinot/pull/3991 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-pinot] branch master updated: Update managing pinot doc (#3991)

2019-03-20 Thread sunithabeeram
This is an automated email from the ASF dual-hosted git repository. sunithabeeram 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 205ec50 Update managing pinot d

[GitHub] [incubator-pinot] mayankshriv commented on a change in pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence

2019-03-20 Thread GitBox
mayankshriv commented on a change in pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence URL: https://github.com/apache/incubator-pinot/pull/3992#discussion_r267478728 ## File path: pinot-core/src/main/java/org/apache/pinot/core/r

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields URL: https://github.com/apache/incubator-pinot/pull/3993#discussion_r267479988 ## File path: pinot-common/src/main/java/org/apache/pinot/common/config/TableConfig.java ##

[incubator-pinot] branch orc updated (63336cc -> ea75c00)

2019-03-20 Thread jenniferdai
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. from 63336cc Fixing maven enforcer dependency issuse new d604449 Revert "Fixing maven enforcer dependenc

[GitHub] [incubator-pinot] jenniferdai opened a new pull request #3994: Adding ORC reader

2019-03-20 Thread GitBox
jenniferdai opened a new pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994 * Performance generating the same segment with ORC and Avro ORC: 3 mins 41 seconds Avro: 3 mins 35 seconds * Will check accuracy, blocked on someone else to give m

[GitHub] [incubator-pinot] xiangfu1 commented on a change in pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence

2019-03-20 Thread GitBox
xiangfu1 commented on a change in pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence URL: https://github.com/apache/incubator-pinot/pull/3992#discussion_r267483867 ## File path: pinot-core/src/main/java/org/apache/pinot/core/real

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #3989: Add experiment section in getting started doc

2019-03-20 Thread GitBox
codecov-io edited a comment on issue #3989: Add experiment section in getting started doc URL: https://github.com/apache/incubator-pinot/pull/3989#issuecomment-474632607 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/3989?src=pr&el=h1) Report > Merging [#3989](https://c

[GitHub] [incubator-pinot] xiangfu1 commented on a change in pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence

2019-03-20 Thread GitBox
xiangfu1 commented on a change in pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence URL: https://github.com/apache/incubator-pinot/pull/3992#discussion_r267483867 ## File path: pinot-core/src/main/java/org/apache/pinot/core/real

[GitHub] [incubator-pinot] kishoreg commented on a change in pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence

2019-03-20 Thread GitBox
kishoreg commented on a change in pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence URL: https://github.com/apache/incubator-pinot/pull/3992#discussion_r267490711 ## File path: pinot-core/src/main/java/org/apache/pinot/core/real

[GitHub] [incubator-pinot] harleyjj opened a new pull request #3995: [TE] frontend - harleyjj/edit-alert - fix subscription group put bug

2019-03-20 Thread GitBox
harleyjj opened a new pull request #3995: [TE] frontend - harleyjj/edit-alert - fix subscription group put bug URL: https://github.com/apache/incubator-pinot/pull/3995 This is an automated message from the Apache Git Service

[GitHub] [incubator-pinot] codecov-io commented on issue #3993: In TableConfig, add checks for mandatory fields

2019-03-20 Thread GitBox
codecov-io commented on issue #3993: In TableConfig, add checks for mandatory fields URL: https://github.com/apache/incubator-pinot/pull/3993#issuecomment-474986968 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/3993?src=pr&el=h1) Report > Merging [#3993](https://codeco

[GitHub] [incubator-pinot] xiangfu1 commented on a change in pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence

2019-03-20 Thread GitBox
xiangfu1 commented on a change in pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence URL: https://github.com/apache/incubator-pinot/pull/3992#discussion_r267483867 ## File path: pinot-core/src/main/java/org/apache/pinot/core/real

[GitHub] [incubator-pinot] akshayrai opened a new pull request #3996: [TE] Clean up the yaml editor calls and messages

2019-03-20 Thread GitBox
akshayrai opened a new pull request #3996: [TE] Clean up the yaml editor calls and messages URL: https://github.com/apache/incubator-pinot/pull/3996 This is an automated message from the Apache Git Service. To respond to the

[incubator-pinot] branch master updated: [TE] frontend - harleyjj/edit-alert - fix subscription group put bug (#3995)

2019-03-20 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 d8061f3 [TE] frontend - harleyjj/

[GitHub] [incubator-pinot] akshayrai merged pull request #3995: [TE] frontend - harleyjj/edit-alert - fix subscription group put bug

2019-03-20 Thread GitBox
akshayrai merged pull request #3995: [TE] frontend - harleyjj/edit-alert - fix subscription group put bug URL: https://github.com/apache/incubator-pinot/pull/3995 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-pinot] harleyjj commented on issue #3996: [TE] Clean up the yaml editor calls and messages

2019-03-20 Thread GitBox
harleyjj commented on issue #3996: [TE] Clean up the yaml editor calls and messages URL: https://github.com/apache/incubator-pinot/pull/3996#issuecomment-474996148 These names seem much more clear - thank you! It seems there are some branch conflicts, though.

[GitHub] [incubator-pinot] codecov-io commented on issue #3992: Fixing type casting issue for BYTES type values during realtime segment persistence

2019-03-20 Thread GitBox
codecov-io commented on issue #3992: Fixing type casting issue for BYTES type values during realtime segment persistence URL: https://github.com/apache/incubator-pinot/pull/3992#issuecomment-474996093 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/3992?src=pr&el=h1) Report

[incubator-pinot] branch add-doc-for-experiment deleted (was f9fb9a5)

2019-03-20 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch add-doc-for-experiment in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was f9fb9a5 Add experiment section in getting started The revisions that were on this branch are

[incubator-pinot] branch add-doc-experiment-with-pinot deleted (was dd11db8)

2019-03-20 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch add-doc-experiment-with-pinot in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was dd11db8 [TE] frontend - harleyjj/report-anomaly - adds back report-anomaly modal to a

[incubator-pinot] branch getting-started-doc deleted (was 7cc4401)

2019-03-20 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch getting-started-doc in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 7cc4401 Add doc on experimenting with Pinot This change permanently discards the following revi

[incubator-pinot] branch make-delete-table-api-async deleted (was 27663ca)

2019-03-20 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch make-delete-table-api-async in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 27663ca Make delete table API async The revisions that were on this branch are still co

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields

2019-03-20 Thread GitBox
Jackie-Jiang commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields URL: https://github.com/apache/incubator-pinot/pull/3993#discussion_r267524256 ## File path: pinot-common/src/main/java/org/apache/pinot/common/config/TableConfig.java ###

[GitHub] [incubator-pinot] fx19880617 merged pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence

2019-03-20 Thread GitBox
fx19880617 merged pull request #3992: Fixing type casting issue for BYTES type values during realtime segment persistence URL: https://github.com/apache/incubator-pinot/pull/3992 This is an automated message from the Apache

[incubator-pinot] branch master updated: Fixing type casting issue for BYTES type values during realtime segment persistence (#3992)

2019-03-20 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 6eb8e79 Fixing type casting issue for

[incubator-pinot] branch master updated: [TE] Clean up the yaml editor calls and messages (#3996)

2019-03-20 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 d78a807 [TE] Clean up the yaml ed

[GitHub] [incubator-pinot] akshayrai merged pull request #3996: [TE] Clean up the yaml editor calls and messages

2019-03-20 Thread GitBox
akshayrai merged pull request #3996: [TE] Clean up the yaml editor calls and messages URL: https://github.com/apache/incubator-pinot/pull/3996 This is an automated message from the Apache Git Service. To respond to the messa

[incubator-pinot] branch orc updated: Editing reusable vectorized row batch to really be reusable

2019-03-20 Thread jenniferdai
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 d620318 Editing reusable vectorized row

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

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267545282 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -0,0 +1,187 @@ +pa

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields

2019-03-20 Thread GitBox
jackjlli commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields URL: https://github.com/apache/incubator-pinot/pull/3993#discussion_r267555038 ## File path: pinot-common/src/main/java/org/apache/pinot/common/config/TableConfig.java ##

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

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267562665 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -0,0 +1,187 @@ +pa

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

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267561974 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -0,0 +1,187 @@ +pa

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

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267563317 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -0,0 +1,187 @@ +pa

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

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267564576 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -0,0 +1,187 @@ +pa

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

2019-03-20 Thread GitBox
mcvsubbu opened a new pull request #3997: Added maxUsableHostMemory argument to realtime provision helper. URL: https://github.com/apache/incubator-pinot/pull/3997 In installations where hosts have memory limits other than 48G, the admin can specify other memory limits to be used to

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

2019-03-20 Thread GitBox
jenniferdai commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267569198 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -0,0 +1,187 @@ +pack

[incubator-pinot] branch orc updated (d620318 -> e09e5fd)

2019-03-20 Thread jenniferdai
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. from d620318 Editing reusable vectorized row batch to really be reusable new 2e6bf2f Fixing dependencies

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

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267574279 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -58,6 +67,24 @@

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

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3994: Adding ORC reader URL: https://github.com/apache/incubator-pinot/pull/3994#discussion_r267574579 ## File path: pinot-orc/src/main/java/org/apache/pinot/orc/data/readers/ORCRecordReader.java ## @@ -0,0 +1,218 @@ +pa

[incubator-pinot] branch orc updated: Addressing comments

2019-03-20 Thread jenniferdai
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 d9b0bce Addressing comments d9b0bce is

[incubator-pinot] branch orc updated (d9b0bce -> d6c3a27)

2019-03-20 Thread jenniferdai
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 d9b0bce Addressing comments discard e09e5fd Addressing comments new d6c3a27 Addressing comments

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

2019-03-20 Thread GitBox
snleee 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_r267578867 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/manager/TableDat

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields

2019-03-20 Thread GitBox
Jackie-Jiang commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields URL: https://github.com/apache/incubator-pinot/pull/3993#discussion_r267580774 ## File path: pinot-common/src/main/java/org/apache/pinot/common/config/TableConfig.java ###

[incubator-pinot] branch table_config_mandatory_fields updated (a409684 -> b57dec9)

2019-03-20 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch table_config_mandatory_fields in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard a409684 In TableConfig, add checks for mandatory fields new b57dec9 In TableCo

[incubator-pinot] branch nested-object-indexing updated (c2d822d -> 3140bb9)

2019-03-20 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch nested-object-indexing in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard c2d822d fixing license header discard 635c451 Adding support for bytes type in realtime

[GitHub] [incubator-pinot] sunithabeeram commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields

2019-03-20 Thread GitBox
sunithabeeram commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields URL: https://github.com/apache/incubator-pinot/pull/3993#discussion_r267582170 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineCl

[incubator-pinot] branch table_config_mandatory_fields updated (b57dec9 -> 9bc06ed)

2019-03-20 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch table_config_mandatory_fields in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard b57dec9 In TableConfig, add checks for mandatory fields new 9bc06ed In TableCo

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields

2019-03-20 Thread GitBox
Jackie-Jiang commented on a change in pull request #3993: In TableConfig, add checks for mandatory fields URL: https://github.com/apache/incubator-pinot/pull/3993#discussion_r267584320 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClu

[GitHub] [incubator-pinot] weiwanning commented on issue #3513: DATETIMECONVERT udf does not work for customized timezone and bucket size > 1 day

2019-03-20 Thread GitBox
weiwanning commented on issue #3513: DATETIMECONVERT udf does not work for customized timezone and bucket size > 1 day URL: https://github.com/apache/incubator-pinot/issues/3513#issuecomment-475067265 hey @npawar, went into the same issue, option2 seems ideal for my use case(better to ma

[incubator-pinot] branch orc updated: Adding ORC Reader Test

2019-03-20 Thread jenniferdai
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 91767a1 Adding ORC Reader Test 91767a1

[GitHub] [incubator-pinot] mcvsubbu opened a new issue #3998: Upgrade to use latest Kafka release

2019-03-20 Thread GitBox
mcvsubbu opened a new issue #3998: Upgrade to use latest Kafka release URL: https://github.com/apache/incubator-pinot/issues/3998 Kafka release has a stable version of 2.1.1 (https://kafka.apache.org/downloads) Pinot should upgrade to a more recent version of Kafka This releas

[GitHub] [incubator-pinot] jihaozh opened a new pull request #4000: [TE] yaml - anomaly filter validation

2019-03-20 Thread GitBox
jihaozh opened a new pull request #4000: [TE] yaml - anomaly filter validation URL: https://github.com/apache/incubator-pinot/pull/4000 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [incubator-pinot] jihaozh opened a new pull request #3999: [TE] Aggregation function and double series aggregation mapping

2019-03-20 Thread GitBox
jihaozh opened a new pull request #3999: [TE] Aggregation function and double series aggregation mapping URL: https://github.com/apache/incubator-pinot/pull/3999 This is an automated message from the Apache Git Service. To r

[GitHub] [incubator-pinot] codecov-io edited a comment on issue #3993: In TableConfig, add checks for mandatory fields

2019-03-20 Thread GitBox
codecov-io edited a comment on issue #3993: In TableConfig, add checks for mandatory fields URL: https://github.com/apache/incubator-pinot/pull/3993#issuecomment-474986968 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/3993?src=pr&el=h1) Report > Merging [#3993](https:/

[incubator-pinot] branch orc updated: fixing unit test

2019-03-20 Thread jenniferdai
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 7f6263b fixing unit test 7f6263b is des

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #3993: In TableConfig, add checks for mandatory fields

2019-03-20 Thread GitBox
Jackie-Jiang merged pull request #3993: In TableConfig, add checks for mandatory fields URL: https://github.com/apache/incubator-pinot/pull/3993 This is an automated message from the Apache Git Service. To respond to the mes

[incubator-pinot] branch table_config_mandatory_fields deleted (was 9bc06ed)

2019-03-20 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch table_config_mandatory_fields in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. was 9bc06ed In TableConfig, add checks for mandatory fields The revisions that were on

[incubator-pinot] branch master updated: In TableConfig, add checks for mandatory fields (#3993)

2019-03-20 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie 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 eccf573 In TableConfig, add checks for

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

2019-03-20 Thread GitBox
mcvsubbu commented on issue #3998: Upgrade to use latest Kafka release URL: https://github.com/apache/incubator-pinot/issues/3998#issuecomment-475079057 Maybe we can consider moving to this https://github.com/linkedin/li-apache-kafka-clients if it is compatible -

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

2019-03-20 Thread GitBox
kishoreg commented on issue #3998: Upgrade to use Kafka release 2.1.1 URL: https://github.com/apache/incubator-pinot/issues/3998#issuecomment-475080724 The first step would be decouple Pinot core from kafka right. Once we have that, we can add multiple implementations likafka, Kafka-0.9.x

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

2019-03-20 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-475081698 Sure, we can take all of the current Kafka code to a separate repo that depends on kafka=0.9.x and build other repos for newer Kafkas (if

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

2019-03-20 Thread GitBox
kishoreg commented on issue #3998: Upgrade to use Kafka release 2.1.1 URL: https://github.com/apache/incubator-pinot/issues/3998#issuecomment-475082511 I was thinking along the same lines. Let’s do that. That will allow us to make sure we are backwards compatible ---

[incubator-pinot] branch master updated: [TE] Aggregation function and double series aggregation mapping (#3999)

2019-03-20 Thread jihao
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 d78160d [TE] Aggregation function and d

[GitHub] [incubator-pinot] jihaozh merged pull request #3999: [TE] Aggregation function and double series aggregation mapping

2019-03-20 Thread GitBox
jihaozh merged pull request #3999: [TE] Aggregation function and double series aggregation mapping URL: https://github.com/apache/incubator-pinot/pull/3999 This is an automated message from the Apache Git Service. To respond

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

2019-03-20 Thread GitBox
ly923976094 commented on issue #3978: The segments are stored in memory URL: https://github.com/apache/incubator-pinot/issues/3978#issuecomment-475092629 I understanding is that when I change the table's loading mode, for example, from HEAP to MMAP, I need to disable the table first, and t

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

2019-03-20 Thread GitBox
ly923976094 edited a comment on issue #3978: The segments are stored in memory URL: https://github.com/apache/incubator-pinot/issues/3978#issuecomment-475092629 I understanding is that when I change the table's loading mode, for example, from HEAP to MMAP, I need to disable the table first

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

2019-03-20 Thread GitBox
snleee 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_r267609303 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTabl