HADOOP-19617 - Improve Some Code.

slfan1989 · slfan1989 · commit edcf6f7bf9db · 2025-08-18T10:51:15.000+08:00
diff --git a/hadoop-cloud-storage-project/hadoop-huaweicloud/src/test/java/org/apache/hadoop/fs/obs/TestOBSContractCreate.java b/hadoop-cloud-storage-project/hadoop-huaweicloud/src/test/java/org/apache/hadoop/fs/obs/TestOBSContractCreate.java
@@ -27,18 +27,19 @@
 /**
  * Create test cases on obs file system.
  */
-@Disabled
 public class TestOBSContractCreate extends AbstractContractCreateTest {
 
   @Override
   protected AbstractFSContract createContract(final Configuration conf) {
     return new OBSContract(conf);
   }
 
+  @Disabled
   @Override
   public void testCreatedFileIsImmediatelyVisible() {
   }
 
+  @Disabled
   @Override
   public void testCreatedFileIsVisibleOnFlush() {
   }
diff --git a/hadoop-cloud-storage-project/hadoop-huaweicloud/src/test/java/org/apache/hadoop/fs/obs/TestOBSContractRename.java b/hadoop-cloud-storage-project/hadoop-huaweicloud/src/test/java/org/apache/hadoop/fs/obs/TestOBSContractRename.java
@@ -26,18 +26,19 @@
 /**
  * Rename test cases on obs file system.
  */
-@Disabled
 public class TestOBSContractRename extends AbstractContractRenameTest {
 
   @Override
   protected AbstractFSContract createContract(final Configuration conf) {
     return new OBSContract(conf);
   }
 
+  @Disabled
   @Override
   public void testRenameFileUnderFileSubdir() {
   }
 
+  @Disabled
   @Override
   public void testRenameFileUnderFile() {
   }

Original file line number	Diff line number	Diff line change
`@@ -27,18 +27,19 @@`
`27`	`27`	`/**`
`28`	`28`	`* Create test cases on obs file system.`
`29`	`29`	`*/`
`30`		`-@Disabled`
`31`	`30`	`public class TestOBSContractCreate extends AbstractContractCreateTest {`
`32`	`31`
`33`	`32`	`@Override`
`34`	`33`	`protected AbstractFSContract createContract(final Configuration conf) {`
`35`	`34`	`return new OBSContract(conf);`
`36`	`35`	`}`
`37`	`36`
	`37`	`+ @Disabled`
`38`	`38`	`@Override`
`39`	`39`	`public void testCreatedFileIsImmediatelyVisible() {`
`40`	`40`	`}`
`41`	`41`
	`42`	`+ @Disabled`
`42`	`43`	`@Override`
`43`	`44`	`public void testCreatedFileIsVisibleOnFlush() {`
`44`	`45`	`}`
Original file line number	Diff line number	Diff line change
`@@ -26,18 +26,19 @@`
`26`	`26`	`/**`
`27`	`27`	`* Rename test cases on obs file system.`
`28`	`28`	`*/`
`29`		`-@Disabled`
`30`	`29`	`public class TestOBSContractRename extends AbstractContractRenameTest {`
`31`	`30`
`32`	`31`	`@Override`
`33`	`32`	`protected AbstractFSContract createContract(final Configuration conf) {`
`34`	`33`	`return new OBSContract(conf);`
`35`	`34`	`}`
`36`	`35`
	`36`	`+ @Disabled`
`37`	`37`	`@Override`
`38`	`38`	`public void testRenameFileUnderFileSubdir() {`
`39`	`39`	`}`
`40`	`40`
	`41`	`+ @Disabled`
`41`	`42`	`@Override`
`42`	`43`	`public void testRenameFileUnderFile() {`
`43`	`44`	`}`