ferhui commented on pull request #2416: URL: https://github.com/apache/hadoop/pull/2416#issuecomment-716935416
> Hi thanks for the patch. It would be the best if you can make the block size configurable from command argument as well. The replication factor is configurable through config as well as arguments. OK. @jojochuang Thanks for review ! There is a checkstyle problem > ./hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/NNThroughputBenchmark.java:172: protected int blockSize;:19: Variable 'blockSize' must be private and have accessor methods. [VisibilityModifier] I keep the style the same as original code. If fix this, It's different from other code, looks strange. How to deal with it ? ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org