Describe the bug
Lua user scripts stopped to work due to "readonly table script" in redis7
To reproduce
see a function that worked with redis 6: https://github.com/getsentry/sentry/blob/ce94aae1da18376b01440e3954f4f292a40d1729/src/sentry/scripts/digests/digests.lua#L11-L20
I haven't found anything related to a readonly table script in release notes.
Expected behavior
there should be an ability to override table's Lua functions
Additional information
see https://github.com/getsentry/sentry/issues/35487
Comment From: oranagra
Could be related to https://github.com/redis/redis/pull/10651 Can you try 6.2.7?
Comment From: kayrus
@oranagra thanks, I tried to find it in issues, but not pulls.
Comment From: oranagra
@kayrus i didn't mean that that PR solves your issue. That's the one that probably created it. In which case 6.2.6 will be OK, but 6.2.7 won't.
Comment From: kayrus
@oranagra it doesn't solve an issue, but I noticed related issues/PR there, which at the end led me to a proper solution. Therefore please feel free to close this issue. I apologies that I haven't been able to find this PR at the first place and consumed your time.
Comment From: oranagra
@kayrus can you please elaborate. Why did that script mess with the Lua table? was it necessary? what solution have you found?
We'd like to keep that change in Redis, but I want to try and estimate how many other apps will be broken by it.
Comment From: oranagra
Ohhh, I now see you already did in the other PR. Thanks.