You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by "gianm (via GitHub)" <gi...@apache.org> on 2023/06/03 00:29:40 UTC
[GitHub] [druid] gianm opened a new pull request, #14367: MSQ: Fix issue with rollup ingestion and aggregators with multiple names.
gianm opened a new pull request, #14367:
URL: https://github.com/apache/druid/pull/14367
The same aggregator can have two output names for a SQL like:
```
INSERT INTO foo
SELECT x, COUNT(*) AS y, COUNT(*) AS z
FROM t
GROUP BY 1
PARTITIONED BY ALL
```
In this case, the SQL planner will create a query with a single "count" aggregator mapped to output names "y" and "z". The prior MSQ code did not properly handle this case, instead throwing an error like:
```
Expected single output for query column[a0] but got [[1, 2]]
```
--
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-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] cryptoe merged pull request #14367: MSQ: Fix issue with rollup ingestion and aggregators with multiple names.
Posted by "cryptoe (via GitHub)" <gi...@apache.org>.
cryptoe merged PR #14367:
URL: https://github.com/apache/druid/pull/14367
--
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-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] LakshSingla commented on pull request #14367: MSQ: Fix issue with rollup ingestion and aggregators with multiple names.
Posted by "LakshSingla (via GitHub)" <gi...@apache.org>.
LakshSingla commented on PR #14367:
URL: https://github.com/apache/druid/pull/14367#issuecomment-1577927404
Note: This also solves the problem where there's ingest query containing a post-aggregator. (The individual aggregators don't appear in the output column, therefore column mapping for those is an empty list)
--
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-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org