v5.5.0 Release Proposal (Stable) #4742
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This changes vcbuild.bat to accept a new parameter (vc2015 or vc2013) to select the version of Visual Studio to use. PR-URL: #4645 Reviewed-By: João Reis <reis@janeasystems.com>
evanlucas
added
the
meta
Some variables are declared with var more than once in the same scope. This change reduces the declarations to one per scope. PR-URL: #4633 Reviewed-By: jasnell - James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
PR-URL: #4691 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
PR-URL: #4708 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
The last sentence of the explanation for the first stream section seemed a bit confusing. I tried to change the sentence to clarify it. Additionally, the sections were turned into a numbered list to be more clear about which section is being described, and improve readability. PR-URL: #4234 Reviewed-By: James M Snell <jasnell@gmail.com>
Add support to fs.createWriteStream and fs.createWriteStream for an autoClose option that behaves similarly to the autoClose option supported by fs.createReadStream and fs.ReadStream. When an instance of fs.createWriteStream created with autoClose === false finishes, it is not destroyed. Its underlying fd is not closed and it is the responsibility of the user to close it. PR-URL: #3679 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Now returns a 417 error status or allows for an event listener on the `checkExpectation` event. Before we were ignoring requests that had misspelled `100-continue` values for expect headers. This is a quick port of the work done here: nodejs/node-v0.x-archive#7132 by alFReD-NSH with surrounding discussion here: nodejs/node-v0.x-archive#4651 Also updates all the instances of the deprecated EventEmitter.listenerCount to the current self.listenerCount. Most of these were in the new code ported over but there was another legacy instance. Refs: #2403 PR-URL: #4501 Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #2205 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
PR-URL: #2205 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
These changes improve parse() performance from ~11-30% on all of the existing querystring benchmarks. PR-URL: #4675 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
If one were to set NODE_REPL_HISTORY to a string that contains only a
space (" "), then the history file would be created with that name
which can cause problems are certain systems.
PR-URL: #4539
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
|
101bca9 is tagged as major, so it can't be included unless we downgrade it to minor. |
|
@evanlucas It's not terribly important, but the conflict should be pretty easy to resolve? It's just git no-longer seeing the start-end of the patch data. |
|
@evanlucas which tooling did you use to produce this list? The arrow functions commit should be in here again unless you excluded it? |
|
I used branch-diff for this |
|
OH, looks like Rod landed a bunch already: https://github.com/nodejs/node/commits/v5.x |
|
ok, so should I just use changelog-maker for the list and strip out the ones that shouldn't land on v5.x? |
|
@evanlucas I would try running (for the proposal message only):
I think that should get the list that Rod did plus yours, if everything works properly. |
|
Ok, that looks better:
|
Provide means to inspect information about the separate heap spaces via a callable API. This is helpful to analyze memory issues. Fixes: #2079 PR-URL: #4463 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Trevor Norris <trev.norris@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com>
Make the byteLength work correctly when input is Buffer. e.g: ```js // The incomplete unicode string Buffer.byteLength(new Buffer([0xe4, 0xb8, 0xad, 0xe6, 0x96])) ``` The old output: 9 The new output: 5 PR-URL: #4738 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
|
@rvagg ok got it. Thanks! Updated the OP with those two entries. Will go through to make sure there aren't any others that have been missed. |
|
@evanlucas You're going to need to hold the release tomorrow until a fix for #4751 lands. It should be simple though so no biggie. Better to fix this regression now than wait. Edit: PR at #4757 |
|
@Fishrock123 agreed |
|
#4762 is done, you'll have to pull it into the proposal branch before the release commit of course |
|
Whoops...accidentally marked to rebase... new CI: https://ci.nodejs.org/job/node-test-pull-request/1325/ |
Reduce the number of stat() system calls that require() makes by caching the results more aggressively. To avoid unbounded growth without implementing a LRU cache, scope the cache to the lifetime of the first call to require(). Recursive calls (i.e. require() calls in the included code) transparently profit from the cache. The benchmarked application is the loopback-sample-app[0] and it sees the number of stat calls at start-up go down by 40%, from 4736 to 2810. [0] https://github.com/strongloop/loopback-sample-app PR-URL: #4575 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Ref: http://eslint.org/docs/rules/space-in-parens.html PR-URL: #4753 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com>
PR-URL: #4753 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com>
Previously, test-cluster-disconnect-leak had two issues: * Magic numbers: How many times to spawn a worker was determined through empirical experimentation. This means that as new platforms and new CPU/RAM configurations are tested, the magic numbers require more and more refinement. This brings us to... * Non-determinism: The test *seems* to fail all the time when the bug it tests for is present, but it's really a judgment based on sampling. "Oh, with 8 workers per CPU, it fails about 80% of the time. Let's try 16..." This revised version of the test takes a different approach. The fix for the bug that the test was written for means that the `disconnect` event will fire reliably for a single worker. So we check for that and the test still fails when the fix is not in the code base and succeeds when it is. Advantages of this approach include: * The test runs much faster. * The test now works on Windows. The previous version skipped Windows. * The test should be reliable on any new platform regardless of CPU and RAM. Ref: #4674 PR-URL: #4736 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
ReadableState has the resumeScheduled property that helps determine if a stream should be resumed. It was not assigned in the constructor. When stream.resume is called on a readable stream that is not flowing, it is set to true. This changes the property map of the ReadableState which can cause a deopt in onEofChunk and needMoreData. PR-URL: #4761 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Brian White <mscdex@mscdex.net>
PR-URL: #4762 Reviewed-By: Rod Vagg <rod@vagg.org>
Modify tools/license-builder.sh to restore the Third-Party Software
licenses for ICU.
Also fix arguments to tail to work on Linux.
rvagg: modified sed command for ICU to replace tabs with spaces and
remove whitespace at the end of lines
PR-URL: #4762
Reviewed-By: Rod Vagg <rod@vagg.org>
Fix a regression introduced in commit 89f056b ("node: improve performance of hrtime()") where the nanosecond field sometimes had a negative value when calculating the difference between two timestamps. Fixes: #4751 PR-URL: #4757 Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Trevor Norris <trev.norris@gmail.com> Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Notable changes: * events: make sure console functions exist (Dave) #4479 * fs: add autoClose option to fs.createWriteStream (Saquib) #3679 * http: improves expect header handling (Daniel Sellers) #4501 * node: allow preload modules with -i (Evan Lucas) #4696 * v8,src: expose statistics about heap spaces (`v8.getHeapSpaceStatistics()`) (Ben Ripkens) #4463 * Minor performance improvements: - lib: Use arrow functions instead of bind where possible (Minwoo Jung) #3622 - module: cache stat() results more aggressively (Ben Noordhuis) #4575 - querystring: improve parse() performance (Brian White) #4675 PR-URL: #4742
PR-URL: #4742
evanlucas
force-pushed
the
v5.5.0-proposal
branch
from
98df288
to
2d46ea0
Compare
|
Had one arm test fail. One more time for good measure. Hopefully we are green this time |
|
Alright...release builds are starting https://ci.nodejs.org/job/iojs+release/363/ |
evanlucas
added a commit
that referenced
this pull request
Jan 21, 2016
Notable changes: * events: make sure console functions exist (Dave) #4479 * fs: add autoClose option to fs.createWriteStream (Saquib) #3679 * http: improves expect header handling (Daniel Sellers) #4501 * node: allow preload modules with -i (Evan Lucas) #4696 * v8,src: expose statistics about heap spaces (`v8.getHeapSpaceStatistics()`) (Ben Ripkens) #4463 * Minor performance improvements: - lib: Use arrow functions instead of bind where possible (Minwoo Jung) #3622 - module: cache stat() results more aggressively (Ben Noordhuis) #4575 - querystring: improve parse() performance (Brian White) #4675 PR-URL: #4742
bripkens
added a commit
to bripkens/docker-node
that referenced
this pull request
Jan 21, 2016
Related: nodejs/node#4742
scovetta
pushed a commit
to scovetta/node
that referenced
this pull request
Apr 2, 2016
Notable changes: * events: make sure console functions exist (Dave) nodejs#4479 * fs: add autoClose option to fs.createWriteStream (Saquib) nodejs#3679 * http: improves expect header handling (Daniel Sellers) nodejs#4501 * node: allow preload modules with -i (Evan Lucas) nodejs#4696 * v8,src: expose statistics about heap spaces (`v8.getHeapSpaceStatistics()`) (Ben Ripkens) nodejs#4463 * Minor performance improvements: - lib: Use arrow functions instead of bind where possible (Minwoo Jung) nodejs#3622 - module: cache stat() results more aggressively (Ben Noordhuis) nodejs#4575 - querystring: improve parse() performance (Brian White) nodejs#4675 PR-URL: nodejs#4742
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#2016-01-18, Version 5.5.0 Release
Looking to release by Tuesday, Jan 19
Notable Changes
v8.getHeapSpaceStatistics()) (Ben Ripkens) #4463Commits
8d0ca10752] - buffer: make byteLength work with Buffer correctly (Jackson Tian) #473883d2b7707e] - buffer: remove unnecessary TODO comments (Peter Geiss) #47198182ec094d] - build: add option to select VS version (julien.waechter) #46454383acd9f4] - build: fix and refactor VTune config in vcbuild.bat (Rod Vagg) #4192be0b0b8cb9] - build: minor corrections in VTune configure text (Rod Vagg) #41929571be12f6] - cluster: fix race condition setting suicide prop (Santiago Gimeno) #4349ebd9addcd1] - crypto: clear error stack in ECDH::Initialize (Fedor Indutny) #468966b9c0d8bd] - debugger: remove variable redeclarations (Rich Trott) #463388b2889679] - dgram: prevent disabled optimization of bind() (Brian White) #4613d56e3f8b67] - doc: restore ICU third-party software licenses (Richard Lau) #4762212a44df03] - doc: clarify protocol default in http.request() (cjihrig) #47143297036345] - doc: update branch-diff arguments in release doc (Rod Vagg) #4691666c089e68] - doc: fix named anchors in addons.markdown and http.markdown (Michael Theriot) #4708310530b7ec] - doc: add path property to Write/ReadStream in fs.markdown (Claudio Rodriguez) #43683470574cb6] - doc: clarify explanation of first stream section (Vitor Cortez) #4234d91646b9c7] - doc: rebuild LICENSE using tools/license-builder.sh (Rod Vagg) #4194265e2f557b] - doc: fix typo in doc/node.1 (Jérémy Lal) #46804c132fe61e] - doc: make references clickable (Roman Klauke) #4654d139704ff7] - doc: improve child_process.execFile() code example (Ryan Sobol) #4504eeb6fdcd0f] - doc: add docs for more stream options (zoubin) #4639b6ab6d2de5] - doc: add branch-diff example to releases.md (Myles Borins) #4636287325c5e8] - docs: update gpg key for Myles Borins (Myles Borins) #465765825b79aa] - docs: fix npm command in releases.md (Myles Borins) #4656f9a59c1d3b] - (SEMVER-MINOR) events: make sure console functions exist (Dave) #44796039a7c1b5] - (SEMVER-MINOR) fs: add autoClose option to fs.createWriteStream (Saquib) #3679ed55169834] - gitignore: never ignore debug module (Michaël Zasso) #2286d755432fa9] - (SEMVER-MINOR) http: improves expect header handling (Daniel Sellers) #45017ce0e04f44] - lib: fix style issues after eslint update (Michaël Zasso) nodejs/io.js#2286ae5bcf9528] - lib: use arrow functions instead of bind (Minwoo Jung) #36220ec093cd41] - lib,test: remove extra semicolons (Michaël Zasso) #2205d8f5bd4fe1] - module: avoid ArgumentsAdaptorTrampoline frame (Ben Noordhuis) #457583f8d98806] - module: cache stat() results more aggressively (Ben Noordhuis) #4575ff64a4c395] - (SEMVER-MINOR) node: allow preload modules with -i (Evan Lucas) #46964bc1a47761] - querystring: improve parse() performance (Brian White) #4675ad63d350d4] - readline: Remove XXX and output debuglog (Kohei TAKATA) #4690da550aa063] - repl: make sure historyPath is trimmed (Evan Lucas) #45391a6e7d1b52] - src: fix negative values in process.hrtime() (Ben Noordhuis) #47578bad51977a] - src: return UV_EAI_NODATA on empty lookup (cjihrig) #4715761cf2bf6a] - src: don't check failure with ERR_peek_error() (Ben Noordhuis) #4731953f4a3999] - stream: prevent object map change in ReadableState (Evan Lucas) #4761e65f1f7954] - test: fix tls-multi-key race condition (Santiago Gimeno) #39663727ae0d7d] - test: use addon.md block headings as test dir names (Rod Vagg) #441247960a07c0] - test: make test-cluster-disconnect-leak reliable (Rich Trott) #47369926b5a25f] - test: fix issues for space-in-parens ESLint rule (Roman Reiss) #4753d1aabd6264] - test: fix style issues after eslint update (Michaël Zasso) nodejs/io.js#2286e98bcfa2cb] - test: remove 1 second delay from test (Rich Trott) #46166cfd0b5a32] - test: fix flaky test-net-socket-local-address (cjihrig) #4650e22cc6c2eb] - test: fix race in test-net-server-pause-on-connect (Rich Trott) #46379164c00bdb] - test: move resource intensive tests to sequential (Rich Trott) #4615d8ba2c0de4] - test: fixhttp-upgrade-clientflakiness (Santiago Gimeno) #46026018fa1f57] - test: fixhttp-upgrade-agentflakiness (Santiago Gimeno) #4520c33f6a87d0] - tools: enable space-in-parens ESLint rule (Roman Reiss) #4753162e16afdb] - tools: enable no-extra-semi rule in eslint (Michaël Zasso) #2205031b87d42d] - tools: add license-builder.sh to construct LICENSE (Rod Vagg) #4194ec8e0ae697] - tools: fix style issue after eslint update (Michaël Zasso) nodejs/io.js#22864d5ee7a512] - tools: update eslint config (Michaël Zasso) nodejs/io.js#22862d441493a4] - tools: update eslint to v1.10.3 (Michaël Zasso) nodejs/io.js#2286aba3cc834e] - tools: fix license-builder.sh for ICU (Richard Lau) #47625f57005ec9] - (SEMVER-MINOR) v8,src: expose statistics about heap spaces (Ben Ripkens) #4463