Bug report

Pickle allows to define thepersistent_id() method in thePickler class. If it is defined, it is called for every pickled object, and its result will be pickled instead if it is not None. In the Python implementationpersistent_id() is also called for the result ofpersistent_id(), but the result of the second call is ignored. The C implementation only callspersistent_id() once. Calling it twice is just a waste of time in best case, and can have other negative impact (for example if reference for every object for which it was called is stored), so I consider it a bug.

It was found during work on#125752.

Linked PRs

• gh-126489: Do not call persistent_id() for a persistent id in Python pickle #126490
• [3.13] gh-126489: Do not call persistent_id() for a persistent id in Python pickle (GH-126490) #126514
• [3.12] gh-126489: Do not call persistent_id() for a persistent id in Python pickle (GH-126490) #126516