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

ASF GitHub Bot commented on HDFS-17140:
---------------------------------------

slfan1989 commented on PR #5924:
URL: https://github.com/apache/hadoop/pull/5924#issuecomment-1672614747

   > @Hexiaoqiao @slfan1989 This improvement may not have much effect, but it 
does reduce the creation of redundant and useless temporary objects. At the 
same time, from the perspective of code style, the implementation here is not 
elegant.
   
   @Hexiaoqiao Thank you very much for your explanation, I have understood this 
change and can accept it, thank you again for your contribution!




> Optimize the BPOfferService.reportBadBlocks() method
> ----------------------------------------------------
>
>                 Key: HDFS-17140
>                 URL: https://issues.apache.org/jira/browse/HDFS-17140
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: datanode
>            Reporter: Liangjun He
>            Assignee: Liangjun He
>            Priority: Minor
>              Labels: pull-request-available
>
> The current BPOfferService.reportBadBlocks() method can be optimized by 
> moving the creation of the rbbAction object outside the loop.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
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