为什么subprocess.Popen不等待子进程终止？

Question

我遇到了Python的subprocess.Popen方法的问题.

这是一个演示问题的测试脚本.它正在Linux机器上运行.

#!/usr/bin/env python
import subprocess
import time

def run(cmd):
  p = subprocess.Popen(cmd, shell=True, stdout=subprocess.PIPE)
  return p

### START MAIN
# copy some rows from a source table to a destination table
# note that the destination table is empty when this script is run
cmd = 'mysql -u ve --skip-column-names --batch --execute="insert into destination (select * from source limit 100000)" test'
run(cmd)

# check to see how many rows exist in the destination table
cmd = 'mysql -u ve --skip-column-names --batch --execute="select count(*) from destination" test'
process = run(cmd)
count = (int(process.communicate()[0][:-1]))

# if subprocess.Popen() waited for the child to terminate than count should be
# greater than 0
if count > 0:
  print "success: " + str(count)
else:
  print "failure: " + str(count)
  time.sleep(5)

  # find out how many rows exists in the destination table after sleeping
  process = run(cmd)
  count = (int(process.communicate()[0][:-1]))
  print "after sleeping the count is " + str(count)

通常这个脚本的输出是:

success: 100000

但有时它是

failure: 0
after sleeping the count is 100000

请注意,在失败的情况下,插入后立即显示0行,但在睡眠5秒后,第二次选择正确显示行数为100000.我的结论是以下之一为真:

1. subprocess.Popen没有等待子线程终止 - 这似乎与文档相矛盾
2. mysql插入不是原子的 - 我对mysql的理解似乎表明插入是原子的
3. 选择没有立即看到正确的行数 - 根据一个比我更了解mysql的朋友,这也不应该发生

我错过了什么？

仅供参考,我知道这是一种从Python与mysql交互的hacky方式,MySQLdb可能没有这个问题,但我很好奇为什么这个方法不起作用.

Answer 1

subprocess.Popen,实例化时,运行该程序.但是,它不会等待它 - 它会在后台触发它,就像你输入cmd &一个shell一样.所以,在上面的代码中,你基本上定义了一个竞争条件 - 如果插入可以及时完成,它将显示正常,但如果没有,你会得到意外的输出.你不是在等待你的第一个run()PID完成,你只是返回它的Popen实例并继续.

我不确定这种行为是如何与文档相矛盾的,因为在Popen上有一些非常明确的方法似乎表明它没有等待,例如:

Popen.wait()
  Wait for child process to terminate. Set and return returncode attribute.

但我同意,可以改进该模块的文档.

要等待程序完成,我建议subprocess使用方便的方法subprocess.call,或者communicate在Popen对象上使用(对于需要stdout的情况).您已经为第二次通话做了这个.

### START MAIN
# copy some rows from a source table to a destination table
# note that the destination table is empty when this script is run
cmd = 'mysql -u ve --skip-column-names --batch --execute="insert into destination (select * from source limit 100000)" test'
subprocess.call(cmd)

# check to see how many rows exist in the destination table
cmd = 'mysql -u ve --skip-column-names --batch --execute="select count(*) from destination" test'
process = subprocess.Popen(cmd, shell=True, stdout=subprocess.PIPE)
try: count = (int(process.communicate()[0][:-1]))
except: count = 0

此外,在大多数情况下,您不需要在shell中运行该命令.这是其中一种情况,但您必须像序列一样重写命令.这样做也可以避免传统的shell注入,而不用担心引用,如下所示:

prog = ["mysql", "-u", "ve", "--execute", 'insert into foo values ("snargle", 2)']
subprocess.call(prog)

这甚至会起作用,并且不会像你期望的那样注入:

prog = ["printf", "%s", "<", "/etc/passwd"]
subprocess.call(prog)

以交互方式尝试.您可以避免shell注入的可能性,尤其是在您接受用户输入的情况下.我怀疑你正在使用与子进程通信的不那么棒的字符串方法,因为你在使序列工作时遇到了麻烦:^)

Answer 2

如果你不是绝对需要使用subprocess和popen,它通常更容易使用os.system.例如,对于快速脚本,我经常做这样的事情:

import os
run = os.system #convenience alias
result = run('mysql -u ve --execute="select * from wherever" test')

与popen不同,os.systemDOES会等待您的进程返回,然后再转到脚本的下一个阶段.

有关它的更多信息,请访问以下文档:http://docs.python.org/library/os.html#os.system