[GitHub] nwangtw commented on issue #2972: Add InfluxDB metrics support

[GitBox <gi...@apache.org>]

nwangtw commented on issue #2972: Add InfluxDB metrics support
URL: https://github.com/apache/incubator-heron/pull/2972#issuecomment-412563956
 
 
   "I could rewrite the client to do raw HTTP requests but I would simply be replicating the work of Influx client."
   
   @Code0x58  made a good point. For a metrics sink, it is quite strange to me that so many dependencies are required. In theory, a sink client should be just a simple wrapper for a few HTTP requests, so it may only need a HTTP lib (and/or thrift requests but still). I am wondering how these dependencies are used and if there is a simpler solution.
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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