【发布时间】:2017-05-19 01:47:20
【问题描述】:
我有一个正在尝试减少内存占用的函数。我可以使用的最大内存量只有 500MB。看来使用.split('\t') 和for 循环确实占用了大量内存。有没有办法可以减少这种内存使用量?
Line # Mem usage Increment Line Contents
==============================================
10 35.4 MiB 0.0 MiB @profile
11 def function(username):
12 35.4 MiB 0.0 MiB key = s3_bucket.get_key(username)
13 85.7 MiB 50.2 MiB file_data = key.get_contents_as_string()
14 159.3 MiB 73.6 MiB g = [x for x in file_data.splitlines() if not x.startswith('#')]
15 144.8 MiB -14.5 MiB del file_data
16 451.8 MiB 307.1 MiB data = [x.split('\t') for x in g]
17 384.0 MiB -67.8 MiB del g
18
19 384.0 MiB 0.0 MiB d = []
20 661.7 MiB 277.7 MiB for row in data:
21 661.7 MiB 0.0 MiB d.append({'key': row[0], 'value':row[3]})
22 583.7 MiB -78.0 MiB del data
25 700.8 MiB 117.1 MiB database[username].insert_many(d)
26 700.8 MiB 0.0 MiB return
更新1
根据@Jean-FrançoisFabre 和@Torxed 的建议,这是一个改进,但生成器似乎仍然占用大量内存。
@martineau 我更喜欢使用 MongoDB .insert_many() 来遍历键并执行 .insert() 会慢得多。
20 35.3 MiB 0.0 MiB @profile
21 def function(username):
22 85.4 MiB 50.1 MiB file_data = s3_bucket.get_key(username).get_contents_as_string()
23 610.5 MiB 525.2 MiB data = (x.split('\t') for x in isplitlines(file_data) if not x.startswith('#'))
24 610.5 MiB 0.0 MiB d = ({'key': row[0], 'value':row[3]} for row in data)
25 123.3 MiB -487.2 MiB database[username].insert_many(d)
26 123.3 MiB 0.0 MiB return
UDPATE2
我已经确定了此配置文件显示的内存使用源:
21 41.6 MiB 0.0 MiB @profile
22 def insert_genotypes_into_mongodb(username):
23 91.1 MiB 49.4 MiB file_data = s3_bucket.get_key(username).get_contents_as_string()
24 91.1 MiB 0.0 MiB genotypes = (x for x in isplitlines(file_data) if not x.startswith('#'))
25 91.1 MiB 0.0 MiB d = ({'rsID': row.split('\t')[0], 'genotype':row.split('\t')[3]} for row in genotypes)
26 # snps_database[username].insert_many(d)
27 91.1 MiB 0.0 MiB return
insert_many() 函数清楚地解决了导致整个列表被加载到内存中的前几行并混淆了分析器。
解决方案是将key分块插入MongoDB:
22 41.5 MiB 0.0 MiB @profile
23 def insert_genotypes_into_mongodb(username):
24 91.7 MiB 50.2 MiB file_data = s3_bucket.get_key(username).get_contents_as_string()
25 180.2 MiB 88.6 MiB genotypes = (x for x in isplitlines(file_data) if not x.startswith('#'))
26 180.2 MiB 0.0 MiB d = ({'rsID': row.split('\t')[0], 'genotype':row.split('\t')[3]} for row in genotypes)
27 91.7 MiB -88.6 MiB chunk_step = 100000
28
29 91.7 MiB 0.0 MiB has_keys = True
30 127.4 MiB 35.7 MiB keys = list(itertools.islice(d,chunk_step))
31 152.5 MiB 25.1 MiB while has_keys:
32 153.3 MiB 0.9 MiB snps_database[username].insert_many(keys)
33 152.5 MiB -0.9 MiB keys = list(itertools.islice(d,chunk_step))
34 152.5 MiB 0.0 MiB if len(keys) == 0:
35 104.9 MiB -47.6 MiB has_keys = False
36 # snps_database[username].insert_many(d[i*chunk_step:(i+1)*chunk_step])
37 104.9 MiB 0.0 MiB return
感谢大家的帮助。
【问题讨论】:
-
迭代生成器和生成器表达式(或其他惰性求值的构造)来代替列表。
-
对此我不确定,但您是否尝试过通过
import gc强制垃圾收集,然后在del g和del data之后使用gc.collect()语句。 -
data = [x.split('\t') for x in g]那是因为您没有将列表用作迭代器,您使用的基本上是x = list(something),它必须等待收集所有数据才能创建data多变的。请改用for obj in x.split()。 -
您能否逐步插入
d的内容,而不是通过一次调用database[username].insert_many(d)将所有这些字典放入列表中?如果是这样,那么您可以使用其他一些建议来增量处理其余数据,而不是一次在内存中读取和维护所有数据。另一种方法是将其写出到一个临时文件中,然后一次读回一行/一行。