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

Limit TOMBSTONE members number to one #311

Open
carpawell opened this issue Dec 2, 2024 · 2 comments
Open

Limit TOMBSTONE members number to one #311

carpawell opened this issue Dec 2, 2024 · 2 comments
Labels
enhancement Improving existing functionality I2 Regular impact S2 Regular significance U3 Regular
Milestone

Comments

@carpawell
Copy link
Member

Is your feature request related to a problem? Please describe.

I'm always frustrated when there is only a single object in the DELETE RPC but a lot of objects are in the tombstone's body. It is even more unexpected after #297 (however, it is not that clear when we are cleaning unfinished big objects).

Describe the solution you'd like

Disallow "big" tombstones. One TS -- one object. Tombstones expire sooner or later and tombstones with many objects are not expected, it is even not possible currently due to DELETE RPC, only if someone creates it manually.

Describe alternatives you've considered

Keep it as is.

Additional context

Uncontrollable number of deleted objects by a single object makes it harder to maintain object meta information: nspcc-dev/neofs-contract#448 (comment).

@carpawell carpawell added I2 Regular impact feature Completely new functionality labels Dec 2, 2024
@carpawell carpawell changed the title Limit tombstone members number to one Limit TOMBSTONE members number to one Dec 2, 2024
@roman-khimov roman-khimov added U4 Nothing urgent U3 Regular S2 Regular significance enhancement Improving existing functionality and removed U4 Nothing urgent feature Completely new functionality labels Dec 2, 2024
@cthulhu-rider
Copy link
Contributor

rly desired but very breakin change, lets do asap. #312 too

@roman-khimov
Copy link
Member

It also allows to do placement by the original object OID in future making auxiliary objects follow original ones with higher probability (reducing negative effects from things getting out of sync).

@roman-khimov roman-khimov added this to the v2.17.0 milestone Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improving existing functionality I2 Regular impact S2 Regular significance U3 Regular
Projects
None yet
Development

No branches or pull requests

3 participants