-
Notifications
You must be signed in to change notification settings - Fork 10k
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
Reduce unnecessary type conversion in writeStream
#17134
Reduce unnecessary type conversion in writeStream
#17134
Conversation
Currently we're unnecessarily converting data between strings and typed-arrays, when dealing with compressible data, in the `writeStream` function. Note how we're *first* getting a string-representation of the stream, which involves converting the underlying typed-array into a string, only to immediately convert this back into a typed-array. This seems completely unnecessary, and is easy enough to avoid, and we'll now only do a *single* type-conversion in this function.
333e508
to
25a1a9d
Compare
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.
LGTM. Thank you.
/botio test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.241.84.105:8877/74e6ea9e5e855f9/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.193.163.58:8877/5e6d08bc0c3b827/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/74e6ea9e5e855f9/output.txt Total script time: 24.64 mins
Image differences available at: http://54.241.84.105:8877/74e6ea9e5e855f9/reftest-analyzer.html#web=eq.log |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/5e6d08bc0c3b827/output.txt Total script time: 36.02 mins
Image differences available at: http://54.193.163.58:8877/5e6d08bc0c3b827/reftest-analyzer.html#web=eq.log |
Currently we're unnecessarily converting data between strings and typed-arrays, when dealing with compressible data, in the
writeStream
function.Note how we're first getting a string-representation of the stream, which involves converting the underlying typed-array into a string, only to immediately convert this back into a typed-array. This seems completely unnecessary, and is easy enough to avoid, and we'll now only do a single type-conversion in this function.