Re: [PR] Fixing array literal usage for vector [pinot]

2024-02-04 Thread via GitHub
codecov-commenter commented on PR #12365: URL: https://github.com/apache/pinot/pull/12365#issuecomment-1926000998 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/12365?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) R

[PR] Fixing array literal usage for vector [pinot]

2024-02-04 Thread via GitHub
xiangfu0 opened a new pull request, #12365: URL: https://github.com/apache/pinot/pull/12365 (no comment) -- 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 to the specific comment. To unsubscribe, e-m

Re: [I] VECTOR_SIMILARITY second operand must be a float array [pinot]

2024-02-04 Thread via GitHub
hdulay commented on issue #12359: URL: https://github.com/apache/pinot/issues/12359#issuecomment-1925912450 Here is the SQL executed in python ```sql SELECT ProductId, Summary, Score, l2_distance(embedding, ARRAY{search_embedding}) AS l2_dist from fineFoodR

(pinot) branch master updated: [Multi-stage] Ser/de stage plan in parallel (#12363)

2024-02-04 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/pinot.git The following commit(s) were added to refs/heads/master by this push: new 0a5a134fb5 [Multi-stage] Ser/de stage plan in par

Re: [PR] [Multi-stage] Ser/de stage plan in parallel [pinot]

2024-02-04 Thread via GitHub
Jackie-Jiang merged PR #12363: URL: https://github.com/apache/pinot/pull/12363 -- 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 to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@pinot

[PR] bumped calcite-core to 1.36.0 [pinot]

2024-02-04 Thread via GitHub
robertzych opened a new pull request, #12364: URL: https://github.com/apache/pinot/pull/12364 (no comment) -- 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 to the specific comment. To unsubscribe, e

Re: [I] CVEs detected in dependencies of pinot-java-client and pinot-common [pinot]

2024-02-04 Thread via GitHub
robertzych commented on issue #12341: URL: https://github.com/apache/pinot/issues/12341#issuecomment-1925824498 @snleee Of the dependencies that I upgraded, all but calcite-core hasn't been upgraded yet. I will create a PR to upgrade calcite-core to the latest version (1.36.0), but because

Re: [PR] Fix backward compatible issue in DistinctCountThetaSketchAggregationFunction [pinot]

2024-02-04 Thread via GitHub
jackjlli commented on code in PR #12347: URL: https://github.com/apache/pinot/pull/12347#discussion_r1477202071 ## pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountRawThetaSketchAggregationFunction.java: ## @@ -53,7 +53,8 @@ public String ex