View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0020443 | mantisbt | installation | public | 2015-12-29 06:04 | 2016-06-12 00:43 |
Reporter | dregad | Assigned To | dregad | ||
Priority | normal | Severity | major | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Product Version | 1.3.0-rc.1 | ||||
Target Version | 1.3.0-rc.2 | Fixed in Version | 1.3.0-rc.2 | ||
Summary | 0020443: New 'api_token' table does not have standard options | ||||
Description | The schema change to implement 0017766 did not assign the standard table options to the CreateTableSQL statement
This will most likely not be a problem for MySQL since the plan is to move away from the MyISAM engine, and the charset will be inherited from global database settings. We need to fix this for PostgreSQL however. Unfortunately, AFAIK the ADOdb data dictionary does not let us alter table options, so it's not possible to do this cleanly without dropping the table and recreating it. That implies
Alternatively, as the presence of an OID field is unlikely to cause any significant issues in Mantis, we could go with a more simple approach of updating upgrade step 200 as appropriate. Considering the market share of postgresql, I would be tempted to go with the 2nd approach. | ||||
Additional Information | Severity set to major because we should not release 1.3.0 with an inconsistent schema. | ||||
Tags | No tags attached. | ||||
Reminder sent to: vboctor Your feedback on this would be appreciated. |
|
Let's update the existing step since Mantis won't care either way whether OIDS exist or not. Do we have a wiki page for what we want schema changes to look like and why? If not, it would be useful to add it. We should also use it as a checklist for pull requests related to the db schema. |
|
Pull request https://github.com/mantisbt/mantisbt/pull/699 |
|
In the end I merged this fix independently from PR 699, as its primary purpose is to support utf8mb4, which is not blocking for 1.3.0 release. |
|