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

Update Azure Connector Dependencis to consume file-datalake #7688

Closed

Conversation

gaoyangxiaozhu
Copy link
Contributor

This is the PR used to update the azure storage sdk for abfs write support. Asking from @majetideepak with this PR #7218

Copy link

netlify bot commented Nov 22, 2023

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit 8258911
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/655d95220036ab0008e0f431

@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 Nov 22, 2023
@gaoyangxiaozhu
Copy link
Contributor Author

hey @majetideepak I send this split PR for dependency update as your comment, could you help review ?

@majetideepak majetideepak requested a review from kgpai November 25, 2023 10:08
@gaoyangxiaozhu
Copy link
Contributor Author

@kgpai could you help review ?

@gaoyangxiaozhu
Copy link
Contributor Author

@kgpai could you help review ?

@majetideepak could you help ping @kgpai for review ?

@kgpai
Copy link
Contributor

kgpai commented Dec 8, 2023

@gaoyangxiaozhu Feel free to reach out to me on slack if you need me to look into anything with some urgency.

@kgpai kgpai closed this Dec 8, 2023
@kgpai kgpai reopened this Dec 8, 2023
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@kgpai merged this pull request in 5c586d0.

Copy link

Conbench analyzed the 1 benchmark run on commit 5c586d07.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants