[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5513: Return Select only Literal Expression at Broker

2020-06-11 Thread GitBox
fx19880617 commented on a change in pull request #5513: URL: https://github.com/apache/incubator-pinot/pull/5513#discussion_r439214719 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java ## @@ -184,6 +192,18 @@ public

[incubator-pinot] branch pinot_query_literal_only updated (dbb957f -> 186c73d)

2020-06-11 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch pinot_query_literal_only in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard dbb957f Address comments discard 7cdd8d6 Support Selection Literal only queries

[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #5543: Added configurable http listeners for controller API

2020-06-11 Thread GitBox
jackjlli commented on a change in pull request #5543: URL: https://github.com/apache/incubator-pinot/pull/5543#discussion_r439202555 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java ## @@ -29,21 +29,28 @@ import

[GitHub] [incubator-pinot] codecov-commenter edited a comment on pull request #5547: Update year in NOTICE-binary

2020-06-11 Thread GitBox
codecov-commenter edited a comment on pull request #5547: URL: https://github.com/apache/incubator-pinot/pull/5547#issuecomment-643058130 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5547?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5513: Return Select only Literal Expression at Broker

2020-06-11 Thread GitBox
fx19880617 commented on a change in pull request #5513: URL: https://github.com/apache/incubator-pinot/pull/5513#discussion_r439201178 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java ## @@ -317,6

[GitHub] [incubator-pinot] codecov-commenter commented on pull request #5547: Update year in NOTICE-binary

2020-06-11 Thread GitBox
codecov-commenter commented on pull request #5547: URL: https://github.com/apache/incubator-pinot/pull/5547#issuecomment-643058130 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5547?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] [incubator-pinot] haibow opened a new pull request #5547: Update year in NOTICE-binary

2020-06-11 Thread GitBox
haibow opened a new pull request #5547: URL: https://github.com/apache/incubator-pinot/pull/5547 ## Description Update year in NOTICE-binary Ref: https://github.com/apache/spark/blob/master/NOTICE-binary ## Upgrade Notes Does this PR prevent a zero down-time upgrade?

[incubator-pinot] branch update_license_binary created (now fe1d02f)

2020-06-11 Thread haibow
This is an automated email from the ASF dual-hosted git repository. haibow pushed a change to branch update_license_binary in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. at fe1d02f Update year in NOTICE-binary This branch includes the following new commits:

[incubator-pinot] 01/01: Update year in NOTICE-binary

2020-06-11 Thread haibow
This is an automated email from the ASF dual-hosted git repository. haibow pushed a commit to branch update_license_binary in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git commit fe1d02f044864a29b2bb97fd22641eb0c4c80a08 Author: haibow AuthorDate: Thu Jun 11 20:37:18 2020

[GitHub] [incubator-pinot] codecov-commenter edited a comment on pull request #5544: Add SCALAR type into TransformFunctionType

2020-06-11 Thread GitBox
codecov-commenter edited a comment on pull request #5544: URL: https://github.com/apache/incubator-pinot/pull/5544#issuecomment-643034473 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5544?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] [incubator-pinot] codecov-commenter commented on pull request #5544: Add SCALAR type into TransformFunctionType

2020-06-11 Thread GitBox
codecov-commenter commented on pull request #5544: URL: https://github.com/apache/incubator-pinot/pull/5544#issuecomment-643034473 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5544?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] [incubator-pinot] snleee commented on a change in pull request #5546: Handle case sensitivity for PinotQuery within BrokerRequest

2020-06-11 Thread GitBox
snleee commented on a change in pull request #5546: URL: https://github.com/apache/incubator-pinot/pull/5546#discussion_r439176537 ## File path: pinot-common/src/main/java/org/apache/pinot/common/utils/CommonConstants.java ## @@ -40,6 +40,7 @@ public static final String

[GitHub] [incubator-pinot] codecov-commenter commented on pull request #5546: Handle case sensitivity for PinotQuery within BrokerRequest

2020-06-11 Thread GitBox
codecov-commenter commented on pull request #5546: URL: https://github.com/apache/incubator-pinot/pull/5546#issuecomment-643030356 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5546?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #5546: Handle case sensitivity for PinotQuery within BrokerRequest

2020-06-11 Thread GitBox
Jackie-Jiang opened a new pull request #5546: URL: https://github.com/apache/incubator-pinot/pull/5546 ## Description Currently the case sensitivity handling only applies to the BrokerRequest, but not to the PinotQuery inside the BrokerRequest. This will cause inconsistent and

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5513: Return Select only Literal Expression at Broker

2020-06-11 Thread GitBox
Jackie-Jiang commented on a change in pull request #5513: URL: https://github.com/apache/incubator-pinot/pull/5513#discussion_r439156588 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java ## @@ -184,6 +192,18 @@

[GitHub] [incubator-pinot] akshayrai opened a new pull request #5545: [TE][subscription] Bug fix to fix the notification pipeline when detections are disabled

2020-06-11 Thread GitBox
akshayrai opened a new pull request #5545: URL: https://github.com/apache/incubator-pinot/pull/5545 Subscription pipeline has an issue of skipping some anomaly notification when one/more of the subscribed detections are disabled. This PR fixes this bug.

[GitHub] [incubator-pinot] codecov-commenter edited a comment on pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
codecov-commenter edited a comment on pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#issuecomment-642283701 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5483?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-commenter edited a comment on pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
codecov-commenter edited a comment on pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#issuecomment-642283701 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5483?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #5544: Add SCALAR type into TransformFunctionType

2020-06-11 Thread GitBox
Jackie-Jiang opened a new pull request #5544: URL: https://github.com/apache/incubator-pinot/pull/5544 ## Description Inside query engine, scalar function is modeled as TransformFunction (ScalarTransformFunctionWrapper) Add SCALAR type into TransformFunctionType so that

[GitHub] [incubator-pinot] kishoreg closed issue #5356: Is DataType.BYTES control in aggregate() aggregateGroupBySV() aggregateGroupByMV() needed?

2020-06-11 Thread GitBox
kishoreg closed issue #5356: URL: https://github.com/apache/incubator-pinot/issues/5356 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

[GitHub] [incubator-pinot] kishoreg commented on issue #5356: Is DataType.BYTES control in aggregate() aggregateGroupBySV() aggregateGroupByMV() needed?

2020-06-11 Thread GitBox
kishoreg commented on issue #5356: URL: https://github.com/apache/incubator-pinot/issues/5356#issuecomment-642992726 Not an issue This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-pinot] kishoreg commented on issue #5475: GuavaOnHeapBloomFilter can be constructed with _bloomFilter as null

2020-06-11 Thread GitBox
kishoreg commented on issue #5475: URL: https://github.com/apache/incubator-pinot/issues/5475#issuecomment-642992198 @snleee is this valid? This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5513: Return Select only Literal Expression at Broker

2020-06-11 Thread GitBox
Jackie-Jiang commented on a change in pull request #5513: URL: https://github.com/apache/incubator-pinot/pull/5513#discussion_r439136268 ## File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java ## @@ -317,6

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5513: Return Select only Literal Expression at Broker

2020-06-11 Thread GitBox
Jackie-Jiang commented on a change in pull request #5513: URL: https://github.com/apache/incubator-pinot/pull/5513#discussion_r439135181 ## File path: pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java ## @@ -278,7 +278,9 @@ private static

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
Jackie-Jiang commented on a change in pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r439133541 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/request/context/FunctionInfo.java ## @@ -0,0 +1,97 @@ +/** + * Licensed

[GitHub] [incubator-pinot] kishoreg commented on a change in pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
kishoreg commented on a change in pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r439082867 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/request/context/FunctionInfo.java ## @@ -0,0 +1,97 @@ +/** + * Licensed to

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5543: Added configurable http listeners for controller API

2020-06-11 Thread GitBox
mcvsubbu commented on a change in pull request #5543: URL: https://github.com/apache/incubator-pinot/pull/5543#discussion_r439132149 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/ControllerConf.java ## @@ -29,21 +29,28 @@ import

[GitHub] [incubator-pinot] daniellavoie commented on pull request #5543: Added configurable http listeners for controller API

2020-06-11 Thread GitBox
daniellavoie commented on pull request #5543: URL: https://github.com/apache/incubator-pinot/pull/5543#issuecomment-642984859 Thank you for the helix context @mcvsubbu! This is helpful. This PR is still a wip, I need to fix some licensing issues and some unit tests regression. I shared

[GitHub] [incubator-pinot] mcvsubbu commented on pull request #5543: Added configurable http listeners for controller API

2020-06-11 Thread GitBox
mcvsubbu commented on pull request #5543: URL: https://github.com/apache/incubator-pinot/pull/5543#issuecomment-642983139 Controller instance ID is used in the (older) way of running helix controller as well as Pinot controller in the same host (Dual mode). Or, when running helix

[GitHub] [incubator-pinot] kishoreg commented on pull request #5543: Added configurable http listeners for controller API

2020-06-11 Thread GitBox
kishoreg commented on pull request #5543: URL: https://github.com/apache/incubator-pinot/pull/5543#issuecomment-642968387 Thanks @daniellavoie Can you paste sample configurations for the following scenarios - Current code - After this PR with new properties and old properties

[GitHub] [incubator-pinot] daniellavoie opened a new pull request #5543: Added configurable http listeners for controller API

2020-06-11 Thread GitBox
daniellavoie opened a new pull request #5543: URL: https://github.com/apache/incubator-pinot/pull/5543 ## Description Introduced new properties to define dynamic listeners to HttpServer. The `controller.port` property is still available and intended to remain supported until future

[incubator-pinot] branch master updated (55b4ddc -> f1f1c41)

2020-06-11 Thread mcvsubbu
This is an automated email from the ASF dual-hosted git repository. mcvsubbu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. from 55b4ddc Change default segment load mode to MMAP. (#5539) add f1f1c41 Changed the stream and

[GitHub] [incubator-pinot] mcvsubbu merged pull request #5542: Changed the stream and metadata interface

2020-06-11 Thread GitBox
mcvsubbu merged pull request #5542: URL: https://github.com/apache/incubator-pinot/pull/5542 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] mcvsubbu commented on a change in pull request #5542: Changed the stream and metadata interface

2020-06-11 Thread GitBox
mcvsubbu commented on a change in pull request #5542: URL: https://github.com/apache/incubator-pinot/pull/5542#discussion_r439088617 ## File path: pinot-controller/src/test/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManagerTest.java ## @@

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5542: Changed the stream and metadata interface

2020-06-11 Thread GitBox
mcvsubbu commented on a change in pull request #5542: URL: https://github.com/apache/incubator-pinot/pull/5542#discussion_r439069170 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java ## @@ -893,20

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5542: Changed the stream and metadata interface

2020-06-11 Thread GitBox
mcvsubbu commented on a change in pull request #5542: URL: https://github.com/apache/incubator-pinot/pull/5542#discussion_r439068418 ## File path: pinot-controller/src/test/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManagerTest.java ## @@

[GitHub] [incubator-pinot] kishoreg commented on a change in pull request #5542: Changed the stream and metadata interface

2020-06-11 Thread GitBox
kishoreg commented on a change in pull request #5542: URL: https://github.com/apache/incubator-pinot/pull/5542#discussion_r439062405 ## File path: pinot-controller/src/test/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManagerTest.java ## @@

[GitHub] [incubator-pinot] codecov-commenter commented on pull request #5542: Changed the stream and metadata interface

2020-06-11 Thread GitBox
codecov-commenter commented on pull request #5542: URL: https://github.com/apache/incubator-pinot/pull/5542#issuecomment-642905349 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5542?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] [incubator-pinot] mcvsubbu opened a new pull request #5542: Changed the stream and metadata interface

2020-06-11 Thread GitBox
mcvsubbu opened a new pull request #5542: URL: https://github.com/apache/incubator-pinot/pull/5542 Changed the interface exposed by streams to return StreamPartitionMsgOffset instead of long. Also changed the LLCRealtimeSegmentZKMetadat class to return String instead of long

[GitHub] [incubator-pinot] Jackie-Jiang commented on pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
Jackie-Jiang commented on pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#issuecomment-642848504 > It seems like we are adding some duplicate code (like Predicate, FilterInfo). We already have them. > > I agree that on server we can move to QueryRequest

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
Jackie-Jiang commented on a change in pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r438974956 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/request/context/QueryContext.java ## @@ -0,0 +1,199 @@ +/** + * Licensed

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
Jackie-Jiang commented on a change in pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r438971580 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/request/context/FilterInfo.java ## @@ -0,0 +1,116 @@ +/** + * Licensed

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
Jackie-Jiang commented on a change in pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r438971580 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/request/context/FilterInfo.java ## @@ -0,0 +1,116 @@ +/** + * Licensed

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
Jackie-Jiang commented on a change in pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r438964215 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/request/context/Expression.java ## @@ -0,0 +1,119 @@ +/** + * Licensed

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
Jackie-Jiang commented on a change in pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r438964215 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/request/context/Expression.java ## @@ -0,0 +1,119 @@ +/** + * Licensed

[GitHub] [incubator-pinot] kishoreg commented on pull request #5539: Change default segment load mode to MMAP.

2020-06-11 Thread GitBox
kishoreg commented on pull request #5539: URL: https://github.com/apache/incubator-pinot/pull/5539#issuecomment-642810922 sorry missed this. why do we have another variable for read mode, I thought we already had something in table config called load mode and it was MMAP by default

[incubator-pinot] branch master updated: Change default segment load mode to MMAP. (#5539)

2020-06-11 Thread mayanks
This is an automated email from the ASF dual-hosted git repository. mayanks pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git The following commit(s) were added to refs/heads/master by this push: new 55b4ddc Change default segment load

[GitHub] [incubator-pinot] mayankshriv merged pull request #5539: Change default segment load mode to MMAP.

2020-06-11 Thread GitBox
mayankshriv merged pull request #5539: URL: https://github.com/apache/incubator-pinot/pull/5539 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] siddharthteotia edited a comment on pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
siddharthteotia edited a comment on pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#issuecomment-642662988 It seems like we are adding some duplicate code (like Predicate, FilterInfo). We already have them. I agree that on server we can move to

[GitHub] [incubator-pinot] siddharthteotia edited a comment on pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
siddharthteotia edited a comment on pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#issuecomment-642662988 It seems like we are adding some duplicate code (like Predicate, FilterInfo). We already have them. I agree that on server we can move to

[GitHub] [incubator-pinot] haibow closed issue #5540: Remove tgz files from source package

2020-06-11 Thread GitBox
haibow closed issue #5540: URL: https://github.com/apache/incubator-pinot/issues/5540 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] siddharthteotia commented on pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
siddharthteotia commented on pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#issuecomment-642662988 It seems like we are adding a some duplicate code (like Predicate, FilterInfo). We already have them. I agree that on server we can move to QueryRequest

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
siddharthteotia commented on a change in pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r438794993 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/request/context/FilterInfo.java ## @@ -0,0 +1,116 @@ +/** + *

[GitHub] [incubator-pinot] siddharthteotia commented on a change in pull request #5483: Add QueryContext to replace BrokerRequest in the query engine

2020-06-11 Thread GitBox
siddharthteotia commented on a change in pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#discussion_r438794883 ## File path: pinot-core/src/main/java/org/apache/pinot/core/query/request/context/Expression.java ## @@ -0,0 +1,119 @@ +/** + *

[GitHub] [incubator-pinot] snleee merged pull request #5541: Remove '/docker', '/kubernetes', '/website' for src release

2020-06-11 Thread GitBox
snleee merged pull request #5541: URL: https://github.com/apache/incubator-pinot/pull/5541 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: Remove '/docker', '/kubernetes', '/website' for src release (#5541)

2020-06-11 Thread snlee
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 3c42e5e Remove '/docker',

[GitHub] [incubator-pinot] codecov-commenter edited a comment on pull request #5513: Return Select only Literal Expression at Broker

2020-06-11 Thread GitBox
codecov-commenter edited a comment on pull request #5513: URL: https://github.com/apache/incubator-pinot/pull/5513#issuecomment-641655758 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5513?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-commenter edited a comment on pull request #5513: Return Select only Literal Expression at Broker

2020-06-11 Thread GitBox
codecov-commenter edited a comment on pull request #5513: URL: https://github.com/apache/incubator-pinot/pull/5513#issuecomment-641655758 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5513?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-commenter commented on pull request #5541: Remove '/docker', '/kubernetes', '/website' for src release

2020-06-11 Thread GitBox
codecov-commenter commented on pull request #5541: URL: https://github.com/apache/incubator-pinot/pull/5541#issuecomment-642531728 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5541?src=pr=h1) Report > Merging

[GitHub] [incubator-pinot] codecov-commenter edited a comment on pull request #5534: Replace SegmentMergeLineage to SegmentLineage

2020-06-11 Thread GitBox
codecov-commenter edited a comment on pull request #5534: URL: https://github.com/apache/incubator-pinot/pull/5534#issuecomment-642525270 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5534?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] [incubator-pinot] codecov-commenter commented on pull request #5534: Replace SegmentMergeLineage to SegmentLineage

2020-06-11 Thread GitBox
codecov-commenter commented on pull request #5534: URL: https://github.com/apache/incubator-pinot/pull/5534#issuecomment-642525270 # [Codecov](https://codecov.io/gh/apache/incubator-pinot/pull/5534?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[incubator-pinot] branch pinot_query_literal_only updated (c682c95 -> dbb957f)

2020-06-11 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch pinot_query_literal_only in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard c682c95 Address comments discard 6cc878c Support Selection Literal only queries

[GitHub] [incubator-pinot] snleee commented on issue #5540: Remove tgz files from source package

2020-06-11 Thread GitBox
snleee commented on issue #5540: URL: https://github.com/apache/incubator-pinot/issues/5540#issuecomment-642502255 Addressed in the #5541 please take a look This is an automated message from the Apache Git Service. To

[GitHub] [incubator-pinot] snleee opened a new pull request #5541: Remove '/docker', '/kubernetes', '/website' for src release

2020-06-11 Thread GitBox
snleee opened a new pull request #5541: URL: https://github.com/apache/incubator-pinot/pull/5541 According to #5540, our source release included some binary files and it has been flagged during the review process for the release. We have decided to exclude some dirs from the

[GitHub] [incubator-pinot] fx19880617 commented on issue #5540: Remove tgz files from source package

2020-06-11 Thread GitBox
fx19880617 commented on issue #5540: URL: https://github.com/apache/incubator-pinot/issues/5540#issuecomment-642448052 We should exclude: ``` ./docker ./kubernetes ./website ``` This is an automated message