[
https://issues.apache.org/jira/browse/HDFS-9353?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15310326#comment-15310326
]
Daniel Templeton commented on HDFS-9353:
----------------------------------------
As a native English speaker, I respectfully disagree. The comment is not just
open for misinterpretation; it's misleading. It took me several reads to
figure out how it could possibly mean what it's supposed to mean, i.e. env
first. Comments are supposed to improve the readability of the code, not
contradict it. This comment should be corrected or deleted.
> Code and comment mismatch in JavaKeyStoreProvider
> ---------------------------------------------------
>
> Key: HDFS-9353
> URL: https://issues.apache.org/jira/browse/HDFS-9353
> Project: Hadoop HDFS
> Issue Type: Task
> Reporter: nijel
> Assignee: Andras Bokor
> Priority: Trivial
>
> In
> org.apache.hadoop.crypto.key.JavaKeyStoreProvider.JavaKeyStoreProvider(URI
> uri, Configuration conf) throws IOException
> The comment mentioned is
> {code}
> // Get the password file from the conf, if not present from the user's
> // environment var
> {code}
> But the code takes the value form ENV first
> I think this make sense since the user can pass the ENV for a particular run.
> My suggestion is to change the comment
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
