ashulin commented on code in PR #4540:
URL:
https://github.com/apache/incubator-seatunnel/pull/4540#discussion_r1163518218
##########
seatunnel-connectors-v2/connector-jdbc/src/main/java/org/apache/seatunnel/connectors/seatunnel/jdbc/internal/dialect/JdbcDialect.java:
##########
@@ -193,4 +195,40 @@ default ResultSetMetaData getResultSetMetaData(
PreparedStatement ps =
conn.prepareStatement(jdbcSourceConfig.getQuery());
return ps.getMetaData();
}
+
+ default String listDatabases() {
+ return "SHOW DATABASES;";
+ }
+
+ default String getUrlFromDatabaseName(String baseUrl, String databaseName,
String suffix) {
+ return baseUrl + databaseName + suffix;
+ }
+
+ default String createDatabaseSql(String databaseName) {
+ return String.format("CREATE DATABASE IF NOT EXISTS %s;",
quoteIdentifier(databaseName));
+ }
+
+ default String dropDatabaseSql(String databaseName) {
+ return String.format("DROP DATABASE IF EXISTS %s;",
quoteIdentifier(databaseName));
+ }
+
+ default String getTableName(ResultSet rs) throws SQLException {
+ return rs.getString(1);
Review Comment:
because Missing context, `1` is a Magic number
--
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]