Make file object key non-nullable

lberrymage · lberrymage · commit 90c07765c5cb · 2025-08-19T17:01:25.000Z
All of the file rows in our production database contain a non-null
object key and Parcelo never sets files' object keys to null, so this
migration is safe for us to perform.
diff --git a/console/src/main/kotlin/app/accrescent/parcelo/console/data/File.kt b/console/src/main/kotlin/app/accrescent/parcelo/console/data/File.kt
@@ -11,7 +11,7 @@ import org.jetbrains.exposed.dao.id.IntIdTable
 
 object Files : IntIdTable("files") {
     val deleted = bool("deleted").default(false)
-    val s3ObjectKey = text("s3_object_key").nullable()
+    val s3ObjectKey = text("s3_object_key")
 }
 
 class File(id: EntityID<Int>) : IntEntity(id) {
diff --git a/console/src/main/resources/db/migration/V4__Make_file_object_key_not_null.sql b/console/src/main/resources/db/migration/V4__Make_file_object_key_not_null.sql
@@ -0,0 +1 @@
+ALTER TABLE files ALTER COLUMN s3_object_key SET NOT NULL;
diff --git a/console/src/main/resources/db/migration/V4__Make_file_object_key_not_null.sql.license b/console/src/main/resources/db/migration/V4__Make_file_object_key_not_null.sql.license
@@ -0,0 +1,3 @@
+Copyright 2025 Logan Magee
+
+SPDX-License-Identifier: AGPL-3.0-only

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ import org.jetbrains.exposed.dao.id.IntIdTable`
`11`	`11`
`12`	`12`	`object Files : IntIdTable("files") {`
`13`	`13`	`val deleted = bool("deleted").default(false)`
`14`		`- val s3ObjectKey = text("s3_object_key").nullable()`
	`14`	`+ val s3ObjectKey = text("s3_object_key")`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`class File(id: EntityID<Int>) : IntEntity(id) {`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+ALTER TABLE files ALTER COLUMN s3_object_key SET NOT NULL;`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+Copyright 2025 Logan Magee`
	`2`	`+`
	`3`	`+SPDX-License-Identifier: AGPL-3.0-only`