Open anth0nyleung opened 12 months ago
Perhaps we should try to make it easy to write crates that extend pgrx with other extension's types like that, or provide a way of automatically binding the custom types. In this specific case, we can try to make it easier for binary-compatible types with existing known types, though.
or provide a way of automatically binding the custom types.
Kinda my current thought process too. I think there's enough information in the pg_type
catalog to make sure we don't mis-handle the backing bytes. But there's a gap from that to "actually usable in a Rust function" that I'm unsure about.
or provide a way of automatically binding the custom types.
Kinda my current thought process too. I think there's enough information in the
pg_type
catalog to make sure we don't mis-handle the backing bytes. But there's a gap from that to "actually usable in a Rust function" that I'm unsure about.
Well, there are transformation functions that Postgres knows about, and presumably the one of "citext -> text" is basically a no-op, so perhaps we can ask Postgres to do it for us, invoking the transformation function to get a type PL/Rust already knows about? And then we push that into the usual extern "C"
shim. We'd offer some kind of syntax for "do this, please".
It is possible this violates the intention of @anth0nyleung though, by exposing a repr of the type.
Hmm. That’s a good thought, actually. Either the xform functions or perhaps through casts (maybe that’s a problem for another day tho).
I think where I’m still a little unsure is how the user tells us which rust type they want the argument to be. For pgrx that’s probably doable with generics at compile time. For plrust I am not sure.
We’d almost need some kind of plrust function where the args are passed as a struct and which has a get_arg
It is possible this violates the intention of @anth0nyleung though, by exposing a repr of the type.
Im probably lacking some background context, im curious
citext is just an example, i think generally we want to enable the support to any type. Im guessing there will be limitation and im interested to understand them. Part of this feature / enhancement request is also because we want to leverage and extend this to the custom type created from the apis we are providing in pg_tle https://github.com/aws/pg_tle/blob/main/docs/09_datatypes.md
PL/Rust does not support non-builtin custom types.
it would be a nice feature if pl/rust function can accept or return them.