[ 
https://issues.apache.org/jira/browse/HADOOP-19425?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17926242#comment-17926242
 ] 

ASF GitHub Bot commented on HADOOP-19425:
-----------------------------------------

slfan1989 commented on PR #7369:
URL: https://github.com/apache/hadoop/pull/7369#issuecomment-2652502050

   @cnauroth @anujmodi2021 I will continue to follow up on the CheckStyle 
issues for this PR. If the CheckStyle issues are related to method names or 
magic numbers, we will not be fixing them for now. Are there any other concerns 
with this PR? If not, we plan to merge it tomorrow. Thank you for your help 
with the verification and review!
   
   Examples of CheckStyle issues that will not be fixed for now:
   
   ```
   
./hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azure/integration/ITestAzureHugeFiles.java:259:
  public void test_040_PositionedReadHugeFile(TestInfo testInfo) throws 
Throwable {:15: Name 'test_040_PositionedReadHugeFile' must match pattern 
'^[a-z][a-zA-Z0-9]*$'. [MethodName]
   
./hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azure/integration/ITestAzureHugeFiles.java:317:
  public void test_050_readHugeFile(TestInfo testInfo) throws Throwable {:15: 
Name 'test_050_readHugeFile' must match pattern '^[a-z][a-zA-Z0-9]*$'. 
[MethodName]
   
./hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azure/integration/ITestAzureHugeFiles.java:343:
  public void test_060_openAndReadWholeFileBlocks(TestInfo testInfo) throws 
Throwable {:15: Name 'test_060_openAndReadWholeFileBlocks' must match pattern 
'^[a-z][a-zA-Z0-9]*$'. [MethodName]
   
./hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azure/integration/ITestAzureHugeFiles.java:413:
  public void test_100_renameHugeFile(TestInfo testInfo) throws Throwable {:15: 
Name 'test_100_renameHugeFile' must match pattern '^[a-z][a-zA-Z0-9]*$'. 
[MethodName]
   
./hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azure/integration/ITestAzureHugeFiles.java:441:
  public void test_999_deleteHugeFiles(TestInfo testInfo) throws IOException 
{:15: Name 'test_999_deleteHugeFiles' must match pattern '^[a-z][a-zA-Z0-9]*$'. 
[MethodName]
   
   
/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azure/ITestBlockBlobInputStream.java:514:
      assertTrue(elapsedTimeMs < 20, String.format(:34: '20' is a magic number. 
[MagicNumber]
   
./hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azure/ITestBlockBlobInputStream.java:578:
          elapsedTimeMs < 20, String.format(:27: '20' is a magic number. 
[MagicNumber]
   ```




> [ABFS] Upgrade JUnit from 4 to 5 in hadoop-azure.
> -------------------------------------------------
>
>                 Key: HADOOP-19425
>                 URL: https://issues.apache.org/jira/browse/HADOOP-19425
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: build, fs/azure, test
>    Affects Versions: 3.5.0
>            Reporter: Shilun Fan
>            Assignee: Shilun Fan
>            Priority: Major
>              Labels: pull-request-available
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to