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

RCORE-2012 Report sync progress for upload download separately #7512

Closed

Conversation

kiburtse
Copy link
Contributor

What, How & Why?

Change the callback in sync implementation for progress to report download/upload progress separately only when it really changes to match object-store user facing api

Fixes #7451

☑️ ToDos

  • 📝 Changelog update
  • 🚦 Tests (or not relevant)
  • C-API, if public C++ API changed
  • bindgen/spec.yml, if public C++ API changed

* Precompute it for existing parameters for initial crud behaviour
* Adjust existing tests
* Add needed spec definitions
…ate field

* Refactor to forward parsed parameters from download message to session as is
reset to 0 from 1 after batch sync or when new data is added during
active sync
@cla-bot cla-bot bot added the cla: yes label Mar 22, 2024
@kiburtse kiburtse changed the base branch from master to kb/sync_progress_estimate_api March 22, 2024 12:44
Base automatically changed from kb/sync_progress_estimate_api to master March 22, 2024 19:32
@kiburtse kiburtse closed this Aug 13, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Report sync progress separately when download or upload progresses
1 participant