Github user pvillard31 commented on the issue:

    https://github.com/apache/nifi/pull/2430
  
    @mattyb149 - this is not the final version, I still have unit tests and 
"additional details" doc to add, but wanted to give you an update in case you 
want to have a look and if you already have feedbacks.
    
    I've reworked the new reporting task as I suggested with two additional 
properties. I also reworked the way metrics are constructed to have the ambari 
format and the "record" format with the minimal code duplication. I also chose 
this implementation (using the JsonTreeReader) so that we can easily reuse the 
code for the other S2S reporting task. If it sounds OK to you, adding an 
optional Record Writer property to the other S2S reporting tasks should be 
really easy (I guess best is to take care of it into a follow-up JIRA).


---

Reply via email to