[
https://issues.apache.org/jira/browse/HDFS-14356?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16824092#comment-16824092
]
Sammi Chen commented on HDFS-14356:
-----------------------------------
Thanks [~PhiloHe] 's effort. Some initial comments,
1. Use constants or enum for pmdk status
{code:java}
+ /* Denotes the state of supporting PMDK:
+ < 0, native code build without PMDK support
+ > 0, supported by native code, but PMDK dynamic library NOT found in
+ execution environment or failed to be loaded
+ ==0, ready for use, supported by native code, and PMDK library
+ loaded successfully
+ The value is set by JNI. */
{code}
2 . Is there potential buffer overflow risk here?
{code:java}
char msg[1000];
{code}
3. Do you plan to support Windows in this patch? If not, please clarify the
supported platform in the title or in the description. Also make sure when
compile native is enabled(-Pnative), following two cases pass
* Linux platform, compile with and without PMDK enabled
* Windows platform, compile without PMDK enabled
> Implement HDFS cache on SCM with native PMDK libs
> -------------------------------------------------
>
> Key: HDFS-14356
> URL: https://issues.apache.org/jira/browse/HDFS-14356
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: caching, datanode
> Reporter: Feilong He
> Assignee: Feilong He
> Priority: Major
> Attachments: HDFS-14356.000.patch, HDFS-14356.001.patch
>
>
> In this implementation, native PMDK libs are involved in mapping HDFS blocks
> to SCM. To use this implementation, user should build hadoop with PMDK libs
> by specifying a build option.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
