[incubator-pinot] branch time-functions-in-query updated (b91b4f3 -> 6d95bb7)

2020-05-19 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch time-functions-in-query in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard b91b4f3 Adding support to execute functions during query compilation add 86aea53

[GitHub] [incubator-pinot] mayankshriv commented on pull request #5410: Fix flaky test for InterSegmentResultTableMultiValueQueriesTest

2020-05-19 Thread GitBox
mayankshriv commented on pull request #5410: URL: https://github.com/apache/incubator-pinot/pull/5410#issuecomment-630934467 In case of ties (multiple groups with same value), using `TOP 1` is also prone to the same issue. You can either fix the group by adding a filter, or using

[incubator-pinot] branch time-functions-in-query updated (6d95bb7 -> 53acdfa)

2020-05-19 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch time-functions-in-query in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 6d95bb7 Adding support to execute functions during query compilation add 53acdfa

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #5282: [TE] ELR migration for Thirdeye

2020-05-19 Thread GitBox
jackjlli commented on a change in pull request #5282: URL: https://github.com/apache/incubator-pinot/pull/5282#discussion_r427497365 ## File path: .travis.yml ## @@ -10,6 +10,7 @@ before_install: - jq --version - curl -sSL -o ~/bin/install-jdk.sh

[incubator-pinot] branch time-functions-in-query updated (53acdfa -> 0e21590)

2020-05-19 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch time-functions-in-query in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 53acdfa Adding support to execute functions during query compilation add 0e21590

[incubator-pinot] branch time-functions-in-query updated (cbb6ad6 -> 55c9814)

2020-05-19 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch time-functions-in-query in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard cbb6ad6 Adding support to execute functions during query compilation add 55c9814

[incubator-pinot] branch time-functions-in-query updated (5ef20ce -> cbb6ad6)

2020-05-19 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch time-functions-in-query in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 5ef20ce Adding support to execute functions during query compilation add cbb6ad6

[incubator-pinot] branch generator-script-fix-for-5338 created (now deac52d)

2020-05-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch generator-script-fix-for-5338 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at deac52d fix generator.sh to follow #5338 changes This branch includes the

[incubator-pinot] 01/01: fix generator.sh to follow #5338 changes

2020-05-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch generator-script-fix-for-5338 in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit deac52db087c57ef4ea72c3e79cb992e70570893 Author: Alexander Pucher AuthorDate: Tue May

[incubator-pinot] branch time-functions-in-query updated (0e21590 -> 5ef20ce)

2020-05-19 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch time-functions-in-query in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 0e21590 Adding support to execute functions during query compilation add 5ef20ce

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #5414: Fix the IndexOutOfBoundException for range index

2020-05-19 Thread GitBox
Jackie-Jiang opened a new pull request #5414: URL: https://github.com/apache/incubator-pinot/pull/5414 Fix the numValuesPerRange calculation logic of range index, which can cause IndexOutOfBoundException This is an

[incubator-pinot] branch time-functions-in-query updated (55c9814 -> 1368749)

2020-05-19 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch time-functions-in-query in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 55c9814 Adding support to execute functions during query compilation add 1368749

[GitHub] [incubator-pinot] apucher commented on issue #5412: [Docs] Integration with ThirdEye: generate and load sample data script doesn't work

2020-05-19 Thread GitBox
apucher commented on issue #5412: URL: https://github.com/apache/incubator-pinot/issues/5412#issuecomment-631042894 Thanks, Oleg. Looks like I missed a change in the CreateSegment command params. Your fix is correct. PR here: #5415

[GitHub] [incubator-pinot] apucher opened a new pull request #5415: fix generator.sh to follow #5338 changes

2020-05-19 Thread GitBox
apucher opened a new pull request #5415: URL: https://github.com/apache/incubator-pinot/pull/5415 Use **tableConfigFile** rather than tableName and segmentName. Fixes #5412 This is an automated message from the Apache Git

