Gnus development mailing list
 help / color / mirror / Atom feed
From: Eric Abrahamsen <eric@ericabrahamsen.net>
To: ding@gnus.org
Subject: Re: [PATCH] Two issues with the gnus-registry
Date: Sun, 16 Nov 2014 11:24:19 +0800	[thread overview]
Message-ID: <87zjbr3k0s.fsf@ericabrahamsen.net> (raw)
In-Reply-To: <87bno8ymyv.fsf@uwo.ca>

Dan Christensen <jdc@uwo.ca> writes:

> Eric Abrahamsen <eric@ericabrahamsen.net> writes:
>
>> 1. The big one changes the database structure, combining :max-hard and
>> :max-soft into :max-size
>
> Does this mean that precious entries will *never* get pruned?
> Will that be a problem?
>
> Also, if precious entries count towards the total, then once
> you have as many precious entries as the total size, no other
> entries will ever be kept...
>
> What are precious entries currently used for?

Yup, the whole point of precious entries is that you don't want them to
be pruned. This was how the registry was supposed to operate to begin
with, it just wasn't working correctly.

And yes, the registry refuses to add new entries over the total size,
precious or not. When it hits the limit, it tries to prune. If all the
entries in the registry are precious, it will lock up. I'd be mighty
surprised if anyone ever gets to this situation.

*No* entries are made precious in the default registry setup, so my
changes here won't affect anyone who hasn't gone and used preciousness
for their own purposes. I think that's why no one has run into this
problem before.

Users and package authors who make heavy use of this feature should
probably have a secondary "pruning" routine on hand that removes the
precious keys from entries that no longer need them. The registry could
probably do a little more to signal that it was full of precious
entries, so those routines could be run automatically.

I'd be happy to discuss how all this should work, and make any changes
that Ted is okay with. I think the registry is a great idea, and one
that's currently underutilized.

Eric




  reply	other threads:[~2014-11-16  3:24 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-24 19:04 Eric Abrahamsen
2014-10-24 20:56 ` Eric Abrahamsen
2014-10-25 19:59   ` Eric Abrahamsen
2014-10-27 15:03 ` Ted Zlatanov
2014-10-27 19:15   ` Eric Abrahamsen
2014-10-28 18:04     ` Eric Abrahamsen
2014-11-07 23:56       ` Eric Abrahamsen
2014-11-08  0:01         ` Eric Abrahamsen
2014-11-08  8:39           ` Eric Abrahamsen
2014-11-10 13:54             ` Ted Zlatanov
2014-11-11  2:55               ` Eric Abrahamsen
2014-11-13 12:05               ` Eric Abrahamsen
2014-11-16  1:04                 ` Dan Christensen
2014-11-16  3:24                   ` Eric Abrahamsen [this message]
2014-12-18 10:07                 ` Ted Zlatanov
2014-12-18 15:00                   ` Eric Abrahamsen
2014-12-18 15:09                     ` Eric Abrahamsen
2014-12-19  0:44                       ` Katsumi Yamaoka
2014-12-19  2:08                         ` Eric Abrahamsen
2014-12-20  3:09                         ` Ted Zlatanov
2014-12-20 11:22                           ` Katsumi Yamaoka
2014-12-20 13:53                             ` Older Emacsen (was: [PATCH] Two issues with the gnus-registry) Ted Zlatanov
2014-12-19  1:30                       ` [PATCH] Two issues with the gnus-registry Ted Zlatanov
2014-10-28 20:10     ` Ted Zlatanov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87zjbr3k0s.fsf@ericabrahamsen.net \
    --to=eric@ericabrahamsen.net \
    --cc=ding@gnus.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).