You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by GitBox <gi...@apache.org> on 2019/03/06 16:55:33 UTC

[GitHub] [hadoop] elek commented on issue #549: HDDS-1213. Support plain text S3 MPU initialization request

elek commented on issue #549: HDDS-1213. Support plain text S3 MPU initialization request
URL: https://github.com/apache/hadoop/pull/549#issuecomment-470186740
 
 
   White space problems are reported by the yetus. I committed to a fix in a separated commit (https://github.com/apache/hadoop/pull/549/commits/a086953243b71796bf06022a0775aebcfd29ea52) to make it easier to review. 
   
   I removed the platform lines (https://github.com/apache/hadoop/pull/549/commits/9f59f5d06960634fcca77b7dd5c07da70dce21ff), but to be honest, they are also independent from the patch. 
   
   So we should either accept both whitespace/platform fixes or create two new jiras for both. Just to be consistent ;-)
   
    (As they are both committed in two separated commits they can be reverted/accepted during the merge.)
   
    
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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