2017-04-13 IRC 会议总结
概述
笔记 / 简短主题
简要讨论了 Ryanofsky 的多进程更改,它将分离 QT 和 bitcoind。虽然将钱包与网络分离更为重要,但 GUI 存在更多紧迫的问题,因为 GUI 循环中过多操作与核心同步执行。相关 PR #10102 尚未完成,但已有的代码可以进行一些审查。
主要主题
- 0.14.1
- 脚本化差异
- 0.15 目标
- 高优先级审查
0.14.1
背景
比特币核心 0.14.1 包含错误修复和优化。4 月 8 日发布了候选发布版本 1 (RC1)。RC2 已于 4 月 17 日(会议后几天)发布。
会议评论
没有关于 RC1 中错误的报告。Cfields 和 BlueMatt 仍然希望在 0.14.1 中加入 PR #10176(优雅地处理 NodeId 溢出),Gmaxwell 和 Wumpus 则更倾向于不延迟发布,也不认为这需要另一个 RC 版本。
会议结论
- 会议后决定,但 0.14.1 推迟发布,并发布了 RC2。
脚本化差异
背景
CFields 提出更改,在提交消息中添加一个验证脚本,该脚本可以由 Travis 验证。此验证脚本可用于简单的操作,例如搜索和替换,这会产生大量差异,但脚本很简单。如果脚本未能将旧提交内容完全转换为新的提交内容,它将被 Travis 拒绝。这将使这些具有大量差异的简单更改更容易审查。
拉取请求 #10193 是一个示例,展示了脚本化差异提交应该是什么样子的。
会议评论
Luke-jr 认为我们不应该信任 Travis,因为它会给人一种虚假的审查感。Gmaxwell 指出,审阅者可以在审查脚本后进行测试,Travis 的目的仅仅是提供反馈,说明你是否正确地格式化了代码,以及它是否在每台计算机上都能运行,而不是作为审查步骤。
Gmaxwell 认为只要我们不指望提交者运行它,这就可以了。
Wumpus 认为这有点危险,因为它实际上会执行任意脚本。Cfields 指出,BlueMatt 之前提到了这一点,并认为可能值得讨论将脚本限制为 "sed",它只进行替换。Cfields 还澄清,该脚本只在 Travis 上自动运行,其他地方都不会运行。
Jtimon 建议,只有在提交标题中有脚本化前缀时才运行脚本,这样可以更容易地看出将要执行脚本,因为大多数审阅者审阅的是代码,而不是提交消息。
会议结论
- 审查 PR #10189(为可脚本化更改添加验证器)
0.15 目标
背景
比特币核心 0.15.0 预计于 2017 年 9 月左右发布。针对 0.15 的拉取请求已 计划 发布,并已 标记为 0.15.0 标签。
Sdaftuar 希望知道每个人对 0.15 的目标是什么,这样其他人就知道哪些应该优先审查。
会议评论
Gmaxwell 希望看到每个 TXO 的 dbcache 和非原子性刷新。Cfields 想知道从 0.15 降级到 0.14 时,每个 TXO 的预期结果是什么。Gmaxwell 澄清,它无法降级,需要重新索引。可能值得在 0.14.2 中添加一些内容,在尝试降级时,优雅地指出格式已更改,这比通用的损坏消息更好。
Jonasschnelli 的目标是:HD 自动恢复、QT 手续费计算器、多钱包和 HD 只观察钱包。
Sdaftuar 希望新手续费估计能够到位。Gmaxwell 认为我们需要对该算法进行高级描述,以便我们可以将其提供给非开发人员(学者)进行审查,这也有助于一般审查,因为它很容易忽略估计器的总体设计。Morcos 意识到,这对于微不足道的感知收益来说是一件非常痛苦的事情,但他确实认为这是值得的。BlueMatt补充说,他认为这并非微不足道的感知收益,因为在钱包开发者会议上讨论的一个主题是整个生态系统的手续费估计有多糟糕,而比特币核心是其中很大的一部分。新的估计器极大地改善了周末期间的手续费估计。
BlueMatt 将致力于多线程 net_processing(和钱包),并使用 CValidationInterface 生成回调。
Sipa 希望看到每个 TXO 的 dbcache、消除刷新时的内存峰值以及更好的 dbcache 驱逐策略。
高优先级审查
BlueMatt 希望添加 PR #10179(让 CValidationInterface 支持在 CScheduler 线程上调用通知),因为它和 #10178 为他将钱包回调移至后台线程的 0.15 目标铺平了道路。
Sipa 希望添加 PR #9792(FastRandomContext 改进并切换到 ChaCha20)
Morcos 认为 #9942(重构 CBlockPolicyEstimator)可以合并,这将使其余的手续费估计更改更小,更容易审查。
Jnewbery 希望对 #10044(在 "make check" 中运行功能测试)进行一些审查。
轻松一刻
BlueMatt because its free, we're already doing 0.14.1 and delaying 1 week isnt gonna kill us
BlueMatt_ But delaying 1 week isn't too bad
BlueMatt wait, who is BlueMatt_ ?
wumpus confused
BlueMatt_ confused
BlueMatt has no idea who BlueMatt_ is
BlueMatt_ has no idea who BlueMatt is
kanzure different timeline, carry on.
luke-jr whois says it's Matt Corallo
BlueMatt not me
gmaxwell wumpus: shoot the T1000 (BlueMatt_) and lets move on.
sipa BlueMatt_: this statement is false
spudowiar You could create format like '*.cpp *.h | s/boost::filesystem/fs/g'
sipa spudowiar: little bobby tables will haunt you
sipa I want pertxoutcache, remove memory peak at flushing, better dbcache eviction policy, ...
sipa oh, and segwit activated? pretty please?
BlueMatt sipa: lol
cfields_ sipa: let's activate segwit after the meeting. We only have 20 min left :p
gmaxwell cfields_: ack
wumpus #action activate segwit
gmaxwell jinx
参与者
IRC 昵称 | 姓名 / 匿名 |
---|---|
wumpus | Wladimir van der Laan |
jonasschnelli | Jonas Schnelli |
sipa | Pieter Wuille |
cfields | Cory Fields |
sdaftuar | Suhas Daftuar |
morcos | Alex Morcos |
luke-jr | Luke Dashjr |
jcorgan | Johnathan Corgan |
CodeShark | Eric Lombrozo |
kanzure | Bryan Bishop |
gmaxwell | Gregory Maxwell |
BlueMatt | Matt Corallo |
BlueMatt_ | T-1000 高级原型(拟态聚合金) |
spudowiar | Saleem Rashid |
petertodd | Peter Todd |
jnewbery | John Newbery |
ryanofsky | Russell Yanofsky |
instagibbs | Gregory Sanders |
phantomcircuit | Patrick Strateman |
MarcoFalke | Marco Falke |
achow101 | Andrew Chow |
免责声明
本总结是在未征求任何讨论参与者意见的情况下编写的,因此任何错误都是总结作者的责任,而非讨论参与者的责任。