capistrant commented on code in PR #12404:
URL: https://github.com/apache/druid/pull/12404#discussion_r887019746
##########
server/src/main/java/org/apache/druid/metadata/SQLMetadataConnector.java:
##########
@@ -327,6 +331,29 @@ tableName, getPayloadType()
)
);
}
+
+ public boolean tableContainsColumn(Handle handle, String table, String
column)
+ {
+ try {
+ DatabaseMetaData databaseMetaData = handle.getConnection().getMetaData();
+ ResultSet columns = databaseMetaData.getColumns(
+ null,
+ null,
+ table,
+ column
+ );
+ return columns.next();
+ }
+ catch (SQLException e) {
+ return false;
+ }
+ }
+
+ public void prepareTaskEntryTable(final String tableName)
+ {
+ createEntryTable(tableName);
Review Comment:
why not update createEntryTable to create the table who we want it going
forward so no alert needs to be run every time a new cluster is coming online?
##########
server/src/main/java/org/apache/druid/metadata/SQLMetadataConnector.java:
##########
@@ -327,6 +331,29 @@ tableName, getPayloadType()
)
);
}
+
+ public boolean tableContainsColumn(Handle handle, String table, String
column)
+ {
+ try {
+ DatabaseMetaData databaseMetaData = handle.getConnection().getMetaData();
+ ResultSet columns = databaseMetaData.getColumns(
+ null,
+ null,
+ table,
+ column
+ );
+ return columns.next();
+ }
+ catch (SQLException e) {
+ return false;
+ }
+ }
+
+ public void prepareTaskEntryTable(final String tableName)
+ {
+ createEntryTable(tableName);
Review Comment:
why not update createEntryTable to create the table how we want it going
forward so no alert needs to be run every time a new cluster is coming online?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
