liubingxing commented on pull request #3714:
URL: https://github.com/apache/hadoop/pull/3714#issuecomment-994766378
Thanks for the review @virajjasani @Hexiaoqiao
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
liubingxing commented on pull request #3714:
URL: https://github.com/apache/hadoop/pull/3714#issuecomment-994243007
@Hexiaoqiao Please take a look.
--
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
liubingxing commented on pull request #3714:
URL: https://github.com/apache/hadoop/pull/3714#issuecomment-994196221
@virajjasani Thank you for your advice. I update the PR, please take a look.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
liubingxing commented on pull request #3714:
URL: https://github.com/apache/hadoop/pull/3714#issuecomment-991399254
@virajjasani Thanks for your review. I have fixed the checkstyle warning and
updated the PR.
--
This is an automated message from the Apache Git Service.
To respond to the
liubingxing commented on pull request #3714:
URL: https://github.com/apache/hadoop/pull/3714#issuecomment-991399254
@virajjasani Thanks for your review. I have fixed the checkstyle warning and
updated the PR.
--
This is an automated message from the Apache Git Service.
To respond to the
liubingxing commented on pull request #3714:
URL: https://github.com/apache/hadoop/pull/3714#issuecomment-990618468
@virajjasani I add UT `testGetDatanodeStorageReportWithNumBLocksNotZero `.
Please take a look.
--
This is an automated message from the Apache Git Service.
To respond to