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

Directory listing speed regression with large numbers of folders #14775

Closed
metadaddy opened this issue Jun 7, 2023 · 0 comments · Fixed by #14792
Closed

Directory listing speed regression with large numbers of folders #14775

metadaddy opened this issue Jun 7, 2023 · 0 comments · Fixed by #14792
Labels
b2 Backblaze B2 Protocol Implementation
Milestone
8.6.2

Comments

@metadaddy
Copy link

metadaddy commented Jun 7, 2023 •
edited by dkocher

Describe the bug
Despite #14542, buckets with large numbers of 'folders' in the root take a long time to load in Cyberduck 8.6.0. The log shows that Cyberduck is calling b2_list_file_names for every 'folder' in the root of the bucket.

To Reproduce
Steps to reproduce the behavior:

  1. Create a bucket in B2 with tens of thousands of 'folders'.
  2. Browse around the bucket in Cyberduck 8.6.0 vs the previous 8.5.7.

Expected behavior
Listing bucket contents should be as fast as it was in prior versions

Desktop:

  • OS: macOS
  • Version 13.4

Screenshots
If applicable, add screenshots to help explain your problem.

Log Files
As a test, I created 1,000 'folders' in a bucket. Cyberduck took 42 seconds to display the bucket listing.

See https://gist.github.com/metadaddy/2447edfa8076742e6e692113705df525
@dkocher dkocher mentioned this issue Jun 11, 2023
Closed
@dkocher dkocher added the b2 Backblaze B2 Protocol Implementation label Jun 16, 2023
@dkocher dkocher changed the title Backblaze B2 bucket file listing speed regression with large numbers of folders Directory listing speed regression with large numbers of folders Jun 16, 2023
@dkocher dkocher linked a pull request Jul 6, 2023 that will close this issue
Revert looking up directory placeholder file #14792
Merged
@dkocher dkocher added this to the 8.6.2 milestone Jul 16, 2023
@dkocher dkocher mentioned this issue Jul 16, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
b2 Backblaze B2 Protocol Implementation
Projects
None yet
Milestone
8.6.2
Development

Successfully merging a pull request may close this issue.

Revert looking up directory placeholder file iterate-ch/cyberduck
2 participants
@dkocher @metadaddy