Bug 7509

Summary: Create index for items.itemcallnumber
Product: Koha Reporter: Srdjan Jankovic <srdjan>
Component: CatalogingAssignee: Srdjan Jankovic <srdjan>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: enhancement    
Priority: P5 - low CC: jcamins, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: patch
bug_7509: Added index on items itemcallnumber
bug_7509: Added index on items itemcallnumber
Bug 7509: Added index on items itemcallnumber
Bug 7509: Added index on items itemcallnumber

Description Srdjan Jankovic 2012-02-07 23:57:02 UTC
Create index on items itemcallnumber. It will facilitate efficient search.
Comment 1 Srdjan Jankovic 2012-02-08 02:25:26 UTC Comment hidden (obsolete)
Comment 2 Liz Rea 2012-02-08 17:39:07 UTC Comment hidden (obsolete)
Comment 3 Liz Rea 2012-02-08 17:48:09 UTC Comment hidden (obsolete)
Comment 4 Jared Camins-Esakov 2012-02-13 01:46:16 UTC
I don't see any reason for using atomicupdate with something this simple.
Comment 5 Jared Camins-Esakov 2012-02-13 01:52:01 UTC Comment hidden (obsolete)
Comment 6 Ian Walls 2012-02-16 16:33:17 UTC
Created attachment 7695 [details] [review]
Bug 7509: Added index on items itemcallnumber

Comment from Liz:
I couldn't get this update to fire - I found a small syntax error in the
updatedatabase.pl for this, which I corrected with this patch.

Functionally, all of this works as tested (once the typos were fixed), but
this probably requires another sign-off.

QA notes: rebased on latest master.  Also added index name `itemcallnumber`
to maintain consistency with all other indexes on items table.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Ian Walls <koha.sekjal@gmail.com>
Comment 7 Ian Walls 2012-02-16 16:34:33 UTC
Simple change to add a new index.  Rebased to latest master and modified some syntax to be consistent with other indexes on the same table.  Marking as Passed QA.
Comment 8 Paul Poulain 2012-02-17 10:33:56 UTC
QA comment:
the ` is a mysqlism, that is not needed.
During Hackfest in Marseille, next month, we will get rid of it, so this patch would have been rejected in April.
Accepted it as it's consistent with existing code, but i also had accepted the "no `" version.
Comment 9 Jared Camins-Esakov 2012-12-31 00:38:24 UTC
There have been no further reports of problems so I am marking this bug resolved.