Is this code right after upgrading to 0.7?

[frederikhors]

This code used to work with sqlx before v0.7:

use sqlx::{self, PgExecutor, Pool, Postgres, Transaction};

pub type DB = Pool<Postgres>;
pub trait Queryer<'c>: PgExecutor<'c> {}
impl<'c> Queryer<'c> for &Pool<Postgres> {}
impl<'c> Queryer<'c> for &'c mut Transaction<'_, Postgres> {}

fn main() {}

Now with 0.7 it errors with:

error[E0277]: the trait bound `&'c mut Transaction<'_, Postgres>: Executor<'c>` is not satisfied
  --> src/main.rs:22:26
   |
22 | impl<'c> Queryer<'c> for &'c mut Transaction<'_, Postgres> {}
   |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `Executor<'c>` is not implemented for `&'c mut Transaction<'_, Postgres>`, which is required by `&'c mut Transaction<'_, Postgres>: PgExecutor<'c>`
   |
   = help: the following other types implement trait `Executor<'c>`:
             <&'c mut PgConnection as Executor<'c>>
             <&'c mut PgListener as Executor<'c>>
             <&Pool<DB> as Executor<'p>>
   = note: required for `&'c mut Transaction<'_, Postgres>` to implement `PgExecutor<'c>`
note: required by a bound in `Queryer`
  --> src/main.rs:20:24
   |
20 | pub trait Queryer<'c>: PgExecutor<'c> {}
   |                        ^^^^^^^^^^^^^^ required by this bound in `Queryer`

Can you please help me understad why?

Should I replace this:

impl<'c> Queryer<'c> for &'c mut Transaction<'_, Postgres> {}

with:

impl<'c> Queryer<'c> for &'c mut PgConnection {}

?

[CommanderStorm]

To prevent an XY-problem, please share more code.
It is hard to judge what you want to achieve=> hard to recommend solutions..

Also note that 0.7 is quite a while ago. Is there something wrong with 0.8?

[frederikhors]

Thanks. It the same with 0.8. If you copy & paste that code in your project you can see the error.

[CommanderStorm]

I got that part, but without knowing what you want to do, proposing a solution is not possible..
Why do you want to redifine Executor??

Tip

Trying to unblock you:
If you look at https://docs.rs/sqlx/latest/sqlx/struct.Transaction.html, you will notice that .execute(&mut *tx) is used, as oposed to .execute(&mut tx) which would be required for your impl<'c> Queryer<'c> for &'c mut Transaction<'_, Postgres> {}