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

Delete temporary files when uploading files to OBS fails #16056

Merged
merged 1 commit into from
Aug 16, 2022

Conversation

qian0817
Copy link
Contributor

What changes are proposed in this pull request?

delete temporary files when uploading files to OBS fails.

Why are the changes needed?

Currently does not delete temporary files when obs delete fails.

Does this PR introduce any user facing changes?

No user facing changes.

@alluxio-bot
Copy link
Contributor

Automated checks report:

  • Commits associated with Github account: PASS
  • PR title follows the conventions: FAIL
    • The title of the PR does not pass all the checks. Please fix the following issues:
      • First word must be capitalized

Some checks failed. Please fix the reported issues and reply 'alluxio-bot, check this please' to re-run checks.

@qian0817 qian0817 changed the title delete temporary files when uploading files to OBS fails Delete temporary files when uploading files to OBS fails Aug 16, 2022
@alluxio-bot
Copy link
Contributor

Automated checks report:

  • Commits associated with Github account: PASS
  • PR title follows the conventions: PASS

All checks passed!

Copy link
Contributor

@LuQQiu LuQQiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!, thanks for fixing the delete logic

@HelloHorizon
Copy link
Contributor

alluxio-bot, merge this please

@alluxio-bot alluxio-bot merged commit 65a8084 into Alluxio:master Aug 16, 2022
Jackson-Wang-7 pushed a commit to Jackson-Wang-7/alluxio that referenced this pull request Nov 24, 2023
### What changes are proposed in this pull request?

delete temporary files when uploading files to OBS fails.

### Why are the changes needed?

Currently does not delete temporary files when obs delete fails.

### Does this PR introduce any user facing changes?

No user facing changes.

pr-link: Alluxio#16056
change-id: cid-159915bffd7e18df4123ff7dd4c44048245cbb83
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.

5 participants