Delete unused URI/actor/etc entries from DB when deleting an event

Bug #491646 reported by Siegfried Gevatter
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Zeitgeist Framework
Fix Released
Medium
Unassigned

Bug Description

<KaKaRoTo> RainCT, I used REFERENCES table(column) ON DELETE CASCADE everywhere

Tags: privacy
Changed in zeitgeist:
importance: Undecided → Medium
milestone: none → 0.3.1
Revision history for this message
Seif Lotfy (seif) wrote : Re: [Bug 491646] [NEW] Delete unused stuff from DB

What do u mean by delete unused stuff?

2009/12/3 Siegfried Gevatter <email address hidden>

> Public bug reported:
>
> <KaKaRoTo> RainCT, I used REFERENCES table(column) ON DELETE CASCADE
> everywhere
>
> ** Affects: zeitgeist
> Importance: Medium
> Status: New
>
> ** Changed in: zeitgeist
> Importance: Undecided => Medium
>
> ** Changed in: zeitgeist
> Milestone: None => 0.3.1
>
> --
> Delete unused stuff from DB
> https://bugs.launchpad.net/bugs/491646
> You received this bug notification because you are subscribed to The
> Zeitgeist Project.
>
> Status in Zeitgeist Framework: New
>
> Bug description:
> <KaKaRoTo> RainCT, I used REFERENCES table(column) ON DELETE CASCADE
> everywhere
>
>

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote : Re: Delete unused stuff from DB

Until we see evidence that our DB indeed clutters up in lots of unused URIs and whatnot (after some DeleteEvents), then I don't think the added complexity is worth it.

A more pressing issue is to delete unused URIs after a DeleteEvents because of privacy reasons - it's fine that we delete all events about http://example.com/explicit_content, but we still have the URI in the 'uri' table. However, we have to make sure that the URI is not reffed by any other existing event before pruning it. But this seems to be another bug, although heavily related to this issue.

tags: added: privacy
Revision history for this message
Siegfried Gevatter (rainct) wrote : Re: [Bug 491646] Re: Delete unused stuff from DB

2009/12/3 Mikkel Kamstrup Erlandsen <email address hidden>:
> A more pressing issue is to delete unused URIs after a DeleteEvents
> because of privacy reasons

Deleting stuff that becomes unliked because of a DeleteEvents call is
the whole point of this bug.

--
Siegfried-Angel Gevatter Pujals (RainCT)
Free Software Developer 363DEAE3

Revision history for this message
Siegfried Gevatter (rainct) wrote : Re: Delete unused stuff from DB

s/unliked/unlinked

Changed in zeitgeist:
milestone: 0.3.1 → 0.3.2
Changed in zeitgeist:
milestone: 0.3.2 → 0.3.3
summary: - Delete unused stuff from DB
+ Delete unused URI/actor/etc entries from DB when deleting an event
Revision history for this message
Siegfried Gevatter (rainct) wrote :

Implemented in revision 1392.

Changed in zeitgeist:
status: New → Fix Released
Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

Uhm - Which revision is this fix in? I can't see it in the rev log...

Revision history for this message
Siegfried Gevatter (rainct) wrote : Re: [Bug 491646] Re: Delete unused URI/actor/etc entries from DB when deleting an event

uhm i think it ended up in some merge commit by seif.
If you want to see the code just look at the new triggers in sql.py

2010/4/8, Mikkel Kamstrup Erlandsen <email address hidden>:
> Uhm - Which revision is this fix in? I can't see it in the rev log...
>
> --
> Delete unused URI/actor/etc entries from DB when deleting an event
> https://bugs.launchpad.net/bugs/491646
> You received this bug notification because you are a direct subscriber
> of the bug.
>

--
Siegfried-Angel Gevatter Pujals (RainCT)
Free Software Developer 363DEAE3

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.