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 714101 - Allow user to specify oldest mail stored in database
Allow user to specify oldest mail stored in database
Status: RESOLVED DUPLICATE of bug 766581
Product: geary
Classification: Other
Component: client
master
Other All
: High normal
: ---
Assigned To: Geary Maintainers
Geary Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-11-14 06:25 UTC by Geary Maintainers
Modified: 2017-12-08 00:08 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Charles Lindsay 2013-11-21 20:24:38 UTC


---- Reported by geary-maint@gnome.bugs 2012-11-14 10:25:00 -0800 ----

Original Redmine bug id: 6073
Original URL: http://redmine.yorba.org/issues/6073
Searchable id: yorba-bug-6073
Original author: Andrea Corbellini
Original description:

I noticed that all the emails and attachments downloaded by Geary are never
deleted, even very old ones.

Geary should try to keep its cache at a reasonable size. The most simple
solution would be to keep just the X most recent messages (25 or 50 is
probably enough).

Update: We'd like to have Geary offer the user a setting that specifies a date
range (six months, year, forever) of the oldest mail to store in the database.

Related issues:
related to geary - Feature #4293: Display messages not in folder as part of
conversation ("... (Fixed)
related to geary - Feature #4236: periodic database cleanup (Open)
related to geary - Feature #6365: Prefetch mail in background (Fixed)
related to geary - Feature #6402: Allow user to specify how much mail to pull
from server (Fixed)
related to geary - 6497: once rows can be deleted, can't store row ids (Open)
duplicated by geary - Feature #6372: Allow user to specify deletion of old
mail (Duplicate)



---- Additional Comments From geary-maint@gnome.bugs 2013-07-11 15:17:00 -0700 ----

### History

####

#1

Updated by Jim Nelson about 1 year ago

  * **Category** set to _engine_
  * **Priority** changed from _Normal_ to _High_
  * **Target version** set to _0.3.0_

This is something we've discussed but haven't ticketed yet. However, instead
of keeping only the most recent 25 - 50 messages, we were thinking that Geary
could be configured to hold mail back to a certain date, i.e the last six
months. Keeping so few messages in the database would seriously cripple
Geary's performance. Would that work for you?

####

#2

Updated by Andrea Corbellini about 1 year ago

Yes, it would.

####

#3

Updated by Jim Nelson about 1 year ago

  * **Subject** changed from _download cache grows infinitely_ to _Allow user to specify oldest mail stored in database_
  * **Description** updated (diff)

####

#4

Updated by Jim Nelson about 1 year ago

  * **Tracker** changed from _Bug_ to _Feature_

####

#5

Updated by Jim Nelson 9 months ago

To be more explicity about this ticket:

Geary should have a Preferences option that allows for the user to select the
oldest message held in the local database. A separate checkbox could specify
whether or not to ever delete old messages. These preferences should be handed
to the Engine via an EngineSettings object (roughly analogous to
AccountInformation). We could conceivably offer the first option initially and
worry about the second later.

Thus, this ticket is dealing with two configuration concepts: (a) how much
mail to pull from the server (#6365), and (b) when to delete mail from the
local database, if ever (#4236).

####

#6

Updated by Jim Nelson 9 months ago

  * **Category** changed from _engine_ to _client_

We discussed this today and landed on the following proposal:

  * The default value is two months.
  * Each account has its own setting (so there's no need for an EngineSettings yet; this value should be stored in AccountInformation).
  * The value may be adjusted in the edit-account pane. It's not available in the Welcome dialog or the add-account pane.
  * The value is adjusted via a slider, the settings to be decided. Text beneath the slider describes the setting: "Download conversations up to 2 months old", up to "Download all conversations".
  * There is no option to delete old messages. That will have to wait for 0.4 or later: #6372

####

#7

Updated by Jim Nelson 9 months ago

  * **Assignee** set to _Eric Gregory_

####

#8

Updated by Jim Nelson 9 months ago

  * **Assignee** deleted (<strike>_Eric Gregory_</strike>)

As Eric pointed out, the task assigned to this ticket doesn't quite match the
original ticket's description, so doing some shuffling here to make it work.

This ticket is for doing the following:

There should be a checkbox beneath the email-age control in Preferences
(#6402) that says, "Delete local conversations older than above value". The
wording can be better. In particular, it should be expressed that Geary does
**not** delete messages from the server.

####

#9

Updated by Jim Nelson 9 months ago

  * **Target version** changed from _0.3.0_ to _0.4.0_

####

#10

Updated by Jim Nelson 8 months ago

A user has reported that this feature would be of some help to him:
https://bugs.launchpad.net/geary/+bug/1157489

####

#11

Updated by Jim Nelson 4 months ago

  * **Target version** changed from _0.4.0_ to _0.5.0_



--- Bug imported by chaz@yorba.org 2013-11-21 20:24 UTC  ---

This bug was previously known as _bug_ 6073 at http://redmine.yorba.org/show_bug.cgi?id=6073

Unknown milestone "unknown in product geary. 
   Setting to default milestone for this product, "---".
Setting qa contact to the default for this product.
   This bug either had no qa contact or an invalid one.
Resolution set on an open status.
   Dropping resolution 

Comment 1 Michael Gratton 2017-01-11 02:57:31 UTC
Cleaning up attachments is covered up by Bug 766581, so I guess this part covers the message body only and/or headers.
Comment 2 Michael Gratton 2017-12-08 00:08:46 UTC
After thinking about this some more, I think Geary should just be using a LRU cache rather than having a pref for it, so I'm going to makr this as a duplicate of Bug 766581.

*** This bug has been marked as a duplicate of bug 766581 ***