-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Storage] Fix storage-blob samples as per the api updates #3281
[Storage] Fix storage-blob samples as per the api updates #3281
Conversation
Not sure how we missed fixing these. I had to do the same in other PR as well. |
|
||
// Parallel uploading with uploadFileToBlockBlob in Node.js runtime | ||
// uploadFileToBlockBlob is only available in Node.js | ||
await uploadFileToBlockBlob(localFilePath, blockBlobClient, { | ||
blockSize: 4 * 1024 * 1024, // 4MB block size | ||
parallelism: 20, // 20 concurrency | ||
progress: ev => console.log(ev) | ||
progress: (ev) => console.log(ev) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe disable the auto-fixing/formatting of the linting extension? that way formatting changes won't be in PRs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please bear with this until @KarishmaGhiya standardizes eslint ☀
The other packages seem to have |
@@ -108,7 +94,7 @@ async function main() { | |||
async function streamToString(readableStream: NodeJS.ReadableStream) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I remembered helper methods like toStream
, toStream
will be integrated into SDK right? @jeremymeng
No description provided.