[GitHub] [incubator-pinot] mcvsubbu opened a new pull request #5411: Upgrade helix to 0.9.7

2020-05-19 Thread GitBox
mcvsubbu opened a new pull request #5411: URL: https://github.com/apache/incubator-pinot/pull/5411 Helix release notes: http://helix.apache.org/0.9.7-docs/releasenotes/release-0.9.7.html This is an automated message from

[GitHub] [incubator-pinot] mcvsubbu commented on pull request #5399: DATE_TIME should work as the primary time column for Pinot tables

2020-05-19 Thread GitBox
mcvsubbu commented on pull request #5399: URL: https://github.com/apache/incubator-pinot/pull/5399#issuecomment-630981224 If you can please do these steps to test it * Create a hybrid table with pinot-0.3.0, let it consume some segments, and have some offline segments as well *

[GitHub] [incubator-pinot] olegchorny opened a new issue #5412: [Docs] Integration with ThirdEye: generate and load sample data script doesn't work

2020-05-19 Thread GitBox
olegchorny opened a new issue #5412: URL: https://github.com/apache/incubator-pinot/issues/5412 https://docs.pinot.apache.org/integrations/thirdeye#prepare-your-data-optional ``` Creating segment for complexWebsite in /tmp/pinotGenerator/complexWebsiteSegment Error:

[incubator-pinot] branch alerts-pagination updated (f45a712 -> dc2e563)

2020-05-19 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a change to branch alerts-pagination in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard f45a712 [TE] alerts search and pagination endpoint add 9b65c9e Do not release the

[incubator-pinot] 01/01: [TE] alerts search and pagination endpoint

2020-05-19 Thread jihao
This is an automated email from the ASF dual-hosted git repository. jihao pushed a commit to branch alerts-pagination in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit dc2e563ba91554146a577e36aabb28c79d707360 Author: Jihao Zhang AuthorDate: Thu May 14 15:26:41 2020

[GitHub] [incubator-pinot] jihaozh opened a new pull request #5413: [TE] alerts search and pagination endpoint

2020-05-19 Thread GitBox
jihaozh opened a new pull request #5413: URL: https://github.com/apache/incubator-pinot/pull/5413 This PR adds the endpoint for alert search and pagination. Specifically, - The endpoint to search and paginate the alerts. - The Alert searcher to retrieve the alerts in the database

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5406: Adding support to execute functions during query compilation

2020-05-19 Thread GitBox
siddharthteotia commented on a change in pull request #5406: URL: https://github.com/apache/incubator-pinot/pull/5406#discussion_r427580739 ## File path: pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java ## @@ -615,7 +620,44 @@ private static

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5399: DATE_TIME should work as the primary time column for Pinot tables

2020-05-19 Thread GitBox
mcvsubbu commented on a change in pull request #5399: URL: https://github.com/apache/incubator-pinot/pull/5399#discussion_r427479912 ## File path: pinot-core/src/main/java/org/apache/pinot/core/segment/creator/impl/V1Constants.java ## @@ -51,6 +51,10 @@ public static

[GitHub] [incubator-pinot] vincentchenjl commented on a change in pull request #5282: [TE] ELR migration for Thirdeye

2020-05-19 Thread GitBox
vincentchenjl commented on a change in pull request #5282: URL: https://github.com/apache/incubator-pinot/pull/5282#discussion_r427632810 ## File path: .travis/.travis_te_nightly_build.sh ## @@ -0,0 +1,36 @@ +#!/bin/bash -x +# +# Licensed to the Apache Software Foundation

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5406: Adding support to execute functions during query compilation

2020-05-19 Thread GitBox
fx19880617 commented on a change in pull request #5406: URL: https://github.com/apache/incubator-pinot/pull/5406#discussion_r427596245 ## File path: pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java ## @@ -615,7 +620,44 @@ private static Expression

[GitHub] [incubator-pinot] kishoreg commented on a change in pull request #5406: Adding support to execute functions during query compilation

