PR comments

interuss · Feb 4, 2025 · ddd69f5 · ddd69f5
1 parent ed79acc
commit ddd69f5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/pkg/scd/store/cockroach/constraints.go b/pkg/scd/store/cockroach/constraints.go
@@ -142,7 +142,6 @@ func (c *repo) UpsertConstraint(ctx context.Context, s *scdmodels.Constraint) (*
 			%s = $8,
 			%s = $9,
 			%s = transaction_timestamp()
-		WHERE scd_constraints.%s = $1
 		RETURNING
 			%s`,
 			constraintFieldsWithoutPrefix,
@@ -156,7 +155,6 @@ func (c *repo) UpsertConstraint(ctx context.Context, s *scdmodels.Constraint) (*
 			constraintFieldsWithIndices[7],
 			constraintFieldsWithIndices[8],
 			constraintFieldsWithIndices[9],
-			constraintFieldsWithIndices[0],
 			constraintFieldsWithPrefix,
 		)
 	)
@@ -228,7 +226,7 @@ func (c *repo) SearchConstraints(ctx context.Context, v4d *dssmodels.Volume4D) (
 				COALESCE(starts_at <= $3, true)
 			AND
 				COALESCE(ends_at >= $2, true)
-			LIMIT $4;
+			LIMIT $4
 			`, constraintFieldsWithoutPrefix)
 	)
 

diff --git a/pkg/scd/store/cockroach/subscriptions.go b/pkg/scd/store/cockroach/subscriptions.go
@@ -194,12 +194,12 @@ func (c *repo) pushSubscription(ctx context.Context, q dsssql.Queryable, s *scdm
 			($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, transaction_timestamp())
 		ON CONFLICT (%s) DO UPDATE
 			SET %s = $2,
-				%s = $3::int,
+				%s = $3,
 				%s = $4,
-				%s = $5::int,
+				%s = $5,
 				%s = $6,
 				%s = $7,
-				%s = $8::bool,
+				%s = $8,
 				%s = $9,
 				%s = $10,
 				%s = $11,