tools.ThreadList.stop_all() leads to AttributeError
Closed, ResolvedPublicBUG REPORT
Actions

Assigned To

Authored By

	Xqt
	May 7 2022, 8:00 AM

Description

List of steps to reproduce:

Add self.threads.stop_all() to weblinkchecker.py e.g. within this at in line 623 before the break statement.
run pwb weblinkchecker -simulate -cat:Neapel

What happens?:

The script fails with AttributeError because a threading.Thread does not have a` stop()` method

    self.exit()
  File "C:\pwb\GIT\core\pywikibot\bot.py", line 1484, in exit
    self.teardown()
  File ".\scripts\weblinkchecker.py", line 623, in teardown
    self.threads.stop_all()
  File "C:\pwb\GIT\core\pywikibot\tools\__init__.py", line 818, in stop_all
    thd.stop()
AttributeError: 'LinkCheckThread' object has no attribute 'stop'. Did you mean: '_stop'?
CRITICAL: Exiting due to uncaught exception <class 'AttributeError'>

Software version:
introduced with rPWBC4d756b3

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	[bugfix] Remove ThreadList.stop_all() method	pywikibot/core	master	+0 -10

Customize query in gerrit

Related Objects

Mentioned Here: rPWBC4d756b3086fc: Intersect generators

Event Timeline

Xqt created this task.May 7 2022, 8:00 AM

Restricted Application added subscribers: pywikibot-bugs-list, Aklapper. · View Herald TranscriptMay 7 2022, 8:00 AM

Xqt claimed this task.May 7 2022, 8:02 AM

Xqt triaged this task as Medium priority.

Xqt added subscribers: Mpaa, jayvdb.

Change 789893 had a related patch set uploaded (by Xqt; author: Xqt):

[pywikibot/core@master] [bugfix] Remove ThreadList.stop_all() method

https://gerrit.wikimedia.org/r/789893

gerritbot added a project: Patch-For-Review.May 7 2022, 8:23 AM

@Mpaa: I am unsure how to proceed. We could

keep the current implementation and add a warning to our documentation that the Thread object hast to be sublassed and provide a stop method if stop_all() is to be used (like ThreadedGenerator does)
remove the stop_all method and stopping must be implemented outside the ThreadList class
ignore AttributeError in stop_all() method and just write a debug message (and add a warning to our documentation)

What do you suggest here?

In T307830#7911218, @Xqt wrote:

@Mpaa: I am unsure how to proceed. We could

keep the current implementation and add a warning to our documentation that the Thread object hast to be sublassed and provide a stop method if stop_all() is to be used (like ThreadedGenerator does)

remove the stop_all method and stopping must be implemented outside the ThreadList class

ignore AttributeError in stop_all() method and just write a debug message (and add a warning to our documentation)

What do you suggest here?

I agree with remove the stop_all method

Xqt closed this task as Resolved.May 7 2022, 11:32 AM

Change 789893 merged by jenkins-bot: