Jack Christensen

What I mean is, what happens when it tries to load `_hstore` when `hstore` is not registered yet?

> Were you thinking that the pgxpool's struct would contain a function used to register the types? Yes, something like this. Though since pgxpool already has a AfterConnect hook it...

Reuse would still be optional. It would be up to the caller to implement. Here is what I'm thinking: ```go makeLoadTypesOnce := func() func(ctx context.Context, conn *pgx.Conn) error { var...

I'd prefer to hold off on the environment variable. We can always add it later, but once it's there we can't remove it.

@nicois There is still the race condition mentioned above. But there is one other thing that needs to be figured out that was brought to my attention by #2089. `EnumCodec`...

> Relating to state mutation: if types and codecs shouldn't be mutated, should they have a Copy() member which produces an equivalent object with a separate internal state? If so,...

As mentioned by others above `CancelRequest` in your application logic is the correct approach *if you must use a single connection*. Unfortunately, it is somewhere between very difficult to impossible...

> Question about, Why we close rows inside Row.Scan(..)? stays. IMO it's what the developer would expect in most cases and it's convenient. A `Scan` error is almost always a...

This behavior goes all the way back to v2 when the `Rows` interface was introduced. Even if I did agree that is was something that should be changed, it's not...

Only reason that should happen is if the driver thinks the column is an int4. Maybe double check the underlying type. Maybe try with a test table with bigint to...