raindrops RubyGem user+dev discussion/patches/pulls/bugs/help
 help / color / mirror / code / Atom feed
From: Eric Wong <e@80x24.org>
To: KJ Tsanaktsidis <ktsanaktsidis@zendesk.com>
Cc: raindrops-public@yhbt.net
Subject: Re: [PATCH] Allow Raindrops objects to be backed by a memfd file
Date: Mon, 22 Nov 2021 17:56:33 +0000	[thread overview]
Message-ID: <20211122175633.GA23628@dcvr> (raw)
In-Reply-To: <CAJ7wOOt20nZaeT-UM=cB5MXb_nogCZSOuNEYRdR73N8AFqfBng@mail.gmail.com>

KJ Tsanaktsidis <ktsanaktsidis@zendesk.com> wrote:
> On Mon, Nov 22, 2021 at 7:48 PM Eric Wong <e@80x24.org> wrote:
> > Anyways thanks again for bringing this up.  Let us know if you
> > want to tackle handling my suggestions or if you want to wait
> > for me to do it.
> 
> I think I committed the two cardinal sins of mailing list etiquette at
> once - replied off list and in HTML /facepalm.

Also, no need to quote excessively :)  Anyways, thanks for
correcting yourself.

Fwiw, any attachment costs around 3x more memory for my spam
filter, and decoding HTML is probably 10x or more in terms of
CPU usage compared to plain-text.

I'm also storing and indexing around 15 million emails across
many projects while trying to get as many independently-run
mirrors up as possible; so yes, these bytes matter :)

> Copying my reply in here again.... sorry!
> 
> --------------------------------------------------------------
> 
> Hi Eric,
> 
> Thank you for the really detailed feedback on this patch! I haven’t really
> written much in the way of Ruby C extensions before, so it’s really
> valuable to me to get your thoughts.

No problem.  While I'm mostly retired from Ruby, it's still a
useful break from my other projects for me to review code.

> I’m definitely happy to send another version of this later this week.

Yeah, no rush; holidays and all.

      reply	other threads:[~2021-11-22 17:56 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-11-22  1:03 KJ Tsanaktsidis
2021-11-22  8:42 ` Eric Wong
2021-11-22 10:13   ` KJ Tsanaktsidis
2021-11-22 17:56     ` Eric Wong [this message]

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

  List information: https://yhbt.net/raindrops/

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

  git send-email \
    --in-reply-to=20211122175633.GA23628@dcvr \
    --to=e@80x24.org \
    --cc=ktsanaktsidis@zendesk.com \
    --cc=raindrops-public@yhbt.net \
    --subject='Re: [PATCH] Allow Raindrops objects to be backed by a memfd file' \
    /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

Code repositories for project(s) associated with this inbox:

	../../raindrops.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).