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.

Sign up for GitHub

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

Jump to bottom

misc(hdfs): Cleanup header (#11585) #11891

Closed
wants to merge 1 commit into from
Closed

misc(hdfs): Cleanup header (#11585) #11891

wants to merge 1 commit into from

Conversation

majetideepak
Copy link
Collaborator

@majetideepak majetideepak commented Dec 17, 2024 •
edited
Loading

No description provided.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 17, 2024
@netlify Netlify
Copy link

netlify bot commented Dec 17, 2024 •
edited
Loading

✅ Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit e8844b4
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/6761cf3f0474bf00088dbeb6

@majetideepak majetideepak changed the title Revert "fix: Refactor package namespace in HdfsFileSystem (#11585)" feat(revert): Refactor package namespace in HdfsFileSystem (#11585) Dec 17, 2024
@majetideepak majetideepak changed the title feat(revert): Refactor package namespace in HdfsFileSystem (#11585) fix(revert): Refactor package namespace in HdfsFileSystem (#11585) Dec 17, 2024
@majetideepak majetideepak changed the title fix(revert): Refactor package namespace in HdfsFileSystem (#11585) fix: Cleanup hdfs header (#11585) Dec 17, 2024
@majetideepak majetideepak marked this pull request as ready for review December 18, 2024 16:09
@majetideepak majetideepak changed the title fix: Cleanup hdfs header (#11585) misc(hdfs): Cleanup header (#11585) Dec 18, 2024
@majetideepak
Copy link
Collaborator Author

@JkSelf can you review this small change? Thanks.

JkSelf
JkSelf reviewed Dec 19, 2024
View reviewed changes
Copy link
Collaborator

@JkSelf JkSelf 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 your work.

@majetideepak majetideepak added the ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall label Dec 19, 2024
@facebook-github-bot
Copy link
Contributor

@pedroerp has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@pedroerp merged this pull request in 2af045d.

@majetideepak majetideepak deleted the revert-hdfs branch December 19, 2024 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JkSelf JkSelf JkSelf approved these changes

Assignees
No one assigned
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@majetideepak @facebook-github-bot @JkSelf