Tim Armstrong created IMPALA-8578:
-------------------------------------

             Summary: Reduce code in metric headers
                 Key: IMPALA-8578
                 URL: https://issues.apache.org/jira/browse/IMPALA-8578
             Project: IMPALA
          Issue Type: Improvement
          Components: Backend
            Reporter: Tim Armstrong


metrics.h and other metric headers are included a lot of places and there is a 
lot of code in the header that has very few callers. It appears to be pulled 
into several hundred compilation units, increasing the compile time of each of 
those and forcing recompilation when the headers are changed.

Some ideas:
* Move function implementations to .cc files. E.g. ToJson() and ToPrometheus() 
don't need to be inlined.
* Move MetricGroup to its own file
* Try to see if we can use forward declarations in more places to avoid 
including it.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to