Skip to content

Commit

Permalink
chore: Deprecate functions that should be replaced by migration funct…
Browse files Browse the repository at this point in the history
…ions (#2333)
  • Loading branch information
joc-a authored Jan 7, 2025
1 parent ffa3eef commit d763e8e
Showing 1 changed file with 11 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -594,6 +594,12 @@ object SchemaUtils {
* to use a lock based on synchronization with a dummy table.
* @see SchemaUtils.withDataBaseLock
*/
@Deprecated(
"Execution of this function might lead to unpredictable state in the database if a failure occurs at any point. " +
"To prevent this, please use `MigrationUtils.statementsRequiredForDatabaseMigration` with a third-party migration tool (e.g., Flyway).",
ReplaceWith("MigrationUtils.statementsRequiredForDatabaseMigration"),
DeprecationLevel.WARNING
)
fun createMissingTablesAndColumns(vararg tables: Table, inBatch: Boolean = false, withLogs: Boolean = true) {
with(TransactionManager.current()) {
db.dialect.resetCaches()
Expand Down Expand Up @@ -637,6 +643,11 @@ object SchemaUtils {
* By default, a description for each intermediate step, as well as its execution time, is logged at the INFO level.
* This can be disabled by setting [withLogs] to `false`.
*/
@Deprecated(
"This function will be removed in future releases.",
ReplaceWith("MigrationUtils.statementsRequiredForDatabaseMigration"),
DeprecationLevel.WARNING
)
fun statementsRequiredToActualizeScheme(vararg tables: Table, withLogs: Boolean = true): List<String> {
val (tablesToCreate, tablesToAlter) = tables.partition { !it.exists() }
val createStatements = logTimeSpent("Preparing create tables statements", withLogs) {
Expand Down

0 comments on commit d763e8e

Please sign in to comment.