Refactor db_injector decorator

[mjpieters]

The decorator now returns a custom descriptor object instead of a wrapper function. The descriptor can then keep the self and database interface objects separate without having to reverse the normal argument order.

[mjpieters]

I was in the process of cleaning up the database interface when I noticed that this decorator wasn't really useful in a method context but I really wanted it to be. Once this lands I'll be using it to clean up the db interface area.

[codspeed-hq]

CodSpeed Performance Report

Merging #16390 will not alter performance

_{Comparing mjpieters:refactor_db_injector (45fa127) with main (df9d64f)}

Summary

✅ 3 untouched benchmarks

[desertaxle]

There are two PrefectBaseModel implementations. One from prefect._internal.schemas.bases and one from prefect.server.utilities.schemas.bases. The latter is the one that should be used server-side and I think that we could ignore this type by default there without crossing the client/server boundary.

[mjpieters]

That is very interesting and would indeed make this much easier to resolve. I have a circular import to resolve though. Tomorrow.

[mjpieters]

Resolved by creating a base descriptor class in the same module. Any subclass of that base is automatically included in the ignored types, and can trivially be imported without dependency issues.

[desertaxle]

This is great! Thank you!

I'm less familiar with the descriptor protocol than I'd like to be. Do you know of any docs or guides that cover it particularly well?

[mjpieters]

This is great! Thank you!

I'm less familiar with the descriptor protocol than I'd like to be. Do you know of any docs or guides that cover it particularly well?

The Python documentation has been improving on this front, the Descriptor Guide in the howto section is very comprehensive now.