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.

Sign up for GitHub

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

Jump to bottom

Implement method to iterate over estimated container sizes #293

Closed
vdomnich-yadro opened this issue Nov 17, 2022 · 1 comment
Closed

Implement method to iterate over estimated container sizes #293

vdomnich-yadro opened this issue Nov 17, 2022 · 1 comment
Assignees
@vdomnich-yadro

Comments

@vdomnich-yadro
Copy link
Contributor

To process all container size estimates on epoch we are using combination of 2 methods of container smart contract:

However, when there are many containers in the system, this method doesn't work efficiently. It takes about 3-4 seconds to process estimates of ~500 containers. To speed up processing on a system with multiple containers we want to implement an alternative method that would have signature:

func IterateContainerSizes(epoch int) iterator.Iterator { }
@vdomnich-yadro vdomnich-yadro mentioned this issue Dec 22, 2022
Closed
roman-khimov added a commit that referenced this issue Jan 17, 2023
@roman-khimov
Merge branch 'support/v0.16' into master
7a29056 
* origin/support/v0.16:
  common: bump version, do netmap migrations pre-0.16.0
  netmap: migrate candidate states for pre-0.16.0
  netmap: fix infinite loop in pre-0.16.0 migration code
  [#300] container: Allow to iterate over container list
  [#293] container: Add IterateContainerSizes method
  [#296] container: Increase default expiration time
  [#291] Debian packaging
@roman-khimov
Copy link
Member

Fixed by #295.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vdomnich-yadro vdomnich-yadro

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@roman-khimov @vdomnich-yadro