Opened 13 years ago

Closed 13 years ago

#6967 closed defect (fixed)

@parallel -- dramatically improve by rewriting with fork directly, using files, timeouts, controlling interfaces, etc.

Reported by: William Stein Owned by: Carl Witty
Priority: major Milestone: sage-4.3.1
Component: misc Keywords:
Cc: Merged in: sage-4.3.1.rc1
Authors: William Stein Reviewers: Robert Miller
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by William Stein)

TODO:

  • refactor use_fork.py

Attachments (3)

trac_6967-part1.patch (10.7 KB) - added by William Stein 13 years ago.
first usable version
trac_6967-part2.patch (15.4 KB) - added by William Stein 13 years ago.
trac_6967-part3.patch (1.9 KB) - added by William Stein 13 years ago.

Download all attachments as: .zip

Change History (9)

Changed 13 years ago by William Stein

Attachment: trac_6967-part1.patch added

first usable version

comment:1 Changed 13 years ago by William Stein

Description: modified (diff)

comment:2 Changed 13 years ago by Mitesh Patel

Should we also allow each child process to pull off a large chunk of the input (e.g., from a deque), when it's more efficient? Determine the chunk size dynamically, a la timeit?

comment:3 Changed 13 years ago by William Stein

Report Upstream: N/A
Type: enhancementdefect

This fixes *major* bugs in @parallel sucking. Without this, @parallel gets confused by Sage-isms, preparsing, state, etc., and really hasn't taken off as a result. This fixes all that.

comment:4 Changed 13 years ago by William Stein

Status: newneeds_review

This also fixes a very serious bug in sage.interfaces.quit which will lead to zombie processes being left around, and improves doctest coverage.

Changed 13 years ago by William Stein

Attachment: trac_6967-part2.patch added

Changed 13 years ago by William Stein

Attachment: trac_6967-part3.patch added

comment:5 Changed 13 years ago by Robert Miller

Authors: William Stein
Reviewers: Robert Miller
Status: needs_reviewpositive_review

Awesome!!!

comment:6 Changed 13 years ago by Robert Miller

Merged in: sage-4.3.1.rc1
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.