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/03/26 18:40:48 UTC
[GitHub] [druid] himanshug edited a comment on issue #9564: Framework for
aggregate testing; Example tests with LongSumAggregatorFactory
himanshug edited a comment on issue #9564: Framework for aggregate testing; Example tests with LongSumAggregatorFactory
URL: https://github.com/apache/druid/pull/9564#issuecomment-604608102
+1 , since it improves the current state. however, even after this there is significant effort required to thoroughly test a new aggregator that only has even primitive type intermediate form :)
FYI to contributors coming here: there is another common utility `AggregationTestHelper` that serves the purpose of writing tests that tries to simulate use of aggregator in real cluster .
----------------------------------------------------------------
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
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org