This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Optimize dict_merge for copy
Type: performance Stage: resolved
Components: Interpreter Core Versions: Python 3.10
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: methane, serhiy.storchaka, yselivanov
Priority: normal Keywords: patch

Created on 2020-07-29 00:29 by methane, last changed 2022-04-11 14:59 by admin. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 21669 closed methane, 2020-07-29 00:41
PR 21674 merged methane, 2020-07-29 03:18
Messages (5)
msg374550 - (view) Author: Inada Naoki (methane) * (Python committer) Date: 2020-07-29 00:29
Although there are dict.copy() and PyDict_Copy(), dict_merge can be used for copying dict.

* d={}; d.update(orig)
* d=dict(orig)
* d=orig.copy() # orig has many dummy keys.
msg374551 - (view) Author: Inada Naoki (methane) * (Python committer) Date: 2020-07-29 00:50
Microbenchmark for commit cf4f61ce50e07f7ccd3aef991647050c8da058f9.

# timeit -s 'd=dict.fromkeys(range(8))' -- 'dict(d)'
Mean +- std dev: [master] 311 ns +- 2 ns -> [patched] 144 ns +- 1 ns: 2.16x faster (-54%)

# timeit -s 'd=dict.fromkeys(range(1000))' -- 'dict(d)'
Mean +- std dev: [master] 21.6 us +- 0.2 us -> [patched] 7.67 us +- 0.09 us: 2.81x faster (-64%)

# timeit -s 'd=dict.fromkeys(range(8))' -- '{}.update(d)'
Mean +- std dev: [master] 301 ns +- 5 ns -> [patched] 149 ns +- 1 ns: 2.01x faster (-50%)

# timeit -s 'd=dict.fromkeys(range(1000))' -- '{}.update(d)'
Mean +- std dev: [master] 21.4 us +- 0.2 us -> [patched] 7.64 us +- 0.07 us: 2.80x faster (-64%)
msg374555 - (view) Author: Inada Naoki (methane) * (Python committer) Date: 2020-07-29 02:05
To reduce code size, I am considering to remove clone_combined_dict. I will check how PyDict_Copy() is performance critical.

This is microbenchmark result of d.copy() and dict(d).

$ ./python -m pyperf timeit --compare-to ./python-master -s 'd=dict.fromkeys(range(1000))' -- 'd.copy()'
python-master: ..................... 4.36 us +- 0.07 us
python: ..................... 5.96 us +- 0.10 us

Mean +- std dev: [python-master] 4.36 us +- 0.07 us -> [python] 5.96 us +- 0.10 us: 1.37x slower (+37%)

$ ./python -m pyperf timeit --compare-to ./python-master -s 'd=dict.fromkeys(range(1000))' -- 'dict(d)'
python-master: ..................... 21.6 us +- 0.2 us
python: ..................... 6.01 us +- 0.09 us

Mean +- std dev: [python-master] 21.6 us +- 0.2 us -> [python] 6.01 us +- 0.09 us: 3.59x faster (-72%)
msg374556 - (view) Author: Inada Naoki (methane) * (Python committer) Date: 2020-07-29 03:10
PyDict_Copy() is not used in eval loop or calling functions. So removing clone_combined_dict() is a considerable option.

Another option is to use clone_combined_dict() in dict_merge, instead of adding dict_copy2().

Pros: No performance regression. PyDict_Copy() is as fast as before.
Cons: Can not "fast copy" split dict and dirty dict.

I suppose most dict used by `dict(d)` or `dict.update(d)` is clean and combined. So I will implement the second option.
msg374784 - (view) Author: Inada Naoki (methane) * (Python committer) Date: 2020-08-04 02:08
New changeset db6d9a50cee92c0ded7c5cb87331c5f0b1008698 by Inada Naoki in branch 'master':
bpo-41431: Optimize dict_merge for copy (GH-21674)
https://github.com/python/cpython/commit/db6d9a50cee92c0ded7c5cb87331c5f0b1008698
History
Date User Action Args
2022-04-11 14:59:34adminsetgithub: 85603
2020-08-04 02:08:38methanesetstatus: open -> closed
type: performance
resolution: fixed
stage: patch review -> resolved
2020-08-04 02:08:31methanesetmessages: + msg374784
2020-07-29 04:50:10serhiy.storchakasetnosy: + serhiy.storchaka, yselivanov
2020-07-29 03:18:29methanesetstage: patch review
pull_requests: + pull_request20819
2020-07-29 03:10:35methanesetmessages: + msg374556
2020-07-29 02:05:35methanesetmessages: + msg374555
2020-07-29 00:50:51methanesetmessages: + msg374551
stage: patch review -> (no value)
2020-07-29 00:41:12methanesetkeywords: + patch
stage: patch review
pull_requests: + pull_request20813
2020-07-29 00:29:38methanecreate