当年 1.6 亿美金估值的公司—— Digg 是如何被一句 Python 函数可变默参 毁掉的

[est]

https://lethain.com/digg-v4/

太戏剧性了。画重点：

2011 年，Google 推出「 Panda 」 机制动摇了很多老的 SEO 手段，digg 流量被腰斩。推出 DiggV4 作战计划。经过紧张的开发发布，不过访客页面没问题，已登录用户打不开 MyNews 页面。开发不得不用临时手段把登录用户的默认页面改成 TopNews

MyNews 只能通过不断重启进程才能短暂修复。初期以为是 cassandra 的缓存击穿了 memcache，后来加紧用 redis 重写了，还是得几个小时重启一次

（折腾了一个月之后）

终于发现原因了：API 服务器是 tornado 写的名字叫 Bobtail。里面最常用的函数是：

def get_user_by_ids(ids=[])

然后这个 ids 就一直被 append 直到撑爆内存

所以这个 MyNews 功能也渐渐用的人少，因为没法定制化看新闻，后来，大家都不去 diggv4 而去 reddit 了。。

后来，digg 以 50w 美金被别人收了。。

作为这次 digg v4 事件的受害者，觉得太神奇了。。

[monsterxx03]

@wwqgtxx 对， 忘记说了，3.4 开始不受这个影响了，生产环境还是 2.7

[misaka19000]

我一直不知道原来不能用可变参数作为默认变量，不过每次当我想这样尝试的时候 Pycharm 都会有警告，所以我从来没有真正的这样干过~~现在总算知道是什么原因了

[jimi2018]

哎，程序员很重要啊。

[susucoolsama]

这些坑真的是语言的设计缺陷吧，我觉得，开发者只有避免了。

[hubqin]

默认参数应该改成不变对象`None`：
def add_end(L=None):
if L is None:
L = []
L.append('END')
return L

[ihipop]

用好的 IDE 这种坑都会被 IDE 标出来的。非得用什么 VIM 不装任何语法分析插件什么的，就得保证自己有足够好的基本功底。

[fwee]

闭嘴！这是一个 feature ！！

[Linxing]

@cf472436288 要求啥经验

[noNOno]

可真是值钱的 bug

[risent]

我去， 这锅当年是算在 cassandra 头上的，同一时期很多公司包括 Twitter 也在准备迁移到 cassandra，出了 digg 这档子事后都赶紧拉倒了。
cassandra 背锅这么多年啊！！

[standin000]

reddit 也是 python 写的，不过最开始是 lisp 的

[yongzhong]

@monsterxx03 看过这个工具,但一直不了解在生产环境怎么使用,hardcode 到代码中然后灰度到集群上进行观察?

[kappa]

@risent 貌似当时的新闻说是 CTO 因为选型 Cassandra 引咎辞职？

[monsterxx03]

@yongzhong 最好能先定位有问题的大概代码段, 写个脚本离线 benchmark 下, 完全没头绪就只能 hardcode 到代码里线上测了, 要小心点(比如 10% 的采样执行), 性能影响没测过.

[est]

@kappa reddit 也是 cassandra。。。

[reus]

坑是小坑，但是如果要你在一大堆代码里定位出这个，估计楼上一些冷嘲热讽的人，一个月都不行吧。
这是语言设计上的问题，这个行为是和直觉相悖的。因为每次函数调用，参数都是独立的，大部分人都会有这个直觉，但谁知道默认参数居然是每次调用都共用的呢？
js、ruby 的行为都不是这样，估计 python 这样的，仅此一家了。

[ericgui]

由于一颗钉子输了一场战争，最后覆灭一个王国，

这个事，似乎有点夸大。

[shyangs]

https://www.v2ex.com/t/163431
我在這裡吐槽過, 還被 Pythoneer 說這是特性, 不是坑.

[Ehco1996]

@monsterxx03 我的问题也排查出来了，就是 pyopenssl 的锅

[XuAaron]

@tabris17 这点还好，关键是列表可变。