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

lakectl: block fs stage command for objects in the storage namespace #6696

Merged
merged 5 commits into from
Oct 4, 2023

Conversation

johnnyaug
Copy link
Contributor

@johnnyaug johnnyaug commented Oct 4, 2023

Resolves #6537.

@johnnyaug johnnyaug added the include-changelog PR description should be included in next release changelog label Oct 4, 2023
@johnnyaug johnnyaug changed the title lakectl: fs stage command only for external objects lakectl: allow fs stage command only for external objects Oct 4, 2023
@johnnyaug johnnyaug requested a review from nopcoder October 4, 2023 08:14
@github-actions
Copy link

github-actions bot commented Oct 4, 2023

♻️ PR Preview 1727ae0 has been successfully destroyed since this PR has been closed.

🤖 By surge-preview

@johnnyaug johnnyaug changed the title lakectl: allow fs stage command only for external objects lakectl: block fs stage command for objects in the storage namespace Oct 4, 2023
Copy link
Contributor

@nopcoder nopcoder left a comment

Choose a reason for hiding this comment

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

LGTM!

@nopcoder nopcoder merged commit a6153ff into master Oct 4, 2023
34 of 35 checks passed
@nopcoder nopcoder deleted the lakectl/allow_stage_only_for_external_objects branch October 4, 2023 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
include-changelog PR description should be included in next release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

API cleanup: consolidate stage ingest and link physical address
2 participants