Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

ognyanstoimenov
Copy link
Collaborator

@ognyanstoimenov ognyanstoimenov commented Aug 26, 2025

Reference Issues/PRs

What does this implement or fix?

#2461 introduced tokens for staged data behind a feature toggle.

This PR removes the feature toggle and updates the docstrings for the new API's. Also depecates write with parameter staged

Any other comments?

Checklist

Checklist for code changes...
  • Have you updated the relevant docstrings, documentation and copyright notice?
  • Is this contribution tested against all ArcticDB's features?
  • Do all exceptions introduced raise appropriate error messages?
  • Are API changes highlighted in the PR description?
  • Is the PR labelled as enhancement or bug so it appears in autogenerated release notes?

@ognyanstoimenov ognyanstoimenov added the minor Feature change, should increase minor version label Aug 26, 2025
"Staging data with write() is deprecated. Use stage() instead.",
DeprecationWarning,
stacklevel=2,
)
Copy link
Collaborator

Choose a reason for hiding this comment

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

We need the same warning with append when incomplete=True

Removes previous (non-snapshotted) versions from the database.
staged : bool, default=False
Whether to write to a staging area rather than immediately to the library.
See documentation on `finalize_staged_data` for more information.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would keep a docstring for this argument, but say that it is deprecated and to use stage instead

Copy link
Collaborator

@G-D-Petrov G-D-Petrov left a comment

Choose a reason for hiding this comment

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

Approved contingent on Alex's comments being implemented

@ognyanstoimenov ognyanstoimenov merged commit fbbe5fd into master Aug 29, 2025
142 of 144 checks passed
@ognyanstoimenov ognyanstoimenov deleted the stage_data_with_tokens_release branch August 29, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Feature change, should increase minor version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants