Skip to content

Reflect async _walk correctly#2040

Merged
martindurant merged 3 commits into
fsspec:masterfrom
martindurant:walk-coro
May 25, 2026
Merged

Reflect async _walk correctly#2040
martindurant merged 3 commits into
fsspec:masterfrom
martindurant:walk-coro

Conversation

@martindurant

Copy link
Copy Markdown
Member

Fixes #2002

@dAnjou , sorry for the slow response.
The issue was, that ._walk is not a coroutine function, but a coroutine generator function, and the wrapping code didn't consider this case.

@martindurant martindurant merged commit 71b8a6b into fsspec:master May 25, 2026
11 checks passed
@martindurant martindurant deleted the walk-coro branch May 25, 2026 18:17
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.

Subclass of AsyncFileSystem doesn't use its _walk, goes straight to AbstractFileSystem.walk instead

1 participant