After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 783213 - redesign comma separated photo_id_list
redesign comma separated photo_id_list
Status: RESOLVED OBSOLETE
Product: shotwell
Classification: Other
Component: tags
0.26.x
Other Linux
: Normal enhancement
: 0.30
Assigned To: Shotwell Maintainers
Shotwell Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-05-29 19:26 UTC by pusterhofer
Modified: 2021-05-19 15:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description pusterhofer 2017-05-29 19:26:18 UTC 
I think the TagTable should be redesigned as queries are more complex and the performance may also increase when avoiding comma separated lists of ids.
Instead use a second table which links the tags with the photo_id. 

TagTable
id|name|time_created

TagPhotoRelationTable
tag_id|photo_id

Here are some reasons why lists should be avoided:
http://www.schemamania.org/sql/#lists
Comment 1 Jens Georg 2017-05-30 05:38:52 UTC 
Yes. The database schema is less then optimal, I know. Feel free to provide a patch (Good luck).
Comment 2 GNOME Infrastructure Team 2021-05-19 15:05:11 UTC 
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/shotwell/-/issues/4843.