hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bob Hansen (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-9627) libhdfs++: Add a mechanism to retrieve human readable error messages through the C API
Date Fri, 08 Jan 2016 14:39:39 GMT

    [ https://issues.apache.org/jira/browse/HDFS-9627?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15089282#comment-15089282
] 

Bob Hansen commented on HDFS-9627:
----------------------------------

Additional minor comment: for consistency, we should mark the methods as {{LIBHDFS_EXTERNAL}},
which means including {{hdfs.h}}, which means we need to disambiguate the C {{hdfs.h}} from
the libdhfdspp {{hdfs.h}} (which declares FileSystem and FileHandle).  Perhaps renaming this
patch's {{hdfspp.h}} to {{hdfs_ext.h}} and renaming {{libhdfs/include/libhdfspp/hdfs.h}} to
{{hdfspp.h}} is a good set of names?

Nothing is ever simple.

> libhdfs++: Add a mechanism to retrieve human readable error messages through the C API
> --------------------------------------------------------------------------------------
>
>                 Key: HDFS-9627
>                 URL: https://issues.apache.org/jira/browse/HDFS-9627
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs-client
>            Reporter: James Clampffer
>            Assignee: James Clampffer
>         Attachments: HDFS-9627.HDFS-8707.000.patch, HDFS-9627.HDFS-8707.000.patch
>
>
> Libhdfs doesn't have this but libhdfs3 has a "hdfsGetLastErrorString" function.  The
C API needs to be able to pass out error messages that are more specific than what errno can
provide.
> This functionality should be exposed via a new public header in order to keep hdfs.h
consistent with the libhdfs header.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message