From 16395489ccc6773497b1e3d00d9dcae64bcfab7c Mon Sep 17 00:00:00 2001 From: Piotr Findeisen Date: Tue, 11 Apr 2023 21:20:58 +0200 Subject: [PATCH] Fix flaky testAddColumnConcurrently Ignite test Hopefully fix `TestIgniteConnectorTest.testAddColumnConcurrently`. The fix hasn't been verified, as the test cannot be run on Apple M1. --- .../io/trino/plugin/ignite/TestIgniteConnectorTest.java | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/plugin/trino-ignite/src/test/java/io/trino/plugin/ignite/TestIgniteConnectorTest.java b/plugin/trino-ignite/src/test/java/io/trino/plugin/ignite/TestIgniteConnectorTest.java index 5ad31a061962..3097a43e2d28 100644 --- a/plugin/trino-ignite/src/test/java/io/trino/plugin/ignite/TestIgniteConnectorTest.java +++ b/plugin/trino-ignite/src/test/java/io/trino/plugin/ignite/TestIgniteConnectorTest.java @@ -307,6 +307,12 @@ protected boolean isColumnNameRejected(Exception exception, String columnName, b return errorMessage.contains("Failed to complete exchange process"); } + @Override + protected void verifyConcurrentAddColumnFailurePermissible(Exception e) + { + assertThat(e).hasMessage("Schema change operation failed: Thread got interrupted while trying to acquire table lock."); + } + @Override public void testAddNotNullColumnToNonEmptyTable() {