You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/12/22 02:05:44 UTC
[GitHub] [druid] beepupa opened a new issue #10702: DoubleFirstAggregatorFactory is not supported during ingestion for rollup
beepupa opened a new issue #10702:
URL: https://github.com/apache/druid/issues/10702
version:0.20
----------------------------------------------------------------
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.
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] beepupa commented on issue #10702: DoubleFirstAggregatorFactory is not supported during ingestion for rollup
Posted by GitBox <gi...@apache.org>.
beepupa commented on issue #10702:
URL: https://github.com/apache/druid/issues/10702#issuecomment-750699167
> According to the [doc](https://github.com/apache/druid/blame/master/docs/querying/aggregations.md#L137),
>
> > (Double/Float/Long) First and Last aggregator cannot be used in ingestion spec, and should only be specified as part of queries.
>
> But the web console DOES allow users to configure a metric aggregated as double/float/long first/last.
> 
>
> And from the current implementation of backend services, these aggregation are not supported.
>
> I checked the history of code, and didn't find the reasons why these aggregators are not supported during ingestion for rollup.
>
> Do you have more background information on this problem @gianm ?
Thank you for answering my question
----------------------------------------------------------------
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.
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] FrankChen021 commented on issue #10702: DoubleFirstAggregatorFactory is not supported during ingestion for rollup
Posted by GitBox <gi...@apache.org>.
FrankChen021 commented on issue #10702:
URL: https://github.com/apache/druid/issues/10702#issuecomment-750696742
@suneet-s I see. Are there any progress being made on long/double first/last ? If no, I'd like to take the chance to implement it.
----------------------------------------------------------------
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.
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] FrankChen021 commented on issue #10702: DoubleFirstAggregatorFactory is not supported during ingestion for rollup
Posted by GitBox <gi...@apache.org>.
FrankChen021 commented on issue #10702:
URL: https://github.com/apache/druid/issues/10702#issuecomment-749898138
According to the [doc](https://github.com/apache/druid/blame/master/docs/querying/aggregations.md#L137),
> (Double/Float/Long) First and Last aggregator cannot be used in ingestion spec, and should only be specified as part of queries.
But the web console DOES allow users to configure a metric aggregated as double/float/long first/last.
And from the current implementation of backend services, these aggregation are not supported.
I checked the history of code, and didn't find the reasons why these aggregators are not supported during ingestion for rollup.
Do you have more background information on this problem @gianm ?
----------------------------------------------------------------
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.
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] beepupa commented on issue #10702: DoubleFirstAggregatorFactory is not supported during ingestion for rollup
Posted by GitBox <gi...@apache.org>.
beepupa commented on issue #10702:
URL: https://github.com/apache/druid/issues/10702#issuecomment-750699228
> @FrankChen021 there's some context in #7599 and #10332
>
> The web console shouldn't offer these as options since they are not yet supported at ingestion time.
>
> cc @vogievetsky
Thank you for answering my question
----------------------------------------------------------------
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.
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] suneet-s commented on issue #10702: DoubleFirstAggregatorFactory is not supported during ingestion for rollup
Posted by GitBox <gi...@apache.org>.
suneet-s commented on issue #10702:
URL: https://github.com/apache/druid/issues/10702#issuecomment-750372981
@FrankChen021 there's some context in #7599 and #10332
The web console shouldn't offer these as options since they are not yet supported at ingestion time.
cc @vogievetsky
----------------------------------------------------------------
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.
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] suneet-s commented on issue #10702: DoubleFirstAggregatorFactory is not supported during ingestion for rollup
Posted by GitBox <gi...@apache.org>.
suneet-s commented on issue #10702:
URL: https://github.com/apache/druid/issues/10702#issuecomment-750765915
> @suneet-s I see. Are there any progress being made on long/double first/last ? If no, I'd like to take the chance to implement it.
@FrankChen021 I'm not aware of anyone working on these issues. Feel free to pick it up - I reviewed the implementation for StringFirst/Last, and would be happy to review these as well
----------------------------------------------------------------
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.
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