2020-05-19 Thread GitBox
kishoreg commented on a change in pull request #5406: URL: https://github.com/apache/incubator-pinot/pull/5406#discussion_r427597340 ## File path: pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java ## @@ -615,7 +620,44 @@ private static Expression

[GitHub] [incubator-pinot] kishoreg commented on a change in pull request #5406: Adding support to execute functions during query compilation

2020-05-19 Thread GitBox
kishoreg commented on a change in pull request #5406: URL: https://github.com/apache/incubator-pinot/pull/5406#discussion_r427597153 ## File path: pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java ## @@ -615,7 +618,47 @@ private static Expression

[GitHub] [incubator-pinot] mayankshriv commented on issue #5416: Functions in Pinot

2020-05-19 Thread GitBox
mayankshriv commented on issue #5416: URL: https://github.com/apache/incubator-pinot/issues/5416#issuecomment-631085745 We can create a hierarchy, with top level classification as: 1. Aggregation: m rows input k rows output (m <=k) 2. Transform: 1 row input 1 row output

[incubator-pinot] branch time-functions-in-query updated (1368749 -> 9a4e8ca)

2020-05-19 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch time-functions-in-query in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 1368749 Adding support to execute functions during query compilation add 9a4e8ca

[GitHub] [incubator-pinot] vincentchenjl commented on a change in pull request #5282: [TE] ELR migration for Thirdeye

2020-05-19 Thread GitBox
vincentchenjl commented on a change in pull request #5282: URL: https://github.com/apache/incubator-pinot/pull/5282#discussion_r427631408 ## File path: thirdeye/thirdeye-frontend/yarn.lock ## @@ -2,710 +2,830 @@ # yarn lockfile v1 -"@babel/code-frame@^7.0.0",

[GitHub] [incubator-pinot] npawar commented on pull request #5399: DATE_TIME should work as the primary time column for Pinot tables

2020-05-19 Thread GitBox
npawar commented on pull request #5399: URL: https://github.com/apache/incubator-pinot/pull/5399#issuecomment-631114033 > If you can please do these steps to test it > > * Create a hybrid table with pinot-0.3.0, let it consume some segments, and have some offline segments as well

[GitHub] [incubator-pinot] mcvsubbu commented on pull request #5411: Upgrade helix to 0.9.7

2020-05-19 Thread GitBox
mcvsubbu commented on pull request #5411: URL: https://github.com/apache/incubator-pinot/pull/5411#issuecomment-631135360 @haibow this is to be included in 0.4.0 This is an automated message from the Apache Git Service. To

