|
| 1 | +# Node Foundation CTC Meeting 2017-03-29 |
| 2 | + |
| 3 | +## Links |
| 4 | + |
| 5 | +* **Audio recording**: https://www.youtube.com/watch?v=NzzRR9e0uPM |
| 6 | +* **GitHub Issue**: [CTC#85](https://github.com/nodejs/CTC/issues/85) |
| 7 | +* **Minutes Google Doc**: <https://docs.google.com/document/d/1nIXl2UMQs1n-yPencn53MYlfu3RnUltX3MfuaStPzx8> |
| 8 | +* _Previous Minutes Google Doc: <https://docs.google.com/document/d/1ZbzN6qO9AqmGjOMR-wfWTmdeE_69rr1hcfhk2oTUGpA>_ |
| 9 | + |
| 10 | +## Present |
| 11 | + |
| 12 | +* Anna Henningsen @addaleax (CTC) |
| 13 | +* Bradley Meck @bmeck (observer/GoDaddy/TC39) |
| 14 | +* Сковорода Никита Андреевич @ChALkeR (CTC) |
| 15 | +* Colin Ihrig @cjihrig (CTC) |
| 16 | +* Evan Lucas @evanlucas (CTC) |
| 17 | +* Franziska Hinkelmann @fhinkel (CTC) |
| 18 | +* Jeremiah Senkpiel @Fishrock123 (CTC) |
| 19 | +* James M Snell @jasnell (CTC) |
| 20 | +* Michael Dawson @mhdawson (CTC) |
| 21 | +* Brian White @mscdex (CTC) |
| 22 | +* Ali Ijaz Sheikh @ofrobots (CTC) |
| 23 | +* Seth Thompson @s3ththompson (observer/Google) |
| 24 | +* MichaëZasso @targos (CTC) |
| 25 | +* Trevor Norris @trevnorris (CTC) |
| 26 | +* Rich Trott @Trott (CTC) |
| 27 | + |
| 28 | + |
| 29 | +## Standup |
| 30 | + |
| 31 | +* Anna Henningsen @addaleax (CTC) |
| 32 | + * Issues & PR review |
| 33 | + * ABI-stable native module API review in particular |
| 34 | +* Bradley Meck @bmeck (observer/GoDaddy/TC39) |
| 35 | + * CC port for ESM underway |
| 36 | + * onboarded Guy Bedford from SystemJS to help |
| 37 | +* Сковорода Никита Андреевич @ChALkeR (CTC) |
| 38 | + * Mostly issue and PR cNothing worth mentioning |
| 39 | +* Colin Ihrig @cjihrig (CTC) |
| 40 | + * Issues and PRs. |
| 41 | +* Evan Lucas @evanlucas (CTC) |
| 42 | + * nothing of note |
| 43 | +* Franziska Hinkelmann @fhinkel (CTC) |
| 44 | + * PR backlog |
| 45 | +* Jeremiah Senkpiel @Fishrock123 (CTC) |
| 46 | + * Promise Unhandled rejection GC new pr thing https://github.com/nodejs/node/pull/12010 |
| 47 | +* James M Snell @jasnell (CTC) |
| 48 | + * landing PRs getting ready for 8.0.0 |
| 49 | +* Michael Dawson @mhdawson (CTC) |
| 50 | + * Away last week at Devoxx conference |
| 51 | + * Build WG meeting |
| 52 | + * AIX/disk speed discussions |
| 53 | + * N-API review/work |
| 54 | + * foundation npm user |
| 55 | + * misc review/comment/land |
| 56 | +* Brian White @mscdex (CTC) |
| 57 | + * _Finally_ finished next set of `fs` optimizations |
| 58 | + * Updated/Rebased own open PRs |
| 59 | + * Commented on issues, reviewed PRs |
| 60 | +* Ali Ijaz Sheikh @ofrobots (CTC) |
| 61 | + * not too much of note |
| 62 | +* Seth Thompson @s3ththompson (observer/Google) |
| 63 | + * v8 team working on landing TurboFan + Ignition in V8 5.9 |
| 64 | +* MichaëZasso @targos (CTC) |
| 65 | + * Nothing of note |
| 66 | +* Trevor Norris @trevnorris (CTC) |
| 67 | + * asynchooks PR |
| 68 | +* Rich Trott @Trott (CTC) |
| 69 | + * lots of little things, but let's talk about Buffer right now... |
| 70 | + |
| 71 | +## Agenda |
| 72 | + |
| 73 | +### nodejs/node |
| 74 | + |
| 75 | +* buffer: auto random fill Buffer(num) and new Buffer(num) [#11808](https://github.com/nodejs/node/pull/11808) |
| 76 | +* buffer: discuss future direction of Buffer constructor API [#9531](https://github.com/nodejs/node/issues/9531) |
| 77 | + |
| 78 | +## Previous Meeting Review |
| 79 | + |
| 80 | +* buffer: discuss future direction of Buffer constructor API [node#9531](https://github.com/nodejs/node/issues/9531) |
| 81 | +* module: add support for abi stable module API [node#11975](https://github.com/nodejs/node/issues/11975) |
| 82 | + |
| 83 | +## Minutes |
| 84 | + |
| 85 | +### buffer: auto random fill Buffer(num) and new Buffer(num) [#11808](https://github.com/nodejs/node/pull/11808) and buffer: discuss future direction of Buffer constructor API [#9531](https://github.com/nodejs/node/issues/9531) |
| 86 | + |
| 87 | +* Rich started by framing the current state of the issue. Some |
| 88 | + consensus that we should do something. Had looked like |
| 89 | + option to do opt-in deprecate flag was non-controversial, but |
| 90 | + seems like there is some questioning/objections. |
| 91 | +* We may need to start with if we are going to state that it |
| 92 | + will be deprecated at some point as that affects people's |
| 93 | + decisions on the other issues. |
| 94 | +* James: we need 3 distinct votes |
| 95 | + * commit to runtime deprecation and timeframe |
| 96 | + * do we put in pending deprecation |
| 97 | + * do we random fill or zero fill by default, and do we backport |
| 98 | +* Let's start with are we going to commit to deprecate or not |
| 99 | + * James, feedback from module developers, don't do it, zero fill |
| 100 | + by default instead, most people did not think it should be |
| 101 | + Backported. |
| 102 | + * Ali, spreadsheet before option of additional flag ? does this |
| 103 | + affect how people voted, |
| 104 | + * James, don't think we are going to reach natural consensus, |
| 105 | + need vote. Seconded by Rich |
| 106 | + |
| 107 | + * Vote1: Should we now commit to deprecating Buffer() constructor |
| 108 | + at a specific point in the future or leave it as is. |
| 109 | + Specifically, deprecating all combinations of Buffer() |
| 110 | + constructor that are not specifically required by |
| 111 | + subclassing Uint8Array. |
| 112 | + Specific point might mean a specific version or when the |
| 113 | + ecosystem usage drops below threshold, or something else |
| 114 | + (a separate question). I.e. are we going to announce now that |
| 115 | + this is going to be runtime-deprecated at some future point, |
| 116 | + possibly without defining that future point now. |
| 117 | + |
| 118 | + **YES** (**Commit** to deprecating): @chalker, @targos, @mscdex, @fhinkel |
| 119 | + |
| 120 | + **NO** (Do not **commit** to deprecating): @evanlucas, @addaleax, @jasnell, |
| 121 | + @ofrobots, @Fishrock123, @mhdawson, @cjihrig |
| 122 | + |
| 123 | + **ABSTAIN**: @trott, @trevnorris |
| 124 | + |
| 125 | + Need to take back to github, to get additional CTC members to vote. |
| 126 | + |
| 127 | + * Vote2: Do we do a conditional pending deprecation warning. |
| 128 | + Normal deprecation warning which will only be seen if they opt in with |
| 129 | + a flag. |
| 130 | + |
| 131 | + **YES**: @chalker, @ofrobots, @targos, @trott, @addaleax, @mhdawson, |
| 132 | + @jasnell |
| 133 | + |
| 134 | + **NO**: @cjihrig, @evanlucas, @mscdex, @fhinkel |
| 135 | + |
| 136 | + **ABSTAIN**: @fishrock123, @trevnorris |
| 137 | + |
| 138 | + Take back to github and get additional votes. |
| 139 | + |
| 140 | + * Vote3: Do we switch to automatically fill for 8.0 and higher. |
| 141 | + |
| 142 | + **Zero**: @jasnell, @fishrock123, @ofrobots, @trevnorris, @mhdawson, |
| 143 | + @targos, @addaleax, @evanlucas, @cjihrig |
| 144 | + |
| 145 | + **Random**: @addaleax, @cjihrig, @targos, @fhinkel |
| 146 | + |
| 147 | + **Neither**: @mscdex, @chalker |
| 148 | + |
| 149 | + **Abstain**: |
| 150 | + |
| 151 | + * Vote 4: If we choose to fill, do we backport (semver-minor) |
| 152 | + |
| 153 | + **ALL**: @fishrock123, @trott, @jasnell |
| 154 | + |
| 155 | + **7**: |
| 156 | + |
| 157 | + **6**: |
| 158 | + |
| 159 | + **4**: |
| 160 | + |
| 161 | + **NONE**: @mscdex, @mhdawson, @evanlucas, @fhinkel |
| 162 | + |
| 163 | + **ABSTAIN**: @targos, @cjihrig, @addaleax, @ofrobots |
| 164 | + |
| 165 | + **ABSTAIN ONLY IF ALL**: @chalker (NO for partial backport) |
| 166 | + |
| 167 | + Take vote back to github to collect rest of votes |
| 168 | + |
| 169 | +## Q&A, Other |
| 170 | + |
| 171 | +* question about what kinds of questions are ok. General questions |
| 172 | + are ok, but those requesting help are best raised in the help repo. |
| 173 | +* question about what would be the best database to use with node. |
| 174 | + Suggestion was to raise in the help repo |
| 175 | + |
| 176 | +## Upcoming Meetings |
| 177 | + |
| 178 | +* **Node.js Foundation Calendar**: https://calendar.google.com/calendar/embed?src=kap.co_i17i575te0aes6kaanfjr2e4hs%40group.calendar.google.com |
| 179 | + |
| 180 | +Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. |
| 181 | + |
| 182 | +* **Collaborators Summit**: May 4-5, |
| 183 | + [summit#39](https://github.com/nodejs/summit/issues/39) |
| 184 | +* **TC39 Meetup**: May 22, |
| 185 | + [CTC#67](https://github.com/nodejs/CTC/issues/67) |
| 186 | +* **Call with V8 team**: March 30, 6 am CEST (in ~11 hours) |
| 187 | + [CTC #76] (https://github.com/nodejs/CTC/issues/76) |
| 188 | +* **CTC**: Apr 5, 2000 UTC |
0 commit comments