Skip to content

Remove bin/alluxio metadata load in favor of load job cmd#18030

Merged
alluxio-bot merged 1 commit intoAlluxio:mainfrom
Xenorith:removeLoadMetadataCmd
Aug 19, 2023
Merged

Remove bin/alluxio metadata load in favor of load job cmd#18030
alluxio-bot merged 1 commit intoAlluxio:mainfrom
Xenorith:removeLoadMetadataCmd

Conversation

@Xenorith
Copy link
Copy Markdown
Contributor

@Xenorith Xenorith commented Aug 18, 2023

loadMetadataCommand was deleted in #18020 so remove the go cli code calling it. users should be using the load command from the job service with the --metadata-only flag to perform the same action, to be restored in a subsequent PR

@Xenorith Xenorith requested a review from yyongycy August 18, 2023 21:57
Copy link
Copy Markdown
Contributor

@yyongycy yyongycy left a comment

Choose a reason for hiding this comment

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

lgtm

@yyongycy yyongycy added the type-debt This issue is about tech debt label Aug 19, 2023
@yyongycy
Copy link
Copy Markdown
Contributor

alluxio-bot, merge this please

@alluxio-bot alluxio-bot merged commit 04c3af8 into Alluxio:main Aug 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-debt This issue is about tech debt

3 participants