[tarantool-patches] Re: [PATCH 04/12] vinyl: rename compact to compaction
Konstantin Osipov
kostja at tarantool.org
Thu Jan 17 16:51:42 MSK 2019
* Vladimir Davydov <vdavydov.dev at gmail.com> [19/01/17 16:08]:
> On Thu, Jan 17, 2019 at 02:34:49PM +0300, Konstantin Osipov wrote:
> > * Vladimir Davydov <vdavydov.dev at gmail.com> [19/01/15 17:20]:
> > > compact_input sounds confusing, because 'compact' works as an adjective
> > > here. Saving 3 characters per variable/stat name related to compaction
> > > doesn't justify this. Let's rename 'compact' to 'compaction' both in
> > > stats and in the code.
> >
> > Actually, compaction_input also can be confused with the current
> > compaction queue size. Perhaps we could play with word "total" to
> > distinguish the current/last amounts and the total amounts in variable
> > names.
>
> The names are long enough as they are so I'd avoid adding total_
> prefixes. For queue, we have compaction_queue.
How do you define "long enough" ? Perhaps they look long inthe
source code but they are definitely not long when displayed on a
computer screen in box.stat.vinyl() output.
--
Konstantin Osipov, Moscow, Russia, +7 903 626 22 32
http://tarantool.io - www.twitter.com/kostja_osipov
More information about the Tarantool-patches
mailing list