phorge-phorge/resources/sql/patches/060.phriction.sql



CREATE TABLE {$NAMESPACE}_phriction.phriction_document (
  id INT UNSIGNED NOT NULL,
  phid VARCHAR(64) BINARY NOT NULL,
  UNIQUE KEY (phid),
  slug VARCHAR(128) NOT NULL,
  UNIQUE KEY (slug),
  depth INT UNSIGNED NOT NULL,
  UNIQUE KEY (depth, slug),
  contentID INT UNSIGNED NOT NULL
) ENGINE=InnoDB;
Phriction basics Summary: Basically a copy/paste of parts of D636, but with two changes: - Fully separate the index table ("document") from the content table ("content"). I think this will be a cleaner solution in the long run. - Build slugs into the document structure. This doesn't do anything useful, it just normalizes slugs and lays some groundwork. Test Plan: - Visited various /w/ pages and saw them normalize correctly. - Verified the DAO works by inserting dummy rows. Reviewed By: codeblock Reviewers: hsb, codeblock, jungejason, aran, tuomaspelkonen CC: aran, codeblock, epriestley Differential Revision: 638 2011-07-11 17:54:22 +02:00
SQL Patch Management: SQL Changes Summary: Splits out the SQL changes. These are most of the changes, but primarily mechanical: - Moved "initialize.sql" to "0000.legacy.sql" and partially reverted to an older version, such that patches 0000 + 000 + 001 + ... + 137 put us in the right state when applied sequentially. - Removed "create database" commands from all SQL. These are handled by separate DB patches now, so we have the data to do operations like "storage databases" (list databases) and "storage destroy" (drop databases). - Removed "phabricator_" namespace from all SQL, and replaced with "{$NAMESPACE}_" token so we can namespace databases. - Shortened some column lengths so patches apply correctly if originally created as InnoDB; also a few similar tweaks elsewhere. Test Plan: See D2323 for discussion and test plan. Reviewers: edward, vrana, btrahan, jungejason Reviewed By: btrahan CC: aran Maniphest Tasks: T140, T345 Differential Revision: https://secure.phabricator.com/D2329 2012-04-30 16:53:53 +02:00
			`CREATE TABLE {$NAMESPACE}_phriction.phriction_document (`
Phriction basics Summary: Basically a copy/paste of parts of D636, but with two changes: - Fully separate the index table ("document") from the content table ("content"). I think this will be a cleaner solution in the long run. - Build slugs into the document structure. This doesn't do anything useful, it just normalizes slugs and lays some groundwork. Test Plan: - Visited various /w/ pages and saw them normalize correctly. - Verified the DAO works by inserting dummy rows. Reviewed By: codeblock Reviewers: hsb, codeblock, jungejason, aran, tuomaspelkonen CC: aran, codeblock, epriestley Differential Revision: 638 2011-07-11 17:54:22 +02:00			`id INT UNSIGNED NOT NULL,`
			`phid VARCHAR(64) BINARY NOT NULL,`
			`UNIQUE KEY (phid),`
Minor, reduce slug column size to 128, see D1391. 2012-01-14 02:16:37 +01:00			`slug VARCHAR(128) NOT NULL,`
Phriction basics Summary: Basically a copy/paste of parts of D636, but with two changes: - Fully separate the index table ("document") from the content table ("content"). I think this will be a cleaner solution in the long run. - Build slugs into the document structure. This doesn't do anything useful, it just normalizes slugs and lays some groundwork. Test Plan: - Visited various /w/ pages and saw them normalize correctly. - Verified the DAO works by inserting dummy rows. Reviewed By: codeblock Reviewers: hsb, codeblock, jungejason, aran, tuomaspelkonen CC: aran, codeblock, epriestley Differential Revision: 638 2011-07-11 17:54:22 +02:00			`UNIQUE KEY (slug),`
			`depth INT UNSIGNED NOT NULL,`
			`UNIQUE KEY (depth, slug),`
			`contentID INT UNSIGNED NOT NULL`
Allow databases to be created outside upgrade_schema scrpt. Summary: Current scripts make it hard to administer Phabricator instance while not having direct (priviledged) access to the database. This change allows scenario where DB administrator creates the databases for you before you run update_schema script. Test Plan: Create the databases before running the update_schema script - it shouldn't complain that the databases already exist. Reviewers: aran, epriestley Reviewed By: epriestley CC: aran, epriestley Differential Revision: 1014 2011-10-17 22:28:23 +02:00			`) ENGINE=InnoDB;`