动态列表分页查询中的死循环

初学写代码的同学容易写出死循环的代码,想不到我们多年的老鸟也栽在了死循环上面。有个查询solr的服务死循环了,导致了solr cpu狂涨。查了许久才问题原因来。

一,背景

有一个Rails项目中有一个服务原来查询数据库的,存在一个很难优化的慢查询,于是想迁移到使用solr查询。修改之后,当天中午上线,到1点左右,运维就发现solr的CPU在狂涨。于是马上想到可能是刚刚上线的查询可能有问题。分析了一下刚刚上线的查询业务,solr中保存的是商品的各种信息,刚刚上线的功能是要查询中所有符合某种条件的商品。分析了一下代码,发现可能是符合条件的商品数量较多,而实际可能使用不了这么多商品,于是想到限制一下查询的总量,即查询到大约5W个商品的时候就结束。然后上线,但情况依旧,又经过了几小时的排查,才发现第二次上线时,使用passenger重启服务失败了,于是找运维手工kill掉旧进程,启动了新进程,问题解决!

上面是遇到的情况以及排查的大概过程。但有两个事情无法解释:

  1. 这个查询业务,查询条件并不算复杂,怎么会导致solr的cpu使用量狂涨?
  2. Rails进程为什么会重启失败?

二,问题原因

排查还遇到一个奇怪的现象,正是这个现象成为了抓住原凶的突破口。我们让运维导出了solr的访问日志,发现solr查询中存在一个极大的offset,要查询的是满足某种条件的所有商品,目前线上满足这个条件的商品数量大概有15w左右,但从日志中发现有高达300w以上的offset,这远远大于我们需要查询的数据量。于是开始怀疑这个查询存在死循。分析了代码之后,果然是存在这种可能的。

查询solr的代码,简化之后大概为这样:

def get_all_product_id
  page = 1
  ids  = []

  begin
    result = solr_query(page: page, per: 500)
    ids   += result.ids

    ids.uniq!
    page  += 1
  end while result.total > ids.size

  ids
end

上面代码中solr_query代表一次solr的查询,代码中一次查询最多500个数据,返回的result中,ids代表本次查询返回的id,result.total代表solr中所有满足条件的商品总数。

这段代码在线下运行完全没有问题,可以正常的查询回数据,不存在死循环。但在线上,solr的offset却一直涨到比要查询的数据量大很多。线上和线下solr的区别在于,线上的solr另外存在一个更新的任务,不断的在更新solr中的数据,这可能是造成循环停不下来的原因。

下面模拟了一下可能出现的情况,假如有一个列表,以下图所示,蓝色背景的是我们需要查询出来的数据,白色背景的是不满足条件的数据,注意第2个数据现在是不满足查询条件的。



假如我们按每页3个元素,那么第一页查询,我们将会获得元素[1,3,4],然而这个列表可能是会动态变化的,当我们查询第二页的时候,列表可能变成了下面这样:

此时,第2个元素又满足条件了,此时查询到的第2页数据将会是:[4,5,6],我们可以看到2被漏掉了,而且重复出现的元素4,然后观察上面的代码,发现我们对id去了重ids.uniq!,此时可以发现,我们查询了所有满足条件的元素一共只有5个(1,3,4,5,6),而实际满足条件的total数量为6,所以上面代码中的停止循环的条件永远达不到,于是代码死循环了!

三,解决方案

其实上面的问题,在查询数据库时,也是可能遇到的,上面导致死循环的一个重要的原因是对查出的数据去了重,这导致查询的数量可能永远达不到total的量,所以循环不会停下来。但也可以从业务上来优化一下,也许我们使用不了这么多数据,当查询到一定量之后,就停止查询。或者需要设定一个强制退出的条件,比如查询次数超过一定值时,强制退出来。

然后大概也可以解释为什么rails进程重启失败了,进程中存在死循环的时候,passenger在重启的时候会认为进程的请求还未处理完,进程为因此hang住。另外,其实rails项目多是采用多进程方式部署,实际的进程数量是非常有限,一旦有死循环存在,很容易让整个系统崩溃。

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 213,558评论 6 492
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,002评论 3 387
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 159,036评论 0 349
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,024评论 1 285
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,144评论 6 385
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,255评论 1 292
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,295评论 3 412
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,068评论 0 268
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,478评论 1 305
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,789评论 2 327
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,965评论 1 341
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,649评论 4 336
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,267评论 3 318
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,982评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,223评论 1 267
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,800评论 2 365
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,847评论 2 351

推荐阅读更多精彩内容

  • 包含的重点内容:JAVA基础JVM 知识开源框架知识操作系统多线程TCP 与 HTTP架构设计与分布式算法数据库知...
    消失er阅读 4,310评论 1 10
  • 1. 什么是solr    Solr 是Apache下的一个顶级开源项目,采用Java开发,它是基于Lucene的...
    东方舵手阅读 940评论 1 5
  • 经典文章大家分享 酒文化的历史渊源 如果不喝酒 刘关张在桃园聊完天就离开, 三国演义,剧终。 如果不喝酒, 孙悟空...
    为真常说阅读 133评论 0 0
  • 前一段时间,进行了一点漫无目的的日更,效果不是很大。 最近读了《刻意练习》和一些干货文章总结了几种有效的日更练习方...
    客舟听雨_e8bc阅读 235评论 2 2
  • 我希望只我一个人的时间很长, 不用交际,不用说些没用的话。 处处遵循自己的内心, 没心没肺习惯了,就不想谈论所谓的...
    南溪有荷阅读 262评论 0 3