Issue 18699: What is Future.running() for in PEP 3148 / concurrent.futures.Future? (original) (raw)

Created on 2013-08-09 23:48 by gvanrossum, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 4271 merged python-dev,2017-11-04 08:14
PR 4272 merged serhiy.storchaka,2017-11-04 08:26
Messages (9)
msg194777 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2013-08-09 23:48
From glyph (in a particularly snarky mood): """ P.S.: I realize that this isn't really within the scope of this message, but as I was reviewing Future's documentation while writing it, I realized what Future.running() does. Wow. What a completely pointless bug magnet. Why does this method even exist? It seems like the only possible use would be to introduce look-before-you-leap bugs into concurrent code, like 'if not my_future.running(): my_future.cancel()'. If it were for tracking the state of long-running Future operations for UI purposes or something, then 'my_future.add_start_callback(callback)' would be a much more useful interface, which at least has a hope of being used correctly. Independently of the rest of stuff in this message, it would be great if that could be deprecated and removed. """ Despite the snarkiness, he does have a point. Perhaps it was copied mindlessly from Java?
msg194779 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2013-08-10 01:58
New changeset 922c0fe6ebee by Guido van Rossum in branch 'default': Get rid of Future.running(); see bug 18699. http://hg.python.org/peps/rev/922c0fe6ebee
msg194780 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2013-08-10 02:02
(Note that my change to PEP 3156 is somewhat separate -- Tulip's Future doesn't have to follow all of the PEP 3148 API and its running() method did nothing anyway. But concurrent.futures.Future has been released for several cycles and we must consider the removal of running() there more carefully.)
msg201272 - (view) Author: Brian Quinlan (bquinlan) * (Python committer) Date: 2013-10-25 18:05
The only thing that I've used .running() for is for UI reasons. But it is easier to just iterative over your futures and call a method than to install a callback and handle the state yourself.
msg205133 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2013-12-03 18:23
I see no point in keeping this open.
msg305539 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-11-04 08:13
New changeset b838cc3ff4e039af949c6a19bd896e98e944dcbe by Serhiy Storchaka (Chillar Anand) in branch 'master': bpo-18699: Corrected documentation for window.chgat in curses module (#1430) https://github.com/python/cpython/commit/b838cc3ff4e039af949c6a19bd896e98e944dcbe
msg305541 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-11-04 08:26
New changeset 6dbecd2cd9c9448b34e1a632509220f3c4cf7587 by Serhiy Storchaka (Miss Islington (bot)) in branch '3.6': bpo-18699: Corrected documentation for window.chgat in curses module (GH-1430) (#4271) https://github.com/python/cpython/commit/6dbecd2cd9c9448b34e1a632509220f3c4cf7587
msg305543 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-11-04 08:43
New changeset fd38819497fd2a52ada674f0c890f5b414d0e87f by Serhiy Storchaka in branch '2.7': [2.7] bpo-18699: Corrected documentation for window.chgat in curses module (GH-1430). (#4272) https://github.com/python/cpython/commit/fd38819497fd2a52ada674f0c890f5b414d0e87f
msg305544 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-11-04 08:47
Wrong issue number. All this is related to
History
Date User Action Args
2022-04-11 14:57:49 admin set github: 62899
2017-11-04 08:47:28 serhiy.storchaka set messages: +
2017-11-04 08:43:00 serhiy.storchaka set messages: +
2017-11-04 08:26:46 serhiy.storchaka set messages: +
2017-11-04 08:26:10 serhiy.storchaka set pull_requests: + <pull%5Frequest4235>
2017-11-04 08:14:25 python-dev set pull_requests: + <pull%5Frequest4234>
2017-11-04 08:13:19 serhiy.storchaka set nosy: + serhiy.storchakamessages: +
2017-05-04 18:52:07 Mariatta set pull_requests: - <pull%5Frequest1535>
2017-05-03 13:31:41 python-dev set pull_requests: + <pull%5Frequest1535>
2013-12-03 18:23:12 gvanrossum set status: open -> closedtype: enhancement -> behaviormessages: + resolution: works for mestage: resolved
2013-12-03 18:03:55 vstinner set nosy: + vstinner
2013-10-25 18:05:47 bquinlan set nosy: + bquinlanmessages: +
2013-08-12 19:58:58 asvetlov set nosy: + asvetlov
2013-08-10 02:02:03 gvanrossum set messages: +
2013-08-10 01:58:20 python-dev set nosy: + python-devmessages: +
2013-08-09 23:48:38 gvanrossum create