TransactionBuilder::start Future dropped before completion

[wquark]

batch_execute submits the creation of the transaction (https://github.com/sfackler/rust-postgres/blob/master/tokio-postgres/src/transaction_builder.rs#L109), but the Transaction object itself may never be constructed in case the future is dropped.

f6189a9 fixed it for Client::transaction.

[sfackler]

Good find. We should probably implement transaction with TransactionBuilder to avoid having two places that need to deal with this.

[wquark]

Makes sense. Anything else you need from my end? Is this something you'd knock out quickly or would you rather have a PR?

[sfackler]

Either way!