CapMoon opened a new pull request, #8563:
URL: https://github.com/apache/hadoop/pull/8563

   HDFS-17939. StripedReader should only scan data blocks when initializing 
zero stripe indices
   
   <!--
     Thanks for sending a pull request!
       1. If this is your first time, please read our contributor guidelines: 
https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute
       2. Make sure your PR title starts with JIRA issue id, e.g., 
'HADOOP-17799. Your PR title ...'.
   -->
   
   ### Description of PR
   In StripedReader#initZeroStrip(), zeroStripeIndices is initialized by 
scanning all internal blocks, including parity blocks.
   
    
   
   The loop should only scan data block indices, which would make the 
implementation match the meaning and allocated size of zeroStripeIndices, and 
avoid suggesting that parity block indices may be populated as zero stripe 
indices.
    
   
   ### How was this patch tested?
   Existing reconstruction tests already cover partial block groups where zero 
data strips are padded for decoding. The proposed change should not affect 
observable behavior, since parity indices are never valid zero stripe indices 
for non-empty block groups.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to