Remove parameters from Glue partition columns (#697)

* Remove parameters from partition columns in Glue

* fix tests

Author: kroushan-nit

xtable-api/src/main/java/org/apache/xtable/spi/sync/CatalogSync.java

@@ -69,7 +69,8 @@ public SyncResult syncTable(
                 tableIdentifier.getId(),
                 table.getBasePath(),
                 table.getTableFormat(),
-                catalogSyncClient.getClass().getName());
+                catalogSyncClient.getClass().getName(),
+                e);
             results.add(
                 getCatalogSyncFailureStatus(
                     catalogSyncClient.getCatalogId(), catalogSyncClient.getClass().getName(), e));

xtable-aws/src/main/java/org/apache/xtable/glue/GlueSchemaExtractor.java

@@ -250,8 +250,13 @@ public List<Column> getPartitionColumns(InternalTable table, Map<String, Column>
      */
     return getPartitionKeys(table).stream()
         .map(
-            pKey ->
-                columnsMap.getOrDefault(pKey, Column.builder().name(pKey).type("string").build()))
+            pKey -> {
+              Column column = columnsMap.get(pKey);
+              return Column.builder()
+                  .name(pKey)
+                  .type(column != null ? column.type() : "string")
+                  .build();
+            })
         .collect(Collectors.toList());
   }
 

xtable-aws/src/test/java/org/apache/xtable/glue/GlueCatalogSyncTestBase.java

@@ -96,7 +96,7 @@ public class GlueCatalogSyncTestBase {
                   getInternalField("booleanField", "boolean", InternalType.BOOLEAN)))
           .build();
   protected static final List<Column> PARTITION_KEYS =
-      Collections.singletonList(getColumn(TableFormat.DELTA, "partitionField", "string"));
+      Collections.singletonList(getColumn("partitionField", "string"));
   protected static final List<Column> DELTA_GLUE_SCHEMA =
       Arrays.asList(
           getColumn(TableFormat.DELTA, "intField", "int"),

xtable-aws/src/test/java/org/apache/xtable/glue/TestGlueSchemaExtractor.java

@@ -92,7 +92,7 @@ public class TestGlueSchemaExtractor extends TestSchemaExtractorBase {
           .partitionFieldNames(Collections.singletonList("dateOfBirth"))
           .build();
 
-  private static Column getColumn(String name, String type) {
+  public static Column getColumn(String name, String type) {
     return Column.builder().name(name).type(type).build();
   }