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

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


The following commit(s) were added to refs/heads/master by this push:
     new d2bb30f  AWS: Fix TestGlueCatalogTable failure (#4355)
d2bb30f is described below

commit d2bb30f863d7276097c39754b9501ba27403b73c
Author: Xiaoxuan <[email protected]>
AuthorDate: Thu Mar 17 16:59:59 2022 -0700

    AWS: Fix TestGlueCatalogTable failure (#4355)
---
 .../java/org/apache/iceberg/aws/glue/TestGlueCatalogTable.java      | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/aws/src/integration/java/org/apache/iceberg/aws/glue/TestGlueCatalogTable.java
 
b/aws/src/integration/java/org/apache/iceberg/aws/glue/TestGlueCatalogTable.java
index 0cdeab6..8429e0f 100644
--- 
a/aws/src/integration/java/org/apache/iceberg/aws/glue/TestGlueCatalogTable.java
+++ 
b/aws/src/integration/java/org/apache/iceberg/aws/glue/TestGlueCatalogTable.java
@@ -22,6 +22,7 @@ package org.apache.iceberg.aws.glue;
 import java.util.List;
 import java.util.Locale;
 import java.util.Optional;
+import java.util.stream.Collectors;
 import org.apache.iceberg.AssertHelpers;
 import org.apache.iceberg.BaseMetastoreTableOperations;
 import org.apache.iceberg.DataFile;
@@ -74,8 +75,9 @@ public class TestGlueCatalogTable extends GlueTestBase {
     
Assert.assertTrue(response.table().parameters().containsKey(BaseMetastoreTableOperations.METADATA_LOCATION_PROP));
     Assert.assertEquals(schema.columns().size(), 
response.table().storageDescriptor().columns().size());
     Assert.assertEquals(partitionSpec.fields().size(), 
response.table().partitionKeys().size());
-    Assert.assertEquals("additionalLocations should match", 
tableLocationProperties.values(),
-        response.table().storageDescriptor().additionalLocations());
+    Assert.assertEquals("additionalLocations should match",
+        
tableLocationProperties.values().stream().sorted().collect(Collectors.toList()),
+        
response.table().storageDescriptor().additionalLocations().stream().sorted().collect(Collectors.toList()));
     // verify metadata file exists in S3
     String metaLocation = 
response.table().parameters().get(BaseMetastoreTableOperations.METADATA_LOCATION_PROP);
     String key = metaLocation.split(testBucketName, -1)[1].substring(1);

Reply via email to