Hi Sean, hi Mike, this is a very interesting, but also challenging project. Essentially this boils down to adding primary keys to tables that don't have them yet. There are a few obstacles though: - The database drivers of OTRS don't support adding of primary keys to existing tables (yet). - Once this is done for the framework, we will also need to enforce primary keys for all OTRS modules. So the steps to achieve this would be: 1. Add the possibility to add primary keys to all database drivers in OTRS (mysql, postgresql, mssql and oracle). Maybe Mike could help here with the non-mssql databases. This must be covered well with unit tests and should be merged as a separate pull request before the work continues. 2. Add primary keys to all tables in the framework that don't have any. Should be unit tested as well, perhaps the existing unit tests will cover parts of this already. 3. Extend OTRSCodePolicy to require primary keys for all tables in OTRS 3.4 and higher, including addon modules. This would be my part. 4. Add primary keys to all modules ported to OTRS 3.4 in future. I guess this would also be on the OTRS side. To get this into OTRS 3.4, 1) and 2) would need to be successfully QA'ed by July, otherwise it would have to be postponed to a later release. What do you think? Regards, mg Am 29.03.14 02:50, schrieb Sean Killeen:
*Summary: *I'd like to alter some DB scripts to add clustered indexes so they work better for MSSQL (and work at all for SQL Azure). How should I go about this?
Hi all,
New to contributing to OTRS and to the dev mailing list so please forgive me if I stumble around a bit.
*Background / Issue*
I wanted to set up OTRS on Azure using a SQL Azure database back-end, so I thought I'd play around a bit.
SQL Azure apparently chokes when confronted with tables that don't have any clustered indexes. Womp womp.
As of v3.3.5, the setup script for OTRS apparently creates 33 tables without clustered indexes or primary keys (the list in full is at the end of this post). This leads to inoperability and a ton of errors in the log when attempting to use OTRS with SQL Azure.
*What I'd like to do about it * * * I'd like to help out with this and submit a pull request for the MSSQL setup to deal with this (as well as make things a little more snappy for MSSQL users). Was hoping to start this out as a Github issue, but I don't believe Issues are enabled for the otrs/otrs project.
However, I'm as of yet completely unfamiliar with the data structure, so I would be guessing at best here and I don't think that's wise. :)
Is there a solid reference point for the database architecture available that I could reference to deduce where primary keys or clustered indexes could be used in the affected tables? Or is there some other approach the community would prefer I take to addressing this?
OTRS is a great product, and I'm looking forward to being able to hopefully give something back.
Thanks, -- Sean
*Appendix: The list of tables without clustered indexes*
Found by running:
SELECT DISTINCT [TABLE] = OBJECT_NAME(OBJECT_ID) FROM SYS.INDEXES WHERE INDEX_ID = 0 AND OBJECTPROPERTY(OBJECT_ID,'IsUserTable') = 1 ORDER BY [TABLE]
acl_sync article_flag customer_preferences generic_agent_jobs gi_object_lock_state group_customer_user group_role group_user link_relation notification_event_item personal_queues pm_entity pm_entity_sync postmaster_filter process_id queue_preferences queue_standard_template role_user search_profile service_customer_user service_preferences service_sla sla_preferences support_bench_test ticket_flag ticket_index ticket_lock_index ticket_loop_protection ticket_watcher user_preferences virtual_fs_preferences web_upload_cache xml_storage
_______________________________________________ OTRS mailing list: dev - Webpage: http://otrs.org/ Archive: http://lists.otrs.org/pipermail/dev To unsubscribe: http://lists.otrs.org/cgi-bin/listinfo/dev
-- Martin Gruner Senior Developer R&D OTRS AG Bahnhofplatz 1a 94315 Straubing T: +49 (0)6172 681988 0 F: +49 (0)9421 56818 18 I: www.otrs.com/ Geschäftssitz: Bad Homburg, Amtsgericht: Bad Homburg, HRB 10751, USt-Nr.: DE256610065 Aufsichtsratsvorsitzender: Burchard Steinbild, Vorstand: André Mindermann (Vorsitzender), Christopher Kuhn, Sabine Riedel Einfache Planung, bessere Übersicht - Mit OTRS 3.3 einfach besseres Service Management - Jetzt downloaden und testen