[incubator-pinot] branch master updated: Upgrade helix to 0.9.7 (#5411)

2020-05-19 Thread mcvsubbu
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 1e0cc16 Upgrade helix to 0.9.7

[GitHub] [incubator-pinot] mcvsubbu merged pull request #5411: Upgrade helix to 0.9.7

2020-05-19 Thread GitBox
mcvsubbu merged pull request #5411: URL: https://github.com/apache/incubator-pinot/pull/5411 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-pinot] apucher merged pull request #5415: fix generator.sh to follow #5338 changes

2020-05-19 Thread GitBox
apucher merged pull request #5415: URL: https://github.com/apache/incubator-pinot/pull/5415 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[incubator-pinot] branch master updated (3453128 -> b61400f)

2020-05-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 3453128 Removes redundant DateTimeFormatter cache in DateTimeHandler (#5405) add b61400f fix

[GitHub] [incubator-pinot] apucher closed issue #5412: [Docs] Integration with ThirdEye: generate and load sample data script doesn't work

2020-05-19 Thread GitBox
apucher closed issue #5412: URL: https://github.com/apache/incubator-pinot/issues/5412 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [incubator-pinot] fx19880617 commented on issue #5416: Functions in Pinot

2020-05-19 Thread GitBox
fx19880617 commented on issue #5416: URL: https://github.com/apache/incubator-pinot/issues/5416#issuecomment-631082032 My overall feeling is that we should just have two types of functions: - aggregation: which knows how to do accumulate - transform: which is pure transform, given

[GitHub] [incubator-pinot] fx19880617 edited a comment on issue #5416: Functions in Pinot

2020-05-19 Thread GitBox
fx19880617 edited a comment on issue #5416: URL: https://github.com/apache/incubator-pinot/issues/5416#issuecomment-631082032 My overall feeling is that we should just have two types of functions: - aggregation: which knows how to do accumulate - transform: which is pure transform,

[GitHub] [incubator-pinot] kishoreg commented on pull request #5345: Create PULL_REQUEST_TEMPLATE.md

2020-05-19 Thread GitBox
kishoreg commented on pull request #5345: URL: https://github.com/apache/incubator-pinot/pull/5345#issuecomment-631133291 whats the next step here? This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-pinot] kishoreg opened a new issue #5416: Functions in Pinot

2020-05-19 Thread GitBox
kishoreg opened a new issue #5416: URL: https://github.com/apache/incubator-pinot/issues/5416 We have multiple types of Functions in Pinot - AggregationFunction - Used during query execution - Stateful - aggregate method can be invoked multiple times with blockValMap

[GitHub] [incubator-pinot] mcvsubbu commented on pull request #5399: DATE_TIME should work as the primary time column for Pinot tables

2020-05-19 Thread GitBox
mcvsubbu commented on pull request #5399: URL: https://github.com/apache/incubator-pinot/pull/5399#issuecomment-631129692 Really appreciate your diligence @npawar thanks This is an automated message from the Apache Git

[GitHub] [incubator-pinot] mcvsubbu commented on pull request #5345: Create PULL_REQUEST_TEMPLATE.md

2020-05-19 Thread GitBox
mcvsubbu commented on pull request #5345: URL: https://github.com/apache/incubator-pinot/pull/5345#issuecomment-631136749 I need some concrete suggestion from @siddharthteotia and approval, since he had comments. We can iterate on this, later

[GitHub] [incubator-pinot] kishoreg commented on pull request #3727: Matches support in pql

2020-05-19 Thread GitBox
kishoreg commented on pull request #3727: URL: https://github.com/apache/incubator-pinot/pull/3727#issuecomment-631145301 Closing since we have the full-text search now This is an automated message from the Apache Git

[incubator-pinot] branch master updated: DATE_TIME should work as the primary time column for Pinot tables (#5399)

2020-05-19 Thread nehapawar
This is an automated email from the ASF dual-hosted git repository. nehapawar 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 85474ba DATE_TIME should work as

[GitHub] [incubator-pinot] npawar merged pull request #5399: DATE_TIME should work as the primary time column for Pinot tables

2020-05-19 Thread GitBox
npawar merged pull request #5399: URL: https://github.com/apache/incubator-pinot/pull/5399 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-pinot] kishoreg closed pull request #3872: Add metrics to track work done in group-by phase

2020-05-19 Thread GitBox
kishoreg closed pull request #3872: URL: https://github.com/apache/incubator-pinot/pull/3872 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-pinot] kishoreg closed pull request #3727: Matches support in pql

2020-05-19 Thread GitBox
kishoreg closed pull request #3727: URL: https://github.com/apache/incubator-pinot/pull/3727 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-pinot] kishoreg commented on pull request #3872: Add metrics to track work done in group-by phase

2020-05-19 Thread GitBox
kishoreg commented on pull request #3872: URL: https://github.com/apache/incubator-pinot/pull/3872#issuecomment-631144738 Closing this since we have groupByLimit reached flag in the response This is an automated message from

[GitHub] [incubator-pinot] mayankshriv opened a new pull request #5417: Fix issue with PerfBenchmarkRunner where server can no longer pre-load segments from dataDir.

2020-05-19 Thread GitBox
mayankshriv opened a new pull request #5417: URL: https://github.com/apache/incubator-pinot/pull/5417 Due to recent PRs (#5383, #5381), PerfBenchmarkRunner now makes additional copy of segments on controller, and makes server download data from controller. The original behavior was

[GitHub] [incubator-pinot] fx19880617 commented on issue #5401: Query Supporting datetime column comparing against currenttime

2020-05-19 Thread GitBox
fx19880617 commented on issue #5401: URL: https://github.com/apache/incubator-pinot/issues/5401#issuecomment-631140531 https://github.com/apache/incubator-pinot/pull/5406 This is an automated message from the Apache Git

[incubator-pinot] 01/01: Reverting the change from #5291

2020-05-19 Thread snlee
This is an automated email from the ASF dual-hosted git repository. snlee pushed a commit to branch hotfix-hashmap-init in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit 2601c01ca1174fb9f178077941512c1f06b6eef2 Author: Seunghyun Lee AuthorDate: Tue May 19 19:12:06

[incubator-pinot] branch hotfix-hashmap-init created (now 2601c01)

2020-05-19 Thread snlee
This is an automated email from the ASF dual-hosted git repository. snlee pushed a change to branch hotfix-hashmap-init in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at 2601c01 Reverting the change from #5291 This branch includes the following new commits:

svn commit: r39702 - /release/incubator/pinot/KEYS

2020-05-19 Thread haibow
Author: haibow Date: Wed May 20 05:41:02 2020 New Revision: 39702 Log: Update Pinot KEYS file for Modified: release/incubator/pinot/KEYS Modified: release/incubator/pinot/KEYS == --- release/incubator/pinot/KEYS

svn commit: r39703 - /release/incubator/pinot/KEYS

2020-05-19 Thread haibow
Author: haibow Date: Wed May 20 05:58:51 2020 New Revision: 39703 Log: Update Pinot KEYS file for Modified: release/incubator/pinot/KEYS Modified: release/incubator/pinot/KEYS == --- release/incubator/pinot/KEYS

[incubator-pinot] branch master updated (85474ba -> 4e3f062)

2020-05-19 Thread mayanks
This is an automated email from the ASF dual-hosted git repository. mayanks pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 85474ba DATE_TIME should work as the primary time column for Pinot tables (#5399) add 4e3f062 Fix

[GitHub] [incubator-pinot] mayankshriv merged pull request #5417: Fix issue with PerfBenchmarkRunner where server can no longer pre-load segments from dataDir.

2020-05-19 Thread GitBox
mayankshriv merged pull request #5417: URL: https://github.com/apache/incubator-pinot/pull/5417 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-pinot] harleyjj opened a new pull request #5418: [TE] frontend - harleyjj/packages - pin compatible package versions a…

2020-05-19 Thread GitBox
harleyjj opened a new pull request #5418: URL: https://github.com/apache/incubator-pinot/pull/5418 …nd set node to 12 1) Sets Node version to 12 because Node 10 will pull incompatible packages that cause frontend tests to fail 2) Pins version for ember-toggle to prevent

[GitHub] [incubator-pinot] harleyjj commented on a change in pull request #5282: [TE] ELR migration for Thirdeye

2020-05-19 Thread GitBox
harleyjj commented on a change in pull request #5282: URL: https://github.com/apache/incubator-pinot/pull/5282#discussion_r427757075 ## File path: thirdeye/thirdeye-frontend/package.json ## @@ -88,7 +87,7 @@ "toastr": "^2.1.4" }, "engines": { -"node": "10.*" +

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5406: Adding support to execute functions during query compilation

2020-05-19 Thread GitBox
fx19880617 commented on a change in pull request #5406: URL: https://github.com/apache/incubator-pinot/pull/5406#discussion_r427744783 ## File path: pinot-common/src/main/java/org/apache/pinot/common/function/ScalarFunctionType.java ## @@ -0,0 +1,69 @@ +/** + * Licensed to

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5406: Adding support to execute functions during query compilation

2020-05-19 Thread GitBox
fx19880617 commented on a change in pull request #5406: URL: https://github.com/apache/incubator-pinot/pull/5406#discussion_r427744550 ## File path: pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java ## @@ -615,7 +618,47 @@ private static Expression