Skip to content

Commit ee2c964

Browse files
ritiktrianzAbdulR3hman
authored andcommitted
The SQLServer Glue Get Table API was not retrieving data as expected due to a case-sensitivity issue with the PARTITION_NUMBER. By converting the PARTITION_NUMBER column to lowercase, we achieved successful data retrieval through the Glue API.
1 parent 5546ae3 commit ee2c964

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

athena-sqlserver/src/test/java/com/amazonaws/athena/connectors/sqlserver/SqlServerMetadataHandlerTest.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ public void doGetTableLayout()
151151
for (int i = 0; i < getTableLayoutResponse.getPartitions().getRowCount(); i++) {
152152
actualValues.add(BlockUtils.rowToString(getTableLayoutResponse.getPartitions(), i));
153153
}
154-
Assert.assertEquals(Arrays.asList("[PARTITION_NUMBER : 1:::pf:::pc]","[PARTITION_NUMBER : 2:::pf:::pc]","[PARTITION_NUMBER : 3:::pf:::pc]"), actualValues);
154+
Assert.assertEquals(Arrays.asList("[partition_number : 1:::pf:::pc]","[partition_number : 2:::pf:::pc]","[partition_number : 3:::pf:::pc]"), actualValues);
155155

156156
SchemaBuilder expectedSchemaBuilder = SchemaBuilder.newBuilder();
157157
expectedSchemaBuilder.addField(FieldBuilder.newBuilder(PARTITION_NUMBER, org.apache.arrow.vector.types.Types.MinorType.VARCHAR.getType()).build());
@@ -197,7 +197,7 @@ public void doGetTableLayoutWithNoPartitions()
197197
for (int i = 0; i < getTableLayoutResponse.getPartitions().getRowCount(); i++) {
198198
actualValues.add(BlockUtils.rowToString(getTableLayoutResponse.getPartitions(), i));
199199
}
200-
Assert.assertEquals(Arrays.asList("[PARTITION_NUMBER : 0]"), actualValues);
200+
Assert.assertEquals(Arrays.asList("[partition_number : 0]"), actualValues);
201201

202202
SchemaBuilder expectedSchemaBuilder = SchemaBuilder.newBuilder();
203203
expectedSchemaBuilder.addField(FieldBuilder.newBuilder(PARTITION_NUMBER, org.apache.arrow.vector.types.Types.MinorType.VARCHAR.getType()).build());

0 commit comments

Comments
 (0)