Tinaj1234 has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/234281

Change subject: Add secondary indexes and some minor issues with database
......................................................................

Add secondary indexes and some minor issues with database

Change-Id: I8c58d99ed1ec1f8be47b6d0acf22562652181e5f
---
M sql/nl_newsletters.sql
M sql/nl_publishers.sql
2 files changed, 8 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Newsletter 
refs/changes/81/234281/1

diff --git a/sql/nl_newsletters.sql b/sql/nl_newsletters.sql
index 257b0c4..8441921 100644
--- a/sql/nl_newsletters.sql
+++ b/sql/nl_newsletters.sql
@@ -3,9 +3,12 @@
 CREATE TABLE /*_*/nl_newsletters(
        --Primary key
        nl_id int unsigned NOT NULL PRIMARY KEY AUTO_INCREMENT,
-       nl_name VARCHAR (50) NOT NULL UNIQUE,
-       nl_desc VARCHAR (256),
+       nl_name varchar(50) NOT NULL UNIQUE,
+       nl_desc varbinary(767),
        nl_main_page_id int NOT NULL,
-       nl_frequency VARCHAR (50) NOT NULL,
+       nl_frequency varchar(50) NOT NULL,
        nl_owner_id int NOT NULL
-)/*$wgDBTableOptions*/;
\ No newline at end of file
+)/*$wgDBTableOptions*/;
+
+CREATE INDEX /*i*/nl_id ON /*_*/nl_newsletters(nl_name);
+CREATE INDEX /*i*/nl_owner_id ON /*_*/nl_newsletters(nl_owner_id);
\ No newline at end of file
diff --git a/sql/nl_publishers.sql b/sql/nl_publishers.sql
index 2c5cef0..d6cef4b 100644
--- a/sql/nl_publishers.sql
+++ b/sql/nl_publishers.sql
@@ -4,5 +4,5 @@
        --Primary key
        newsletter_id int REFERENCES nl_newsletter(nl_id),
        publisher_id int NOT NULL,
-       PRIMARY KEY (newsletter_id, publisher_id)
+       PRIMARY KEY (publisher_id, newsletter_id)
 )/*$wgDBTableOptions*/;

-- 
To view, visit https://gerrit.wikimedia.org/r/234281
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8c58d99ed1ec1f8be47b6d0acf22562652181e5f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Newsletter
Gerrit-Branch: master
Gerrit-Owner: Tinaj1234 <tinajohnson.1...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to