anmolanmol1234 commented on code in PR #6069: URL: https://github.com/apache/hadoop/pull/6069#discussion_r1394192118
########## hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemChecksum.java: ########## @@ -55,7 +54,7 @@ public class ITestAzureBlobFileSystemChecksum extends AbstractAbfsIntegrationTes private static final int MB_8 = 8 * ONE_MB; private static final int MB_15 = 15 * ONE_MB; private static final int MB_16 = 16 * ONE_MB; - private static final String invalidText = "Text for Invalid MD5 Computation"; + private static final String TEXT_FOR_INVALID_MD5_COMPUTATION = "Text for Invalid MD5 Computation"; Review Comment: Can we shorten this string to only INVALID_MD5_TEXT ? -- 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: common-issues-unsubscr...@hadoop.apache.org 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
