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

session: Purge epoch headers #288

Merged
merged 1 commit into from
Feb 28, 2024
Merged

session: Purge epoch headers #288

merged 1 commit into from
Feb 28, 2024

Conversation

cthulhu-rider
Copy link
Contributor

No description provided.

Copy link
Member

@roman-khimov roman-khimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also deprecate, not drop? We are aiming to have stable API. Although the line is very blurry here, these are just attribute names.

@cthulhu-rider
Copy link
Contributor Author

as well as #287 (comment)

done

roman-khimov
roman-khimov previously approved these changes Feb 26, 2024
Previously added `__NEOFS__NETMAP*` X-headers did not justify themselves:
 - no one has ever used them;
 - even if you try, it’s not clear how to work with them.

Data users operate with containers and objects, they are unaware and
uninterested in system time details. If an object is available, system
must be able to respond to them. Where and when exactly to look for data
is best known only to the storage system itself.

Closes #282.

Signed-off-by: Leonard Lyubich <[email protected]>
@cthulhu-rider cthulhu-rider merged commit 482791b into master Feb 28, 2024
3 checks passed
@cthulhu-rider cthulhu-rider deleted the 282-meta-epochs branch February 28, 2024 06:28
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