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

Add move request to job factory #17712

Merged
merged 1 commit into from
Jun 30, 2023
Merged

Add move request to job factory #17712

merged 1 commit into from
Jun 30, 2023

Conversation

ssyssy
Copy link
Contributor

@ssyssy ssyssy commented Jun 29, 2023

What changes are proposed in this pull request?

Add move job request

Why are the changes needed?

Need for distMv command.

Does this PR introduce any user facing changes?

na

@ssyssy ssyssy requested a review from jja725 June 29, 2023 19:57
@ssyssy ssyssy force-pushed the addMoveRequest branch 2 times, most recently from c0208bc to 6dbee90 Compare June 29, 2023 20:13
@ssyssy ssyssy requested a review from jja725 June 29, 2023 20:14
@ssyssy ssyssy force-pushed the addMoveRequest branch 3 times, most recently from dc3fd83 to fe2d84b Compare June 29, 2023 21:11
Copy link
Contributor

@jja725 jja725 left a comment

Choose a reason for hiding this comment

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

LGTM

@ssyssy
Copy link
Contributor Author

ssyssy commented Jun 30, 2023

alluxio-bot, merge this please

@alluxio-bot
Copy link
Contributor

merge failed:
Merge refused because pull request does not have label start with type-

@ssyssy ssyssy added the type-feature This issue is a feature request label Jun 30, 2023
@ssyssy
Copy link
Contributor Author

ssyssy commented Jun 30, 2023

alluxio-bot, merge this please

@alluxio-bot alluxio-bot merged commit 0ffd94e into main Jun 30, 2023
15 checks passed
Xenorith pushed a commit that referenced this pull request Jun 30, 2023
### What changes are proposed in this pull request?

Add move job request

### Why are the changes needed?

Need for distMv command.

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

na

			pr-link: #17712
			change-id: cid-86899c3b7d75bb24077c3f19dc2431e27aac293c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-feature This issue is a feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants