Remove challenges table (letsencrypt#5285)

aarongable · web-flow · commit b4db45035783 · 2021-02-22T11:25:19.000-08:00
This table is not used by any Boulder logic nor by any
offline stats/metrics gathering.
diff --git a/sa/_db-next/migrations/20210211174900_DropTableChallenges.sql b/sa/_db-next/migrations/20210211174900_DropTableChallenges.sql
@@ -0,0 +1,23 @@
+
+-- +goose Up
+-- SQL in section 'Up' is executed when this migration is applied
+
+DROP TABLE `challenges`;
+
+-- +goose Down
+-- SQL section 'Down' is executed when this migration is rolled back
+
+CREATE TABLE `challenges` (
+  `id` bigint(20) NOT NULL AUTO_INCREMENT,
+  `authorizationID` varchar(255) NOT NULL,
+  `LockCol` bigint(20) DEFAULT NULL,
+  `type` varchar(255) NOT NULL,
+  `status` varchar(255) NOT NULL,
+  `error` mediumblob DEFAULT NULL,
+  `validated` datetime DEFAULT NULL,
+  `token` varchar(255) NOT NULL,
+  `validationRecord` mediumblob DEFAULT NULL,
+  `keyAuthorization` varchar(255) DEFAULT NULL,
+  PRIMARY KEY (`id`),
+  KEY `authorizationID_challenges_idx` (`authorizationID`)
+) ENGINE=InnoDB DEFAULT CHARSET=utf8;
diff --git a/sa/model.go b/sa/model.go
@@ -182,8 +182,6 @@ type challModel struct {
 
 	// TODO(#1818): Remove, this field is unused, but is kept temporarily to avoid a database migration.
 	Validated bool `db:"validated"`
-
-	LockCol int64
 }
 
 // newReg creates a reg model object from a core.Registration

Original file line number	Diff line number	Diff line change
`@@ -182,8 +182,6 @@ type challModel struct {`
`182`	`182`
`183`	`183`	`// TODO(#1818): Remove, this field is unused, but is kept temporarily to avoid a database migration.`
`184`	`184`	Validated bool `db:"validated"`
`185`		`-`
`186`		`- LockCol int64`
`187`	`185`	`}`
`188`	`186`
`189`	`187`	`// newReg creates a reg model object from a core.Registration`