Test changes

Author: shameersss1

hadoop-tools/hadoop-aws/src/site/markdown/tools/hadoop-aws/index.md

@@ -31,6 +31,12 @@ before 2021.
 Consult [S3A and Directory Markers](directory_markers.html) for
 full details.
 
+### <a name="bucket-name-compatibility"></a> S3 Bucket Name Compatibility
+
+This release adds support for S3 bucket names containing dots followed by numbers
+(e.g., `my-bucket-v1.1`, `data-store.v2.3`). Previous versions of the Hadoop S3A
+client failed to initialize such buckets due to URI parsing limitations.
+
 ## <a name="documents"></a> Documents
 
 * [Connecting](./connecting.html)

hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/ITestS3AConfiguration.java

@@ -606,17 +606,6 @@ public void testConfOptionPropagationToFS() throws Exception {
     assertOptionEquals(updated, "fs.s3a.propagation", "propagated");
   }
 
-  @Test
-  public void testBucketNameWithDotAndNumber() throws Exception {
-    Configuration config = new Configuration();
-    Path path = new Path("s3a://test-bucket-v1.1");
-    try (FileSystem fs = path.getFileSystem(config)) {
-      assertThat(fs instanceof S3AFileSystem)
-          .describedAs("FileSystem should be S3AFileSystem instance")
-          .isTrue();
-    }
-  }
-
   @Test
   @Timeout(10)
   public void testS3SpecificSignerOverride() throws Exception {

hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/TestBucketConfiguration.java

@@ -28,6 +28,7 @@
 import org.junit.jupiter.api.io.TempDir;
 
 import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.fs.FileSystem;
 import org.apache.hadoop.fs.s3a.auth.delegation.EncryptionSecrets;
 import org.apache.hadoop.fs.s3native.S3xLoginHelper;
 import org.apache.hadoop.security.ProviderUtils;
@@ -94,6 +95,38 @@ public void testS3xLoginHelperWithDotInBucketName() throws Throwable {
     assertEquals("s3a://bucket.v1.2.test", multiDotResult.toString());
   }
 
+  @Test
+  public void testBucketNameWithDotAndNumber() throws Exception {
+    Configuration config = new Configuration();
+    URI uri = URI.create("s3a://test-bucket-v1.1");
+    String bucket = uri.getHost();
+    if (bucket == null) {
+      bucket = uri.getAuthority();
+    }
+    assertThat(bucket)
+        .describedAs("Bucket name should be extracted correctly")
+        .isEqualTo("test-bucket-v1.1");
+  }
+
+  @Test
+  public void testFileSystemCacheForBucketWithDotAndNumber() throws Exception {
+    Configuration config = new Configuration();
+    URI uri1 = URI.create("s3a://test-bucket-v1.1");
+    URI uri2 = URI.create("s3a://test-bucket-v1.2");
+    
+    FileSystem fs1a = FileSystem.get(uri1, config);
+    FileSystem fs1b = FileSystem.get(uri1, config);
+    FileSystem fs2 = FileSystem.get(uri2, config);
+    
+    assertThat(fs1a)
+        .describedAs("FileSystem.get should return same cached instance for same URI")
+        .isSameAs(fs1b);
+    
+    assertThat(fs1a)
+        .describedAs("FileSystem.get should return different instance for different bucket")
+        .isNotSameAs(fs2);
+  }
+
   @Test
   public void testBucketConfigurationPropagation() throws Throwable {
     Configuration config = new Configuration(false);