bpo-27575: port set intersection logic into dictview intersection by fgregg · Pull Request #7696 · python/cpython

@fgregg

This turns a patch contributed by @dsu1995 on bpo into a github PR.

The patch ports the logic of set intersection into the intersection of dictviews. It will address performance issues when taking the intersection of large dictviews.

https://bugs.python.org/issue27575

dysu added 5 commits

June 14, 2018 12:21

@the-knights-who-say-ni

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA).

Unfortunately our records indicate you have not signed the CLA. For legal reasons we need you to sign this before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

When your account is ready, please add a comment in this pull request
and a Python core developer will remove the CLA not signed label
to make the bot check again.

Thanks again for your contribution, we look forward to reviewing it!

@fgregg

@fgregg

@fgregg

@fgregg

@rhettinger, this PR is ready for review. I've verified all the code paths are covered by the tests.

@fgregg

@fgregg

@Mariatta

Friendly reminder to use "Co-authored by" in the commit message in situations like this, where patch was originally started by someone else. Thanks.

Devguide reference
GitHub article

@fgregg

Hi @Mariatta!

I wasn't sure if that guidance applied here, because I didn't apply @dsu1995 patches and them make a new commit. Instead, I cherry-picked from his git branch.

As such, David Su is already listed as the sole author of his commits.

If this is not the preferred option, I can redo this branch so that it does not have @dsu1995's commits and instead just has a commit where I applied his patch. In that commit I will list him as the Co-Author in the commit message.

Please advise.

@fgregg

ZackerySpytz

methane

@bedevere-bot

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

dsu1995

@csabella

@fgregg, please address the code review requests. Also, due to the code review requests, the 'Co-authored by:' seems appropriate.

@rhettinger

@fgregg

thanks for finishing this up, @rhettinger! And for the opportunity to contribute to CPython

@dsu1995

Thank you @rhettinger! And thank you @fgregg for migrating my bpo patch to a Github PR and cleaning up the code!

lisroach pushed a commit to lisroach/cpython that referenced this pull request

Sep 10, 2019

DinoV pushed a commit to DinoV/cpython that referenced this pull request

Jan 14, 2020

websurfer5 pushed a commit to websurfer5/cpython that referenced this pull request

Jul 20, 2020