libmdbx icon indicating copy to clipboard operation
libmdbx copied to clipboard

Alter the terminology

Open erthink opened this issue 5 years ago • 4 comments

Historically libmdbx uses the terminology (i.e. environment, database, sub-database, DBI, etc) inherited from the LMDB. However, the terms suggested by @snej seem to me so much more successful and appropriate.

erthink avatar Nov 16 '20 22:11 erthink

Thanks 😊 I've been using those terms in my code for a while, but I hadn't proposed them for libmdbx because I figured you'd want to stay consistent with LMDB.

snej avatar Nov 16 '20 23:11 snej

unclear why need special term "snapshot"? why "read transaction" is not enough?

AskAlexSharov avatar Nov 19 '20 02:11 AskAlexSharov

When learning LMDB I initially found "transaction" confusing for a read-only mode, especially since it's required for any access to keys/values.

Also, my wrapper has an OO API, with regular transactions a subclass of read-only ones, and I always find it a bad sign when a base class name is an extension of a subclass name.

snej avatar Nov 23 '20 17:11 snej

For what it's worth, I adopted @snej 's terminology for the Ruby bindings. Coming from ZFS systems, "snapshot" feels the perfect term for its behavior.

mahlonsmith avatar May 10 '21 06:05 mahlonsmith