[GitHub] [incubator-pinot] akshayrai opened a new issue #5521: [Preview][UI] Request top make the default preview end time set to the dataset watermark

akshayrai opened a new issue #5521: URL: https://github.com/apache/incubator-pinot/issues/5521 Can we default the preview time-window to the max dataset timestamp so that users can always preview some data? **Priority:** Low **Possible Solution:** Query the end time of the dataset

[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #5501: Support escaping single quote for SQL literal

Jackie-Jiang merged pull request #5501: URL: https://github.com/apache/incubator-pinot/pull/5501 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

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

fx19880617 commented on a change in pull request #5513: URL: https://github.com/apache/incubator-pinot/pull/5513#discussion_r436894620 ## File path: pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java ## @@ -676,14 +678,25 @@ protected static Expressi

[GitHub] [incubator-pinot] fx19880617 commented on pull request #5438: Add additional datetime functionality

fx19880617 commented on pull request #5438: URL: https://github.com/apache/incubator-pinot/pull/5438#issuecomment-640925133 Could you try to resolve the conflict and then we can merge this. This is an automated message from t

[GitHub] [incubator-pinot] akshayrai opened a new issue #5520: [Preview][UI] The terms old and new on the preview table are not intuitive.

akshayrai opened a new issue #5520: URL: https://github.com/apache/incubator-pinot/issues/5520 The Detection Settings column in the preview table with Old and New values is confusing. Can we redesign it and make it more intuitive? **Priority:** Low **Solution:** “Old” -> “Anomalie

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5508: In Travis quickstart, wait for process finishing clean up and terminate

mcvsubbu commented on a change in pull request #5508: URL: https://github.com/apache/incubator-pinot/pull/5508#discussion_r436891528 ## File path: .travis/.travis_quickstart.sh ## @@ -69,7 +75,7 @@ sleep 30 for i in $(seq 1 200) do COUNT_STAR_RES=`curl -X POST --header 'Ac

[GitHub] [incubator-pinot] fx19880617 commented on a change in pull request #5508: Fix Travis quickstart tests

fx19880617 commented on a change in pull request #5508: URL: https://github.com/apache/incubator-pinot/pull/5508#discussion_r437130895 ## File path: .travis/.travis_quickstart.sh ## @@ -30,60 +30,75 @@ java -version # Quickstart DIST_BIN_DIR=`ls -d pinot-distribution/target

[GitHub] [incubator-pinot] npawar commented on a change in pull request #5430: One to many records from record reader/decoder

npawar commented on a change in pull request #5430: URL: https://github.com/apache/incubator-pinot/pull/5430#discussion_r437095719 ## File path: pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/LLRealtimeSegmentDataManager.java ## @@ -466,19 +469,28 @@ priv

[GitHub] [incubator-pinot] akshayrai opened a new issue #5527: [TE-doc] Require better documentation on the items listed

akshayrai opened a new issue #5527: URL: https://github.com/apache/incubator-pinot/issues/5527 Doc - How to configure monitoringGranularity? Priority: Low Solution: Document it. Effort: Low Doc - How to add a row count metric/alert? Priority: Low Solution: Add document

[GitHub] [incubator-pinot] akshayrai opened a new issue #5518: [RCA][UI] Need a quick way to know what metrics exist on ThirdEye

akshayrai opened a new issue #5518: URL: https://github.com/apache/incubator-pinot/issues/5518 Currently there is no place to search for metrics. The drop-down in the RCA doesn't list unless we type a few characters. Can we make it list a few metrics by default? **Priority:** Low

[GitHub] [incubator-pinot] akshayrai opened a new issue #5523: [Performance-Mertric][UI] Clean up the default representation for performance metrics

akshayrai opened a new issue #5523: URL: https://github.com/apache/incubator-pinot/issues/5523 The response rate by default shows -% . We should update this and make it more clear what it means. The changes should ideally reflect all the places where the numbers are shown. **Priorit

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5508: Fix Travis quickstart tests

Jackie-Jiang commented on a change in pull request #5508: URL: https://github.com/apache/incubator-pinot/pull/5508#discussion_r437047356 ## File path: .travis/.travis_quickstart.sh ## @@ -69,7 +75,7 @@ sleep 30 for i in $(seq 1 200) do COUNT_STAR_RES=`curl -X POST --header

[GitHub] [incubator-pinot] chenboat commented on pull request #5336: [Part 4] Deep-store bypass for LLC: Add a peer to peer segment fetcher.

chenboat commented on pull request #5336: URL: https://github.com/apache/incubator-pinot/pull/5336#issuecomment-640962562 > May be you can introduce a multiple URI segment fetcher, provide all the peer segment URIs to it, and let the SegmentFetcher select a random uri in each retry?

[GitHub] [incubator-pinot] haibow opened a new pull request #5529: Update license and notice for 0.4.0 release

haibow opened a new pull request #5529: URL: https://github.com/apache/incubator-pinot/pull/5529 ## Description Update license and notice for 0.4.0 release ## Upgrade Notes Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #5506: Moved StreamPartitionMsgOffset to be an interface

mcvsubbu commented on a change in pull request #5506: URL: https://github.com/apache/incubator-pinot/pull/5506#discussion_r437526456 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java ## @@ -515,7 +5

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5461: Adding Support for SQL CASE Statement

Jackie-Jiang commented on a change in pull request #5461: URL: https://github.com/apache/incubator-pinot/pull/5461#discussion_r436907477 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/CaseTransformFunction.java ## @@ -0,0 +1,436 @@ +/*

[incubator-pinot] branch support_case_when_statement updated (41aba47 -> d6c0ca8)

This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch support_case_when_statement in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git. discard 41aba47 Address comments discard f793def Address comments discard d65cf51 Address

<    1   2