← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] ~tushar5526/launchpad:use-escape-string-syntax into launchpad:master

 

Tushar Gupta has proposed merging ~tushar5526/launchpad:use-escape-string-syntax into launchpad:master.

Commit message:
At the time of this commit, `standard_conforming_strings` is set to `false` in the LP database. Starting with PostgreSQL 9, the default value for this setting was set to `true`.

To enable `standard_conforming_strings` in the future, all SQL queries that rely on escape sequences must be updated to use the `E'...'` (escape string) syntax. This ensures that the queries behave consistently regardless of whether the setting is `on` or `off`.


Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~tushar5526/launchpad/+git/launchpad/+merge/493109

At the time of this commit, `standard_conforming_strings` is set to `false` in the LP database. Starting with PostgreSQL 9, the default value for this setting was set to `true`.

To enable `standard_conforming_strings` in the future, all SQL queries that rely on escape sequences must be updated to use the `E'...'` (escape string) syntax. This ensures that the queries behave consistently regardless of whether the setting is `on` or `off`.

-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~tushar5526/launchpad:use-escape-string-syntax into launchpad:master.
diff --git a/database/schema/patch-2211-46-0.sql b/database/schema/patch-2211-46-0.sql
new file mode 100644
index 0000000..b7e32ea
--- /dev/null
+++ b/database/schema/patch-2211-46-0.sql
@@ -0,0 +1,15 @@
+-- Copyright 2025 Canonical Ltd.  This software is licensed under the
+-- GNU Affero General Public License version 3 (see the file LICENSE).
+
+SET client_min_messages=ERROR;
+
+-- StructuralSubscription
+COMMENT ON COLUMN StructuralSubscription.product IS E'The subscription\'s target, when it is a product.';
+COMMENT ON COLUMN StructuralSubscription.productseries IS E'The subscription\'s target, when it is a product series.';
+COMMENT ON COLUMN StructuralSubscription.project IS E'The subscription\'s target, when it is a project.';
+COMMENT ON COLUMN StructuralSubscription.milestone IS E'The subscription\'s target, when it is a milestone.';
+COMMENT ON COLUMN StructuralSubscription.distribution IS E'The subscription\'s target, when it is a distribution.';
+COMMENT ON COLUMN StructuralSubscription.distroseries IS E'The subscription\'s target, when it is a distribution series.';
+COMMENT ON COLUMN StructuralSubscription.sourcepackagename IS E'The subscription\'s target, when it is a source-package';
+
+INSERT INTO LaunchpadDatabaseRevision VALUES (2211, 46, 0);

Follow ups