You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-issues@hadoop.apache.org by "Virajith Jalaparti (JIRA)" <ji...@apache.org> on 2019/02/11 19:18:00 UTC

[jira] [Comment Edited] (HDFS-14263) Remove unnecessary block file exists check from FsDatasetImpl#getBlockInputStream()

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

Virajith Jalaparti edited comment on HDFS-14263 at 2/11/19 7:17 PM:
--------------------------------------------------------------------

Hi [~surendrasingh], thanks for posting this. Re  [^HDFS-14263.001.patch] , we can skip the check for {{ProvidedReplica}}. it can just be the null check. Ideally, there shouldn't be anything special we do for {{ProvidedReplica}}.

+1 once this is fixed.


was (Author: virajith):
Hi [~surendrasingh], thanks for posting this. Re  [^HDFS-14263.001.patch] , we can skip the check for {{ProvidedReplica}}. it can just be the null check. Ideally, there shouldn't be anything special we do for {{ProvidedReplica}}.

> Remove unnecessary block file exists check from FsDatasetImpl#getBlockInputStream()
> -----------------------------------------------------------------------------------
>
>                 Key: HDFS-14263
>                 URL: https://issues.apache.org/jira/browse/HDFS-14263
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: datanode
>    Affects Versions: 3.1.1
>            Reporter: Surendra Singh Lilhore
>            Assignee: Surendra Singh Lilhore
>            Priority: Major
>         Attachments: HDFS-14263.001.patch
>
>
> As discussed in HDFS-10636, {{FsDatasetImpl#getBlockInputStream()}} doing unnecessary block replica exist check.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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