Skip to content

Commit 3d8c174

Browse files
authored
Fix MySQL JDBC query properties extension when SSL is required on server (#36581)
1 parent 5291aab commit 3d8c174

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

kernel/data-pipeline/dialect/mysql/src/main/java/org/apache/shardingsphere/data/pipeline/mysql/datasource/MySQLJdbcQueryPropertiesExtension.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ public final class MySQLJdbcQueryPropertiesExtension implements JdbcQueryPropert
3636
private final Properties completeIfMissedQueryProps = new Properties();
3737

3838
public MySQLJdbcQueryPropertiesExtension() {
39-
toBeOverrideQueryProps.setProperty("useSSL", Boolean.FALSE.toString());
4039
toBeOverrideQueryProps.setProperty("useServerPrepStmts", Boolean.FALSE.toString());
4140
toBeOverrideQueryProps.setProperty("rewriteBatchedStatements", Boolean.TRUE.toString());
4241
toBeOverrideQueryProps.setProperty("yearIsDateType", Boolean.FALSE.toString());

kernel/data-pipeline/dialect/mysql/src/test/java/org/apache/shardingsphere/data/pipeline/mysql/datasource/MySQLJdbcQueryPropertiesExtensionTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,8 +53,7 @@ private void assertExtension(final JdbcQueryPropertiesExtension actual) {
5353
}
5454

5555
private void assertQueryProperties(final Properties actual, final String expectedNetTimeoutForStreamingResults) {
56-
assertThat(actual.size(), is(8));
57-
assertThat(actual.getProperty("useSSL"), is(Boolean.FALSE.toString()));
56+
assertThat(actual.size(), is(7));
5857
assertThat(actual.getProperty("useServerPrepStmts"), is(Boolean.FALSE.toString()));
5958
assertThat(actual.getProperty("rewriteBatchedStatements"), is(Boolean.TRUE.toString()));
6059
assertThat(actual.getProperty("yearIsDateType"), is(Boolean.FALSE.toString()));

0 commit comments

Comments
 (0)