[ https://issues.apache.org/jira/browse/HDFS-9118?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
James Clampffer updated HDFS-9118: ---------------------------------- Attachment: HDFS-9118.HDFS-8707.003.patch New patch, incorporates a lot of [~bobhansen]'s suggestions. -rebased onto head -log macros to avoid creating LogMessage objects if they won't be logged -add a separate public header for the C logging data structures and defines so that the implementation doesn't have to drag in the whole public API -get rid of lock macros -more helpful output when null pointers are passed to operator<<(const char*) and operator<<(std::string*) -remove some garbage that slipped into the last patch (stuff commented out but not deleted) > Add logging system for libdhfs++ > -------------------------------- > > Key: HDFS-9118 > URL: https://issues.apache.org/jira/browse/HDFS-9118 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client > Affects Versions: HDFS-8707 > Reporter: Bob Hansen > Assignee: James Clampffer > Attachments: HDFS-9118.HDFS-8707.000.patch, > HDFS-9118.HDFS-8707.001.patch, HDFS-9118.HDFS-8707.002.patch, > HDFS-9118.HDFS-8707.003.patch > > > With HDFS-9505, we've starting logging data from libhdfs++. Consumers of the > library are going to have their own logging infrastructure that we're going > to want to provide data to. > libhdfs++ should have a logging library that: > * Is overridable and can provide sufficient information to work well with > common C++ logging frameworks > * Has a rational default implementation > * Is performant -- This message was sent by Atlassian JIRA (v6.3.4#6332)