Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Renaming thread #9

Closed
madolson opened this issue Mar 22, 2024 · 136 comments
Closed

Renaming thread #9

madolson opened this issue Mar 22, 2024 · 136 comments

Comments

@madolson
Copy link
Member

madolson commented Mar 22, 2024

Starting a thread to find ideas for folks to propose alternative naming ideas. If you have an idea, please post it here in a single comment. 👍 for ideas that people like. If your name is not descriptive or clear what it means, also please include a description about why.

@madolson
Copy link
Member Author

openkvs

@madolson madolson changed the title Remaning thread Renaming thread Mar 22, 2024
@hpatro
Copy link
Contributor

hpatro commented Mar 22, 2024

openkv

@mullim
Copy link

mullim commented Mar 22, 2024

OpenCache

@kevinmcgehee
Copy link
Contributor

OpenStruct

@viktorix
Copy link

Radish 😁

@adiologydev
Copy link

PrismDB
PrismKV

RealmDB
RealmKV

These are my suggestions :D

@joshmanders
Copy link

DynamoKV

@antstanley
Copy link

Redis came from Remote Dictionary Server... so maybe Open Dictionary Server -> Odis

@mcollina
Copy link

CampoKV, because Salvatore was born in Campobello di Licata.

@RussellRollins
Copy link

bluewas

@bentotten
Copy link
Contributor

Bredis and have it be stacked toast 🍞

@bentotten
Copy link
Contributor

Greenis

@mswilson
Copy link

Dass (DAta Structure Server).

And when you get it as a service, it's DassaaS

@madolson
Copy link
Member Author

madolson commented Mar 23, 2024

Keyvi
kivi

Or some similar spelling. I recently went to New Zealand, if you haven't seen kiwi birds they are so adorable!

@nichchun
Copy link
Contributor

nichchun commented Mar 23, 2024

fledis - For those of us who have "fled" open source Redis after Garantia Data took it

@KarthikSubbarao
Copy link
Member

KarthikSubbarao commented Mar 23, 2024

Keyvi kivi

Or some similar spelling. I recently went to New Zealand, if you haven't seen kiwi birds they are so adorable!

+1. I was suggesting kiwi earlier since it sounds close to KV (Key-Value).

@Conan-Kudo
Copy link
Contributor

This could get... confusing with "kiwi" (an image build tool). I would appreciate it if we didn't do that. :)

@rlunar
Copy link

rlunar commented Mar 23, 2024

OrcaKV
VoltKV (VoltDB already exists)
BrisKV
SwiftKV (I know not this one)
SleeKV

@nadav-levanoni
Copy link
Contributor

SalvatoreDB antirezDB

@Johno72
Copy link

Johno72 commented Mar 23, 2024

Whodis
Newdis

@rbtcollins
Copy link

bluedis

@PingXie
Copy link
Member

PingXie commented Mar 23, 2024

"KV" is likely selling this project short IMO. I also feel that we should move on from "*dis". What happened has happened. Let's move on and build something amazing with a clean image for the community. There are plenty of other options to connect with the former Redis community members/users.

@nadav-levanoni
Copy link
Contributor

nadav-levanoni commented Mar 23, 2024

MemoryDB

Redis has more functionality than a simple key-value store, and this name is generic and quick to understand universally.

This conflicts with the existing MemoryDB service which may be an issue.

@franck-chester
Copy link

oops-i-redis-again
re-redis

@vithalreddy
Copy link

EmberDB

@rudygt
Copy link

rudygt commented Mar 23, 2024

ReVo (redis evolution)

@kkofler
Copy link

kkofler commented Mar 23, 2024

Maybe a play on musical notes? "Redis" is "DD#": "re" is Latin for "D", "dis" is German for "D#". There are a lot of names that could be thought up based on this theme.

@deanlj
Copy link

deanlj commented Mar 23, 2024

ReOpen but may have trademark conflicts

@riferrei
Copy link

OpenRedis?

It worked pretty for OpenSearch, with a name that clearly states the goal of the project: being the [true] open-source version of Elasticsearch. Also, I agree with others about the K/V semantics. Redis is far more than just a key-value store. It's vast set of supported data structures is what makes it so flexible and powerful.

@natoscott
Copy link
Contributor

okiwis is the best shortening we can come up with in this case to keep it distinctive enough.

'kiwis' is just fine and rolls off the tongue IMO. 'okiwis' adds a syllable and sounds like a mistake when you say it out loud ("oh, kiwis").

OK, that's enough bike shedding for me. :)

@viktorix
Copy link

