erickguan commented on PR #5226: URL: https://github.com/apache/opendal/pull/5226#issuecomment-2430054918
I experimented a bit ([patch](https://gist.github.com/erickguan/80bbf2ea82a10c69d260c53f0dd2f97b)) with the async support for listing. Async helps reduce the run time of course. But extending `OpList` is a breaking change. Since OpenDAL has a compat package, I would happily sync with you when you are preparing a breaking release to reduce the amount of work. (no rush in async support) Some other findings when running behavior tests but not related to the PR: 1. the behavior test fails on my machine but not in CI. I am casually digging the cause. 2. the uuid random generation in behavior tests generate 3 levels of UUID. This will have truly low chance of collision on CI. But I also want to reduce the level when running against local environment. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
