This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch fixing_packaging_issue
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit ea36238e636b1e0bfa788e0bc23b5f7731a04efd
Author: Xiang Fu
AuthorDate: Mon Jan 20 15:27:32 2020
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch fixing_packaging_issue
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at ea36238 Fixing java 11 runtime issue
This branch includes the following new commits:
fx19880617 opened a new pull request #4998: Fixing java 11 runtime issue
URL: https://github.com/apache/incubator-pinot/pull/4998
This is an automated message from the Apache Git Service.
To respond to the message, please log
siddharthteotia commented on issue #4993: Support Text column type in Pinot
(both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576459692
> Related comment in PR 4954. I don't have an answer, but we do need to
resolve it
>
> Also, are we
siddharthteotia commented on issue #4993: Support Text column type in Pinot
(both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576460097
> This is an amazing feature. let's create an issue to add the user doc for
this feature.
Yes I wil
siddharthteotia commented on issue #4993: Support Text column type in Pinot
(both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576460048
> Also, is this supported in the SQL path?
Yes it will be supported. As noted in my comments, Calcit
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368757789
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/realtime/impl/inver
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368757857
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/realtime/impl/inver
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368757847
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/realtime/impl/inver
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368758087
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/indexsegment/mutabl
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368758007
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/indexsegment/mutabl
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368758339
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/common/predicate/Te
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368758547
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/common/predicate/Te
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch switch-ui-to-sql
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from f0576e5 Make sql as default query format.
add b8e96d7 Update TransformExpressionTree.java
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368759415
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/indexsegment/mutabl
siddharthteotia commented on issue #4993: Support Text column type in Pinot
(both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576461826
@mcvsubbu , I have addressed your review comments. Please take another look.
-
siddharthteotia edited a comment on issue #4993: Support Text column type in
Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576460048
> Also, is this supported in the SQL path?
Yes it will be supported. As noted in my comments
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368759932
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/indexsegment/mutabl
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368760010
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/indexsegment/mutabl
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch upgrade_docker_build_maven_to_3.6
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard d9d02a3 update ingestion job specs for docker examples
add 546382c update
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368759880
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/dociditera
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368760225
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/indexsegment/mutabl
kishoreg commented on a change in pull request #4998: Fixing java 11 runtime
issues
URL: https://github.com/apache/incubator-pinot/pull/4998#discussion_r368759896
##
File path: pinot-tools/src/main/resources/appAssemblerScriptTemplate
##
@@ -107,6 +107,37 @@ if [ -n "$CLAS
kishoreg commented on a change in pull request #4998: Fixing java 11 runtime
issues
URL: https://github.com/apache/incubator-pinot/pull/4998#discussion_r368759279
##
File path: pinot-tools/src/main/resources/appAssemblerScriptTemplate
##
@@ -107,6 +107,37 @@ if [ -n "$CLAS
kishoreg commented on a change in pull request #4998: Fixing java 11 runtime
issues
URL: https://github.com/apache/incubator-pinot/pull/4998#discussion_r368758973
##
File path: pinot-distribution/pom.xml
##
@@ -93,6 +93,26 @@
+
Review comment
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368758007
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/indexsegment/mutabl
siddharthteotia commented on a change in pull request #4993: Support Text
column type in Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#discussion_r368757789
##
File path:
pinot-core/src/main/java/org/apache/pinot/core/realtime/impl/inver
siddharthteotia commented on issue #4993: Support Text column type in Pinot
(both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576464652
TODOs:
(1) Add the new column type
(2) Extend Calcite SQL grammar and parser to support compilati
siddharthteotia edited a comment on issue #4993: Support Text column type in
Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576464652
TODOs:
(1) Add the new data type.
(2) Extend Calcite SQL grammar and parser to support com
siddharthteotia edited a comment on issue #4993: Support Text column type in
Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576464652
TODOs (in addition to the ones added in the code)
(1) Add the new data type.
(2) Extend Ca
kishoreg merged pull request #4983: Make PQL case insensitive
URL: https://github.com/apache/incubator-pinot/pull/4983
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
This is an automated email from the ASF dual-hosted git repository.
kishoreg pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from f990656 Pinot batch ingestion hadoop (#4987)
add e46eec3 Make PQL case insensitive (#4983)
No new
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch switch-ui-to-sql
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from b8e96d7 Update TransformExpressionTree.java
add dfb36bf Adding sql endpoint in controller
fx19880617 commented on a change in pull request #4998: Fixing java 11 runtime
issues
URL: https://github.com/apache/incubator-pinot/pull/4998#discussion_r368764254
##
File path: pinot-distribution/pom.xml
##
@@ -93,6 +93,26 @@
+
Review comme
codecov-io edited a comment on issue #4994: Make sql as default query format.
URL: https://github.com/apache/incubator-pinot/pull/4994#issuecomment-575616116
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4994?src=pr&el=h1)
Report
> Merging
[#4994](https://codecov.io/gh/
siddharthteotia edited a comment on issue #4993: Support Text column type in
Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576460048
> Also, is this supported in the SQL path?
Yes it will be supported.
---
siddharthteotia removed a comment on issue #4993: Support Text column type in
Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576464652
TODOs (in addition to the ones added in the code)
(1) Add the new data type.
(2) Extend C
codecov-io edited a comment on issue #4993: Support Text column type in Pinot
(both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-575877560
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4993?src=pr&el=h1)
Report
> Merging
[
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch fixing_packaging_issue
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from ea36238 Fixing java 11 runtime issue
add 6295eab Address comments
No new revisions
siddharthteotia edited a comment on issue #4993: Support Text column type in
Pinot (both offline and realtime)
URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576459692
> Related comment in PR 4954. I don't have an answer, but we do need to
resolve it
>
> Also,
jackjlli commented on a change in pull request #4979: Update quota manager to
reduce zk access
URL: https://github.com/apache/incubator-pinot/pull/4979#discussion_r368771972
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HelixExternalViewBasedQueryQuota
jackjlli commented on a change in pull request #4979: Update quota manager to
reduce zk access
URL: https://github.com/apache/incubator-pinot/pull/4979#discussion_r368773524
##
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/queryquota/HelixExternalViewBasedQueryQuota
jackjlli commented on issue #4979: Update quota manager to reduce zk access
URL: https://github.com/apache/incubator-pinot/pull/4979#issuecomment-576479299
@mcvsubbu Currently the rate has to be re-calculated when number of online
brokers and table config exists, even these two don't change
codecov-io commented on issue #4998: Fixing java 11 runtime issues
URL: https://github.com/apache/incubator-pinot/pull/4998#issuecomment-576480386
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4998?src=pr&el=h1)
Report
> Merging
[#4998](https://codecov.io/gh/apache/incu
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch fixing_packaging_issue
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 6295eab Address comments
add 661abf0 Address comments
This update added new revisio
codecov-io edited a comment on issue #4979: Update quota manager to reduce zk
access
URL: https://github.com/apache/incubator-pinot/pull/4979#issuecomment-573975171
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4979?src=pr&el=h1)
Report
> Merging
[#4979](https://codeco
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch switch-ui-to-sql
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard dfb36bf Adding sql endpoint in controller
add 7dd4559 Adding sql endpoint in controller
T
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch switch-ui-to-sql
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 7dd4559 Adding sql endpoint in controller
discard b8e96d7 Update TransformExpressionTree.java
kishoreg commented on a change in pull request #4994: Make sql as default query
format.
URL: https://github.com/apache/incubator-pinot/pull/4994#discussion_r368783669
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotQueryResource.java
###
kishoreg commented on a change in pull request #4994: Make sql as default query
format.
URL: https://github.com/apache/incubator-pinot/pull/4994#discussion_r368784176
##
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotQueryResource.java
###
codecov-io edited a comment on issue #4998: Fixing java 11 runtime issues
URL: https://github.com/apache/incubator-pinot/pull/4998#issuecomment-576480386
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4998?src=pr&el=h1)
Report
> Merging
[#4998](https://codecov.io/gh/apac
codecov-io edited a comment on issue #4997: Update dockerfile for pinot image
and pinot-presto image
URL: https://github.com/apache/incubator-pinot/pull/4997#issuecomment-575875680
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4997?src=pr&el=h1)
Report
> Merging
[#4997
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch switch-ui-to-sql
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
from 0cca514 Adding sql endpoint in controller
add 95ed4c2 Address comments
No new revisions w
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch switch-ui-to-sql
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard 95ed4c2 Address comments
add db98b2a Address comments
This update added new revisions aft
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch switch-ui-to-sql
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
discard db98b2a Address comments
add e474484 Address comments
This update added new revisions aft
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 e46eec3 Make PQL case insensitive (#4983)
add e29b88a Fixing java 11 runtime issues (#4998)
No new
fx19880617 merged pull request #4998: Fixing java 11 runtime issues
URL: https://github.com/apache/incubator-pinot/pull/4998
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
fx19880617 commented on a change in pull request #4994: Make sql as default
query format.
URL: https://github.com/apache/incubator-pinot/pull/4994#discussion_r368786402
##
File path: pinot-controller/src/main/resources/static/js/init.js
##
@@ -67,13 +67,13 @@ $(document).r
codecov-io edited a comment on issue #4994: Make sql as default query format.
URL: https://github.com/apache/incubator-pinot/pull/4994#issuecomment-575616116
#
[Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/4994?src=pr&el=h1)
Report
> Merging
[#4994](https://codecov.io/gh/
59 matches
Mail list logo