quiwi or quiwe both pronounced kiwi 😄
qui in French is pronounced as ki in kiwi. So technically, in English it would be whowe 🤣

@vongosling
Copy link

vongosling commented Mar 27, 2024

Second OpenKV, or say, RainbowKV. Hope that after the rainbow, our new products will be more open and the community will thrive.

@rphlmr
Copy link

rphlmr commented Mar 27, 2024

Oredis
Open Remote Dictionary Server.

@chinguyen21
Copy link

OpenIMDb
Open In-memory Database.

@martialblog
Copy link

Sider - Redis spelled backwards... also a different spelling of cider 🍏 Who doesn't like fermented apple juice!

@zuiderkwast
Copy link
Contributor

zuiderkwast commented Mar 27, 2024

Many good points. Let's come up with a 5-letter name that isn't similar to any other software name. I live on an island called Orust.

@CRConrad
Copy link

and odis user is taken in github. Before suggesting a name I suggest to check if it is available at least on github.

So Microsoft now owns the very namespace of open source projects? (Or is this just a symptom of kids just barely old enough to know what a file is, but not old enough to know that GitHub is not the same as git?)

@madolson
Copy link
Member Author

So Microsoft now owns the very namespace of open source projects? (Or is this just a symptom of kids just barely old enough to know what a file is, but not old enough to know that GitHub is not the same as git?)

Although that is A problem, it's not THE problem. The problem is that we're concerned people (the average consumer) might legitimately think it stands for "open redis", which is something we need to avoid for legal reasons.

@CRConrad
Copy link

Phoenix, like rising from the ashes. Not bad.

Dig deep down in the history files and you'll find out how the Firefox browser got its name... Yes, they, too, started from Phoenix.

@bjosv
Copy link
Contributor

bjosv commented Mar 27, 2024

Additional ideas for a 5-letter name:

Keval - key-value but squashed..
Clavi - declension of the Latin word for key.

..naming is hard for sure.

@elroyPDX
Copy link

keester - Key Store

@jamietanna
Copy link

Valkey 👀 https://www.linuxfoundation.org/press/linux-foundation-launches-open-source-valkey-community

@zuiderkwast
Copy link
Contributor

Yes! Do you like the name? I'm closing this thread but you can continue commenting here on the name if you want. Thank you for all suggestions!

@Conan-Kudo
Copy link
Contributor

Valkey is a great name! 👍🏾

@joshmanders
Copy link

Thank you for not going with anything revolving around kiwi, I don't know why but it didn't jive with me.

@PenguRinTwo
Copy link

That's an awesome name

@kkofler
Copy link

kkofler commented Mar 29, 2024

Kinda disappointing that we are being asked for suggestions and then a name gets picked that was not suggested here (making the attempt at involving the community kinda pointless), but otherwise I see no issue with the name Valkey.

@joshmanders
Copy link

Kinda disappointing that we are being asked for suggestions and then a name gets picked that was not suggested here (making the attempt at involving the community kinda pointless), but otherwise I see no issue with the name Valkey.

The suggestion could have been made elsewhere not just this thread.

@YohannesTz
Copy link

Safo

@zuiderkwast
Copy link
Contributor

@kkofler I'm sorry you feel that way. It has been a chaos of discussions in multiple forums. All suggestions have been considered. Many of the suggestions were rejected, either by LF or by somebody else, because of trademark issues (too similar to redis or some other trademark according to the idea of "likelihood of confusion" in trademark law) or because of collision with some other name. Another thing is that we were a little afraid that if we post a name, someone else can register the domain name before we do it.

@YohannesTz
Copy link

@kkofler I'm sorry you feel that way. It has been a chaos of discussions in multiple forums. All suggestions have been considered. Many of the suggestions were rejected, either by LF or by somebody else, because of trademark issues (too similar to redis or some other trademark according to the idea of "likelihood of confusion" in trademark law) or because of collision with some other name. Another thing is that we were a little afraid that if we post a name, someone else can register the domain name before we do it.

I understand but I don't get how "Safo" could remotely be close to "redis". anyways ggs

@zuiderkwast
Copy link
Contributor

@YohannesTz ????

@kkofler
Copy link

kkofler commented Mar 29, 2024

Your "Safo" suggestion came too late, the name is already decided.

@PatrickJS
Copy link
Contributor

PatrickJS commented Mar 30, 2024

@kkofler I'm sorry you feel that way. It has been a chaos of discussions in multiple forums. All suggestions have been considered. Many of the suggestions were rejected, either by LF or by somebody else, because of trademark issues (too similar to redis or some other trademark according to the idea of "likelihood of confusion" in trademark law) or because of collision with some other name. Another thing is that we were a little afraid that if we post a name, someone else can register the domain name before we do it.

