[FLINK-19435][connectors/jdbc] Add hang test case to reveal deadlock …

…when loading different sql driver classes concurrently using Class.forName This closes #14361
apache · Dec 21, 2020 · 7ca2dd3 · 7ca2dd3
1 parent 94e4d2a
commit 7ca2dd3
Show file tree

Hide file tree

Showing 9 changed files with 707 additions and 0 deletions.
diff --git a/flink-connectors/flink-connector-jdbc/pom.xml b/flink-connectors/flink-connector-jdbc/pom.xml
@@ -150,4 +150,17 @@ under the License.
 		</dependency>
 
 	</dependencies>
+
+	<build>
+		<plugins>
+			<plugin>
+				<groupId>org.apache.maven.plugins</groupId>
+				<artifactId>maven-surefire-plugin</artifactId>
+				<configuration>
+					<!-- Disable jvm process reuse to test driver class loading issues -->
+					<reuseForks>false</reuseForks>
+				</configuration>
+			</plugin>
+		</plugins>
+	</build>
 </project>
diff --git a/...link-connector-jdbc/src/test/java/org/apache/flink/connector/jdbc/fakedb/FakeDBUtils.java b/...link-connector-jdbc/src/test/java/org/apache/flink/connector/jdbc/fakedb/FakeDBUtils.java
@@ -0,0 +1,39 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.flink.connector.jdbc.fakedb;
+
+/**
+ * Utilities and constants for FakeDB.
+ */
+public class FakeDBUtils {
+	public static final String URL_PREFIX = "jdbc:fake:";
+
+	public static final String TEST_DB_URL = composeDBUrl("test");
+
+	public static final String DRIVER1_CLASS_NAME = "org.apache.flink.connector.jdbc.fakedb.driver.FakeDriver1";
+	public static final String DRIVER2_CLASS_NAME = "org.apache.flink.connector.jdbc.fakedb.driver.FakeDriver2";
+
+	public static String composeDBUrl(String db) {
+		return URL_PREFIX + db;
+	}
+
+	public static boolean acceptsUrl(String url) {
+		return url.startsWith(URL_PREFIX);
+	}
+}
diff --git a/...ctor-jdbc/src/test/java/org/apache/flink/connector/jdbc/fakedb/driver/FakeConnection.java b/...ctor-jdbc/src/test/java/org/apache/flink/connector/jdbc/fakedb/driver/FakeConnection.java
@@ -0,0 +1,318 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.flink.connector.jdbc.fakedb.driver;
+
+import java.sql.Array;
+import java.sql.Blob;
+import java.sql.CallableStatement;
+import java.sql.Clob;
+import java.sql.Connection;
+import java.sql.DatabaseMetaData;
+import java.sql.NClob;
+import java.sql.PreparedStatement;
+import java.sql.SQLClientInfoException;
+import java.sql.SQLException;
+import java.sql.SQLWarning;
+import java.sql.SQLXML;
+import java.sql.Savepoint;
+import java.sql.Statement;
+import java.sql.Struct;
+import java.util.Map;
+import java.util.Properties;
+import java.util.concurrent.Executor;
+
+/**
+ * A fake sql connection implementation which throws {@link SQLException} in most of its methods.
+ */
+public abstract class FakeConnection implements Connection {
+	private boolean closed = false;
+
+	@Override
+	public Statement createStatement() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public PreparedStatement prepareStatement(String sql) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public CallableStatement prepareCall(String sql) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public String nativeSQL(String sql) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void setAutoCommit(boolean autoCommit) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public boolean getAutoCommit() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void commit() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void rollback() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void close() throws SQLException {
+		closed = true;
+	}
+
+	@Override
+	public boolean isClosed() throws SQLException {
+		return closed;
+	}
+
+	@Override
+	public DatabaseMetaData getMetaData() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void setReadOnly(boolean readOnly) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public boolean isReadOnly() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void setCatalog(String catalog) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public String getCatalog() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void setTransactionIsolation(int level) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public int getTransactionIsolation() throws SQLException {
+		return TRANSACTION_NONE;
+	}
+
+	@Override
+	public SQLWarning getWarnings() throws SQLException {
+		return null;
+	}
+
+	@Override
+	public void clearWarnings() throws SQLException {
+	}
+
+	@Override
+	public Statement createStatement(int resultSetType, int resultSetConcurrency) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public PreparedStatement prepareStatement(String sql, int resultSetType, int resultSetConcurrency) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public CallableStatement prepareCall(String sql, int resultSetType, int resultSetConcurrency) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public Map<String, Class<?>> getTypeMap() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void setTypeMap(Map<String, Class<?>> map) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void setHoldability(int holdability) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public int getHoldability() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public Savepoint setSavepoint() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public Savepoint setSavepoint(String name) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void rollback(Savepoint savepoint) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void releaseSavepoint(Savepoint savepoint) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public Statement createStatement(int resultSetType, int resultSetConcurrency, int resultSetHoldability) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public PreparedStatement prepareStatement(String sql, int resultSetType, int resultSetConcurrency, int resultSetHoldability) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public CallableStatement prepareCall(String sql, int resultSetType, int resultSetConcurrency, int resultSetHoldability) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public PreparedStatement prepareStatement(String sql, int autoGeneratedKeys) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public PreparedStatement prepareStatement(String sql, int[] columnIndexes) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public PreparedStatement prepareStatement(String sql, String[] columnNames) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public Clob createClob() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public Blob createBlob() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public NClob createNClob() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public SQLXML createSQLXML() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public boolean isValid(int timeout) throws SQLException {
+		return !isClosed();
+	}
+
+	@Override
+	public void setClientInfo(String name, String value) throws SQLClientInfoException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void setClientInfo(Properties properties) throws SQLClientInfoException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public String getClientInfo(String name) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public Properties getClientInfo() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public Array createArrayOf(String typeName, Object[] elements) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public Struct createStruct(String typeName, Object[] attributes) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void setSchema(String schema) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public String getSchema() throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void abort(Executor executor) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public void setNetworkTimeout(Executor executor, int milliseconds) throws SQLException {
+		throw new UnsupportedOperationException();
+	}
+
+	@Override
+	public int getNetworkTimeout() throws SQLException {
+		return 0;
+	}
+
+	@Override
+	@SuppressWarnings("unchecked")
+	public <T> T unwrap(Class<T> iface) throws SQLException {
+		if (iface.isInstance(this)) {
+			return (T) this;
+		}
+		throw new SQLException(getClass() + " does not implement " + iface);
+	}
+
+	@Override
+	public boolean isWrapperFor(Class<?> iface) throws SQLException {
+		return iface.isInstance(this);
+	}
+}