512mb limit: No longer true?

[stockholmux]

Documentation lists 512mb on key and string sizes many times:

• /topics/setrange.md
• /topics/bitmaps.md
• /topics/keyspace.md
• /topics/protocol.md

and probably more.

It appears the limitation was removed in 2021 see 4568 on redis. Looks like on issue 1653 on redis-doc there was a never merged issue to update this.

Can anyone confirm that Valkey is no longer subject to the 512mb limit?

[madolson]

As discussed, the 512MB limit is no longer a hard limit and can be changed with proto-max-bulk-len. I think we should frame it as, "it's untested to use a higher value than the default, please use at your own risk.

[zuiderkwast]

Why not do PRs directly instead of reporting as issues?

[stockholmux]

@zuiderkwast This is all linked to #91 which is a review process not necessarily an update process. This is so we don't block publishing of the website by waiting to fix dozens of issues.

If something looks super bad, I can suppress publishing until it's fixed, or alternately for not as bad pages, have a warning that says that we're working on it.

Once the review is finished: ready to publish.

[zuiderkwast]

OK, I just thought that in some cases, just posting the changes as a PR seems to be as small and fast as explaining them in an issue. It's sometimes a bit hard to know what exactly you want to be changed from these issues.

[stockholmux]

re: bit hard to know what exactly you want to be changed from these issues

Probably fair! I'll make sure and take another pass on my own issues.