8000 gh-91602: Add iterdump() support for filtering database objects. by felixxm · Pull Request #114501 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

gh-91602: Add iterdump() support for filtering database objects. #114501

< 8000 /div>
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Feb 6, 2024

Conversation

felixxm
Copy link
Contributor
@felixxm felixxm commented Jan 23, 2024
  • Docs changes.
  • More tests.
  • Release notes.

8000
@erlend-aasland
Copy link
Contributor

I see that the check-if-generated-files-are-up-to-date check still fails; you need to do a full make regen-all, in order to generate the needed static string object for the filter keyword parameter.

@felixxm
Copy link
Contributor Author
felixxm commented Jan 24, 2024

I see that the check-if-generated-files-are-up-to-date check still fails; you need to do a full make regen-all, in order to generate the needed static string object for the filter keyword parameter.

Thanks for another tip 🎁 I'm not sure why but it also generated some unrelated lines 🤔.

@erlend-aasland
Copy link
Contributor

Thanks for another tip 🎁 I'm not sure why but it also generated some unrelated lines 🤔.

Yeah, that's strange; I'd rebase onto main and make regen-all again 😄

@erlend-aasland
Copy link
Contributor

BTW, can you please use git pull --noff main when aligning with main, instead of force pushing? :)

@felixxm
Copy link
Contributor Author
felixxm commented Jan 24, 2024

Thanks for all your help, much appreciated. I will work on tests and docs later today.

@felixxm felixxm changed the title [WIP] gh-91602: Add iterdump() support for filtering database objects. gh-91602: Add iterdump() support for filtering database objects. Jan 24, 2024
@felixxm felixxm marked this pull request as ready for review January 24, 2024 19:52
@felixxm felixxm requested a review from berkerpeksag as a code owner January 24, 2024 19:52
@felixxm
Copy link
Contributor Author
felixxm commented Jan 24, 2024

Now it's ready for review 🎉

@erlend-aasland
Copy link
8000 Contributor

Very nice; 321b3e0 made the job a lot easier. I still got some nitpicks, so bear with me :)

@felixxm
Copy link
Contributor Author
felixxm commented Jan 25, 2024

Very nice; 321b3e0 made the job a lot easier. I still got some nitpicks, so bear with me :)

🍿

Copy link
Contributor
@erlend-aasland erlend-aasland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The refcout refactoring in 8cf51d4 is superb; I was just about to suggest it 🚀. I left a small comment regarding how we do the vector call.

When doing changes that affect reference counting, I normally do a refleak test locally:

$ ./python.exe -m test -R : test_sqlite3

Copy link
Contributor
@erlend-aasland erlend-aasland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After this final change, we should be ready to land.

@felixxm
Copy link
Contributor Author
felixxm commented Jan 25, 2024

@erlend-aasland Thanks for all your help 🎁

Copy link
Contributor
@erlend-aasland erlend-aasland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now for the docs; I left some comments :)

@erlend-aasland
Copy link
Contributor

!buildbot .refleak.

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @erlend-aasland for commit e314153 🤖

The command will test the builders whose names match following regular expression: .*refleak.*

The builders matched are:

  • aarch64 RHEL8 Refleaks PR
  • s390x RHEL8 Refleaks PR
  • AMD64 Windows11 Refleaks PR
  • AMD64 Ubuntu NoGIL Refleaks PR
  • AMD64 Fedora Stable Refleaks PR
  • ARM64 MacOS M1 Refleaks NoGIL PR
  • aarch64 Fedora Rawhide Refleaks PR
  • AMD64 Fedora Rawhide Refleaks PR
  • PPC64LE Fedora Rawhide Refleaks PR
  • s390x Fedora Refleaks PR
  • PPC64LE CentOS9 Refleaks PR
  • s390x Fedora Rawhide Refleaks PR
  • PPC64LE RHEL8 Refleaks PR
  • PPC64LE RHEL7 Refleaks PR
  • AMD64 CentOS9 Refleaks PR
  • PPC64LE Fedora Stable Refleaks PR
  • aarch64 Fedora Stable Refleaks PR
  • aarch64 CentOS9 Refleaks PR
  • AMD64 RHEL8 Refleaks PR
  • s390x RHEL7 Refleaks PR
  • AMD64 RHEL7 Refleaks PR

@erlend-aasland
Copy link
Contributor

The buildbot/s390x Fedora Rawhide Refleaks PR buildbot failure is unrelated to this PR.

@erlend-aasland erlend-aasland merged commit 1a10437 into python:main Feb 6, 2024
@erlend-aasland
Copy link
Contributor

Thanks a lot, @felixxm!

@felixxm felixxm deleted the issue-91602 branch February 6, 2024 12:05
fsc-eriker pushed a commit to fsc-eriker/cpython that referenced this pull request Feb 14, 2024
…python#114501)

Add optional 'filter' parameter to iterdump() that allows a "LIKE"
pattern for filtering database objects to dump.

Co-authored-by: Erlend E. Aasland <erlend@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0