2016-11-17 IRC 会议总结
概述
主要议题
- Shared_ptr
- 优先级移除
- 账户移除
- 辅助区块请求
Shared_ptr
背景
在许多地方,我们开始使用 'shared_ptr' 代替对象本身,这样它就可以在许多数据结构之间共享,而无需进行复制。
会议评论
Sipa 有 3 个 pull request 来在更多地方引入 shared_ptr,分别是 #9125 (将 CBlock 更改为 CTransaction 的 shared_ptr 的向量),#8580(使 CTransaction 实际上不可变) 和 #8589 (在 #8580 的基础上,将 CTxInWitness 内联到 CTxIn 中)。第一个是后面两个的必要重构,并且性能提高了 3-4%。第二个可能会更具争议性,因为它会显著影响钱包代码。Wumpus 认为 CwalletTx 继承自 CTransaction 的旧行为是滥用继承的一个很好的例子。
会议结论
- 审阅 #9125
优先级移除
背景
优先级系统以前是根据年龄、大小和输入数量为交易分配优先级,这使得一些交易免费。它具有庞大的代码库,并且一直在努力移除该系统,因为无法期望矿工持续包含 0 费率交易。
会议评论
Morcos 指出优先级代码不再发挥任何作用,也许可以设定一个目标,在 0.15 版本中移除所有优先级代码。Luke-Jr 可能不同意,尽管他不在会议上。已经合并了许多关于优先级的代码。
Gmaxwell 同意,并认为通过明智地使用费用增量可以满足对保留优先级的任何需求。
“移除优先级”有 4 个部分:rpc、估计、挖矿和中继。估计已经移除。Gmaxwell 希望看到中继移除,因为它目前导致带宽浪费,因为它在中继不会被挖出的交易。
会议结论
- 更改默认值为禁用优先级中继。
- 与 Luke-Jr 重新讨论,因为他是在会议上没有参与的保留优先级的坚定支持者。
账户移除
背景
目前,比特币核心使用一个账户系统。使用它的第三方可能会遇到多个问题,并且已经长期共识该系统应该被移除,并由 '标签' 系统取代。
会议评论
今年早些时候,Wumpus 开启了一个pull request,为钱包引入 '标签' API(之前在2016-07-14 会议中提到)。它仍然没有得到太多审阅。
在移除账户之前,应该先为标签系统发布一两个版本,因为有些人仍然依赖于账户系统,但将其用作标签。
Wumpus 提到,应该有一个防止用户使用账户和标签 API 的保护措施,因为这可能会导致问题。
Instagibbs 想知道是否有任何人与仍然使用账户的开发人员交谈。MarcoFalke 认为 Dooglus 使用它,但他的用例将被新的标签 API 覆盖。
会议结论
- 审阅 #7729 (rpc: 为钱包引入 '标签' API)
- ping dooglus 获取用户反馈。
辅助区块请求
背景
Jonasschnelli 开启了一个pull request,引入辅助区块请求(以前是“带外区块请求”。这个功能允许在磁盘上可用时请求区块,如果不可用,则会下载并优先于正常的 IBD 下载。这个更改需要运行 SPV 钱包,这在上周会议中提到。
会议评论
多个开发人员发现难以理解,并想知道是否有更一般的关于高级设计及其工作原理的描述。Jonasschnelli 解释道:你向你的节点发出请求,给我区块“D、F、G”,节点下载区块“D、F、G”并将其通过带有 validate=false 的信号传递。
Sipa 喜欢总体概念,但认为实现需要随着正在进行的网络重构而改变。
BlueMatt 质疑区块是否真的应该存储,或者我们是否可以将其直接传递给钱包。如果它不存储区块,则需要在混合模式下下载两次,因此它至少应该有机会存储它。BlueMatt 希望看到 p2p 逻辑决定是否将区块发送给 ProcessNewBlock。
幽默
morcos Lets talk about potential for account or priority removal (2 separate topics)
jonasschnelli #topic account or priority removal
jonasschnelli #topic "account removal"
gmaxwell "Bitcoin developers oppose accountability."
jtimon can't we replace accounts with labels all at once?
jtimon it's not like we haven't been warning against the use of accounts for ages
instagibbs jtimon, at some point I don't think people believe the deprecated warning
instagibbs we should put scary ascii art :)
gmaxwell okay, I think we should take the proposed action of everyone reading and commenting on 7729 and move to another subject.
gmaxwell Or otherwise, we could instead engage in the age old art of completely uninformed combat.
gmaxwell "I haven't read 7729 but I oppose any change that causes blindness in small children!"
petertodd gmaxwell: I didn't read your last comment, but ACK
jonasschnelli If there are no other topic, we could talk about "auxiliary block requests" if some are interested in it?
jtimon what is that?
gmaxwell jonasschnelli: will that cause blindness in small children?
参与者
IRC 昵称 | 姓名/匿名 |
---|---|
sipa | Pieter Wuille |
gmaxwell | Gregory Maxwell |
wumpus | Wladimir van der Laan |
morcos | Alex Morcos |
jtimon | Jorge Timón |
BlueMatt | Matt Corallo |
Chris_Stewart_5 | Chris Stewart |
jonasschnelli | Jonas Schnelli |
Michagogo | Michagogo |
achow101 | Andrew Chow |
cfields | Cory Fields |
MarcoFalke | Marco Falke |
CodeShark | Eric Lombrozo |
jcorgan | Johnathan Corgan |
petertodd | Peter Todd |
instagibbs | Gregory Sanders |
免责声明
此总结是在未征求讨论中任何参与者的意见的情况下编写的,因此任何错误都是总结作者的责任,而不是讨论参与者的责任。