this is a really great explanation as I did kind of feel the same way thinking it was more likely OpenKV/OpenKVS but given all things the team has to consider it's understandable.

@madolson
Copy link
Member Author

this is a really great explanation as I did kind of feel the same way thinking it was more likely OpenKV/OpenKVS but given all things the team has to consider it's understandable.

OpenKV was also considered a high risk because there were other comparable names. I was hoping we would go with that too after all the more fun names got rejected :/

@PatrickJS
Copy link
Contributor

PatrickJS commented Apr 1, 2024

long live Valkey


here's some random logos

Valkey

Valkey

naglera pushed a commit to naglera/placeholderkv that referenced this issue Apr 8, 2024
…is missed cases to redis-server. (#12322)

Observed that the sanitizer reported memory leak as clean up is not done
before the process termination in negative/following cases:

**- when we passed '--invalid' as option to redis-server.**

```
 -vm:~/mem-leak-issue/redis$ ./src/redis-server --invalid

*** FATAL CONFIG FILE ERROR (Redis 255.255.255) ***
Reading the configuration file, at line 2
>>> 'invalid'
Bad directive or wrong number of arguments

=================================================================
==865778==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 8 byte(s) in 1 object(s) allocated from:
    #0 0x7f0985f65867 in __interceptor_malloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:145
    valkey-io#1 0x558ec86686ec in ztrymalloc_usable_internal /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:117
    valkey-io#2 0x558ec86686ec in ztrymalloc_usable /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:135
    valkey-io#3 0x558ec86686ec in ztryrealloc_usable_internal /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:276
    valkey-io#4 0x558ec86686ec in zrealloc /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:327
    valkey-io#5 0x558ec865dd7e in sdssplitargs /home/ubuntu/mem-leak-issue/redis/src/sds.c:1172
    valkey-io#6 0x558ec87a1be7 in loadServerConfigFromString /home/ubuntu/mem-leak-issue/redis/src/config.c:472
    valkey-io#7 0x558ec87a13b3 in loadServerConfig /home/ubuntu/mem-leak-issue/redis/src/config.c:718
    valkey-io#8 0x558ec85e6f15 in main /home/ubuntu/mem-leak-issue/redis/src/server.c:7258
    valkey-io#9 0x7f09856e5d8f in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58

SUMMARY: AddressSanitizer: 8 byte(s) leaked in 1 allocation(s).

```

**- when we pass '--port' as option and missed to add port number to redis-server.**

```
vm:~/mem-leak-issue/redis$ ./src/redis-server --port

*** FATAL CONFIG FILE ERROR (Redis 255.255.255) ***
Reading the configuration file, at line 2
>>> 'port'
wrong number of arguments

=================================================================
==865846==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 8 byte(s) in 1 object(s) allocated from:
    #0 0x7fdcdbb1f867 in __interceptor_malloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:145
    valkey-io#1 0x557e8b04f6ec in ztrymalloc_usable_internal /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:117
    valkey-io#2 0x557e8b04f6ec in ztrymalloc_usable /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:135
    valkey-io#3 0x557e8b04f6ec in ztryrealloc_usable_internal /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:276
    valkey-io#4 0x557e8b04f6ec in zrealloc /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:327
    valkey-io#5 0x557e8b044d7e in sdssplitargs /home/ubuntu/mem-leak-issue/redis/src/sds.c:1172
    valkey-io#6 0x557e8b188be7 in loadServerConfigFromString /home/ubuntu/mem-leak-issue/redis/src/config.c:472
    valkey-io#7 0x557e8b1883b3 in loadServerConfig /home/ubuntu/mem-leak-issue/redis/src/config.c:718
    valkey-io#8 0x557e8afcdf15 in main /home/ubuntu/mem-leak-issue/redis/src/server.c:7258
    valkey-io#9 0x7fdcdb29fd8f in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58

Indirect leak of 10 byte(s) in 1 object(s) allocated from:
    #0 0x7fdcdbb1fc18 in __interceptor_realloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:164
    valkey-io#1 0x557e8b04f9aa in ztryrealloc_usable_internal /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:287
    valkey-io#2 0x557e8b04f9aa in ztryrealloc_usable /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:317
    valkey-io#3 0x557e8b04f9aa in zrealloc_usable /home/ubuntu/mem-leak-issue/redis/src/zmalloc.c:342
    valkey-io#4 0x557e8b033f90 in _sdsMakeRoomFor /home/ubuntu/mem-leak-issue/redis/src/sds.c:271
    valkey-io#5 0x557e8b033f90 in sdsMakeRoomFor /home/ubuntu/mem-leak-issue/redis/src/sds.c:295
    valkey-io#6 0x557e8b033f90 in sdscatlen /home/ubuntu/mem-leak-issue/redis/src/sds.c:486
    valkey-io#7 0x557e8b044e1f in sdssplitargs /home/ubuntu/mem-leak-issue/redis/src/sds.c:1165
    valkey-io#8 0x557e8b188be7 in loadServerConfigFromString /home/ubuntu/mem-leak-issue/redis/src/config.c:472
    valkey-io#9 0x557e8b1883b3 in loadServerConfig /home/ubuntu/mem-leak-issue/redis/src/config.c:718
    valkey-io#10 0x557e8afcdf15 in main /home/ubuntu/mem-leak-issue/redis/src/server.c:7258
    valkey-io#11 0x7fdcdb29fd8f in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58

SUMMARY: AddressSanitizer: 18 byte(s) leaked in 2 allocation(s).

```

As part analysis found that the sdsfreesplitres is not called when this condition checks are being hit.

Output after the fix:


```
vm:~/mem-leak-issue/redis$ ./src/redis-server --invalid

*** FATAL CONFIG FILE ERROR (Redis 255.255.255) ***
Reading the configuration file, at line 2
>>> 'invalid'
Bad directive or wrong number of arguments
vm:~/mem-leak-issue/redis$

===========================================
vm:~/mem-leak-issue/redis$ ./src/redis-server --jdhg

*** FATAL CONFIG FILE ERROR (Redis 255.255.255) ***
Reading the configuration file, at line 2
>>> 'jdhg'
Bad directive or wrong number of arguments

---------------------------------------------------------------------------
vm:~/mem-leak-issue/redis$ ./src/redis-server --port

*** FATAL CONFIG FILE ERROR (Redis 255.255.255) ***
Reading the configuration file, at line 2
>>> 'port'
wrong number of arguments
```

Co-authored-by: Oran Agra <[email protected]>
madolson added a commit that referenced this issue Jul 24, 2024
Fixes #796

Currently rioConnWite uses 1024 bytes chunk when feeding the replication
sockets on RDB write. This value seems too small and we will end up with
high syscall overhead.
**This PR sets the max chunk size to 16K.**

Using a simple test program we did not observe any significant
improvement in read/write times going with chunks bigger than 4K but
that might be la bottleneck on network throughput. We did observe sweet
point of CPU utilization at 16K when using TLS.

```
lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 24.04 LTS
Release:	24.04
Codename:	noble
```

```
uname -a
Linux ip-172-31-22-140 6.8.0-1009-aws #9-Ubuntu SMP Fri May 17 14:39:23 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
```
All files were compiled with O3 optimization level.
```
gcc --version
gcc (Ubuntu 13.2.0-23ubuntu4) 13.2.0
```

**Results:**

Chunk Size | write-time sec | writes total | write cpu-time (usr+sys) |
read-time sec | read total syscalls | read cpu-time (usr+sys)
-- | -- | -- | -- | -- | -- | --
1K | 0.162946 | 102400 | 0.185916 | 0.168479 | 2447 | 0.026945
4K | 0.163036 | 25600 | 0.122629 | 0.168627 | 715 | 0.023382
8K | 0.163942 | 12800 | 0.121131 | 0.168887 | 704 | 0.039388
16K | 0.163614 | 6400 | 0.104742 | 0.168202 | 2483 | 0.025574
64K | 0.16279 | 1600 | 0.098792 | 0.168854 | 1068 | 0.046929
1K - TLS | 0.32648 | 102400 | 0.366961 | 0.330785 | 102400 | 0.337377
4K - TLS | 0.164296 | 25600 | 0.183326 | 0.169032 | 25600 | 0.129952
8K - TLS | 0.163977 | 12800 | 0.163118 | 0.169484 | 12800 | 0.098432
16K - TLS | 0.164861 | 6400 | 0.150666 | 0.169878 | 6383 | 0.094794
64K - TLS | 0.163704 | 6400 | 0.156125 | 0.169323 | 6388 | 0.089971

---------

Signed-off-by: Ran Shidlansik <[email protected]>
Signed-off-by: ranshid <[email protected]>
Signed-off-by: Binbin <[email protected]>
Co-authored-by: Madelyn Olson <[email protected]>
Co-authored-by: Binbin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests