Skip to content

Conversation

@Satbek
Copy link
Contributor

@Satbek Satbek commented Nov 20, 2025

added module bucket_ref_unref.lua with functions bucket_refrw, bucket_unrefrw, bucket_refro, etc ...

module looks at rebalancer.safe_mode flag. if it's disabled module does nothing, otherwise module call ref fucnctions

bucket_ref fail indicated bucket movement.
it will help with problem with double buckets.

What has been done? Why? What problem is being solved?

I didn't forget about

  • Tests
  • Changelog
  • Documentation

Closes #???

@Satbek Satbek marked this pull request as draft November 20, 2025 16:04
@Satbek Satbek changed the title call bucket_ref/bucket_unref on crud operations TNTP-2109: call bucket_ref/bucket_unref on crud operations Nov 21, 2025
@Satbek Satbek force-pushed the TNTP-2109-bucket-ref-unref branch 9 times, most recently from 45dc21b to e19fc3e Compare November 24, 2025 13:20
@Satbek Satbek requested a review from ita-sammann November 26, 2025 10:54
@Satbek Satbek requested a review from ita-sammann November 28, 2025 07:34
@Satbek Satbek marked this pull request as ready for review December 2, 2025 08:05
@Satbek Satbek force-pushed the TNTP-2109-bucket-ref-unref branch 4 times, most recently from 123d504 to 83c2fdc Compare December 5, 2025 10:09
In fast mode ref/unref do nothing.
In safe mode they call bucket_refro,bucket_refrw,
bucket_unrefo,bucket_unrefrw.

Also added ref error handle from storages.
On ref error router will reset bucket, change replicaset
for single operations and retry request.
* add access to _bucket space in privillage tests
* fix select_readview_test
* enable double buckets test
* add safe_mode_disable for cartirdge reload test: test_storage
* add safe_mode_disable for test_any_vshard_call_timeout
@Satbek Satbek force-pushed the TNTP-2109-bucket-ref-unref branch from 83c2fdc to d41e509 Compare December 5, 2025 10:11
@ita-sammann
Copy link

Commits cherry-picked to #467

@ita-sammann ita-sammann closed this Dec 5, 2025
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

Successfully merging this pull request may close these issues.

3 participants