[ 
https://issues.apache.org/jira/browse/HDFS-10787?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Anatoli Shein updated HDFS-10787:
---------------------------------
    Attachment: HDFS-10787.HDFS-8707.003.patch

Thanks for the reveiw, [~James C].

* In the new patch I added logic for reading multiple directories from 
HADOOP_CONF_DIR, and logging warnings during the validation if some resources 
in them are missing.
* The examples now reuse the file system connection code from tools_common.h.
* This patch also includes a flag BUILD_SHARED_HDFSPP which is set to TRUE by 
default, but can be set to FALSE to prevent shared library from being built 
(this is needed for building LIBHDFSPP as part of an external project like ORC).
* This patch also includes a number of fixes for warnings that arise when 
building libhdfspp on OSx clang compiler, the addressed warnings include: extra 
semi-colons, unused variables, missing override keywords, implicit conversions, 
and copy elision warnings.

> libhdfs++: hdfs_configuration and configuration_loader should be accessible 
> from our public API
> -----------------------------------------------------------------------------------------------
>
>                 Key: HDFS-10787
>                 URL: https://issues.apache.org/jira/browse/HDFS-10787
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs-client
>            Reporter: Anatoli Shein
>            Assignee: James Clampffer
>         Attachments: HDFS-10787.HDFS-8707.000.patch, 
> HDFS-10787.HDFS-8707.001.patch, HDFS-10787.HDFS-8707.002.patch, 
> HDFS-10787.HDFS-8707.003.patch
>
>
> Currently, libhdfspp examples and tools all have this:
> #include "hdfspp/hdfspp.h"
> #include "common/hdfs_configuration.h"
> #include "common/configuration_loader.h"
> This is done in order to read configs and connect. We want  
> hdfs_configuration and configuration_loader to be accessible just by 
> including our hdfspp.h. One way to achieve that would be to create a builder, 
> and would include the above libraries.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to