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

Allow calling app to cancel enumeration and handle scanning directory errors in traverser #1803

Merged
merged 8 commits into from
Jun 14, 2022

Conversation

Strikerzee
Copy link
Contributor

Enable calling app to cancel enumeration

  • Used appCtx in parallel-walk so that the app can cancel the enumeration as it requires.
  • It doesn't effect azcopy working.

Handle scanning directory errors in traverser.

  • Fail visibly when scanning fails for files and directories
  • Add in more error paths
  • Fix processOneDirectory changes and add more comments

@zezha-msft zezha-msft added this to the xdm-integration milestone Jun 2, 2022
@mohsha-msft
Copy link
Contributor

Please rebase with dev branch. The PR is missing unit test cases. Please write test cases to test the fuctionalities.

@Strikerzee
Copy link
Contributor Author

Please rebase with dev branch. The PR is missing unit test cases. Please write test cases to test the fuctionalities.

Created a work item for the same.

@Strikerzee Strikerzee force-pushed the personal/arnav/cancel-enumeration-fix-scandir-err branch from da38610 to 1743806 Compare June 10, 2022 05:30
nitin-deamon and others added 6 commits June 10, 2022 05:36
- Used appCtx in parallel-walk so that when app want to cancel the
enumeration it can do.
- It doesn't effect azcopy working.
* Fail visibly when scanning fails for files and directories

* add in more error paths

* fix processOneDirectory changes and add more comments
@Strikerzee Strikerzee force-pushed the personal/arnav/cancel-enumeration-fix-scandir-err branch from 1743806 to e1bd6f1 Compare June 10, 2022 05:37
cmd/zc_enumerator.go Show resolved Hide resolved
cmd/zc_traverser_local.go Outdated Show resolved Hide resolved
cmd/zc_traverser_local.go Outdated Show resolved Hide resolved
cmd/zc_traverser_local.go Outdated Show resolved Hide resolved
cmd/zc_traverser_local.go Outdated Show resolved Hide resolved
common/parallel/FileSystemCrawler.go Show resolved Hide resolved
@Strikerzee Strikerzee merged commit 0e4c1a7 into dev Jun 14, 2022
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.

None yet

5 participants