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 2019/05/21 10:04:53 UTC

[GitHub] [incubator-druid] viongpanzi opened a new pull request #7716: AggregatorUtil should cache parsed expression to avoid memory problem (OOM/FGC) when Expression is used in metricsSpec

viongpanzi opened a new pull request #7716: AggregatorUtil should cache parsed expression to avoid memory problem (OOM/FGC) when Expression is used in metricsSpec
URL: https://github.com/apache/incubator-druid/pull/7716
 
 
   Fix #7715 
   
   In this pull request:
   1. Add parsed expression cache in AggregatorUtil
   2. Add some test cases
   3. Add expression schema in BenchmarkSchemas 
   
   With new code, steps in #7715 used to reproduce the issue are all successful and complete fast.
   
   The benchmark result show that the performance of Expression processing is twice as fast than before if memory is sufficient.

----------------------------------------------------------------
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