Github user steveloughran commented on the issue:

    https://github.com/apache/spark/pull/21071
  
    I like this, but you'll need people with authority to trigger the builds 
and reviews.
    
    There's some discussion kicked off last week on the ASF incubator about the 
fact that htrace has been dormant for a while and should it be retired...I 
think I and others would be happy to pull it into hadoop-core itself, given 
there's dependencies going in to HDFS & places. That's as an API for tracing to 
feed into things like Zipkin; nothing grand. There's a risk that the spark 
project will hold back on this until that's been clarified.
    
    Minor code issues
    * this turns HTrace on always; do you think it should be optional
    * needs some tests to  handle malformed span-ids coming in?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to