xr-chen commented on code in PR #766:
URL: https://github.com/apache/incubator-xtable/pull/766#discussion_r2621953985
##########
xtable-core/src/test/java/org/apache/xtable/ITConversionController.java:
##########
@@ -746,6 +746,29 @@ public void testIcebergCorruptedSnapshotRecovery() throws
Exception {
}
}
+ @Test
+ public void testColumnMappingEnabledDeltaToIceberg() {
Review Comment:
I added delta's physical column names into the Icerberg's
`schema.name-mapping.default`, and the converted table could read data from the
correct place now and could return the same content as the original delta
table. But I got a weird issue during testing,
- All tests could pass with this new test disabled when running `mvn verify`
- This test could pass when running it independently
- When running this test with all other tests together,
`ITConversionController.testVariousOperations` will fail
@the-other-tim-brown, is there anything shared among the test cases?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]