This is an automated email from the ASF dual-hosted git repository.

abhishekrb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git


The following commit(s) were added to refs/heads/master by this push:
     new 06744d3827c Changing a string to key:value pair to fix flakiness in 
`testSerializationWithDefaults` (#15317)
06744d3827c is described below

commit 06744d3827c76617f40b0da2f73b3445853c0b4a
Author: Krishna Anandan <[email protected]>
AuthorDate: Tue Nov 14 17:59:07 2023 -0600

    Changing a string to key:value pair to fix flakiness in 
`testSerializationWithDefaults` (#15317)
    
    * + Fix for Flaky Test
    
    * + Replacing TreeMap with LinkedHashMap
    
    * + Changing data structure from LinkedHashMap to HashMap
    
    * Fixed flaky test in 
S3DataSegmentPusherConfigTest.testSerializationValidatingMaxListingLength
    
    * Minor Changes
---
 .../org/apache/druid/storage/s3/S3DataSegmentPusherConfigTest.java  | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/extensions-core/s3-extensions/src/test/java/org/apache/druid/storage/s3/S3DataSegmentPusherConfigTest.java
 
b/extensions-core/s3-extensions/src/test/java/org/apache/druid/storage/s3/S3DataSegmentPusherConfigTest.java
index c35733618a7..bf5f07efb93 100644
--- 
a/extensions-core/s3-extensions/src/test/java/org/apache/druid/storage/s3/S3DataSegmentPusherConfigTest.java
+++ 
b/extensions-core/s3-extensions/src/test/java/org/apache/druid/storage/s3/S3DataSegmentPusherConfigTest.java
@@ -29,6 +29,7 @@ import javax.validation.ConstraintViolation;
 import javax.validation.Validation;
 import javax.validation.Validator;
 import java.io.IOException;
+import java.util.Map;
 import java.util.Set;
 
 public class S3DataSegmentPusherConfigTest
@@ -51,9 +52,10 @@ public class S3DataSegmentPusherConfigTest
     String jsonConfig = "{\"bucket\":\"bucket1\",\"baseKey\":\"dataSource1\"}";
     String expectedJsonConfig = 
"{\"bucket\":\"bucket1\",\"baseKey\":\"dataSource1\","
                                 + 
"\"disableAcl\":false,\"maxListingLength\":1024,\"useS3aSchema\":false}";
-
     S3DataSegmentPusherConfig config = JSON_MAPPER.readValue(jsonConfig, 
S3DataSegmentPusherConfig.class);
-    Assert.assertEquals(expectedJsonConfig, 
JSON_MAPPER.writeValueAsString(config));
+    Map<String, String> expected = JSON_MAPPER.readValue(expectedJsonConfig, 
Map.class);
+    Map<String, String> actual = 
JSON_MAPPER.readValue(JSON_MAPPER.writeValueAsString(config), Map.class);
+    Assert.assertEquals(expected, actual);
   }
 
   @Test


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

Reply via email to