Georg Semmler

Results 618 comments of Georg Semmler

Closed as this is currently not planed as part of diesel. I encourage interested users to publish such a connection implementation as third party crate for now. After such a...

This is currently not supported. In theory we could have another type that implements a boxed variant of this. That written: At least I do not have the capacity to...

@Ten0 We could probably do something to what we do [here](https://github.com/diesel-rs/diesel/blob/c237a74b63900b734c659a201fc1d6b87108033b/diesel/src/connection/transaction_manager.rs#L313-L325) for `RollbackError`? That would mean we return `SerializationFailure` directly and everything else wrapped into a `CommitTransactionFailed` error?

I've tried to create a minimal reproducible example without diesel [here](https://gist.github.com/weiznich/8f4b7a19ef9789fe960aee3fbed1c66c), but I failed. The example compiles, but does not reproduce the exponential behaviour. I post it nevertheless here, maybe...

I've tried to just reproduce @Ten0 results locally and I've failed doing that. So here is what I done so far: * Modify the linked repo to include up to...

> Incremental compilation cache is actually pretty smart here: if you modify the value in the println, or the value of a number, etc, it will actually hit the cache...

Thanks for this information. It's good to hear that this is already worked on. I've run some more profiling on this. Everything that follows is with 17 tables + recorded...

It's interesting that not appending to the large tuple is expensive, but rather than that checking if a select clause is valid. The corresponding trait impl is rather simple: https://github.com/diesel-rs/diesel/blob/5c0452eae52ebd5ec28f489ce1d395c4330936c2/diesel/src/type_impls/tuples.rs#L193...

I spend a bit more time to look into this. Commenting out the following trait bounds reduces compilation time drastically for me (from 5s to < 1s for 17 joins)...

Sorry to say that, but I would like to avoid both. In my opinion it is a really bad idea to make this behaviour configurable, as it makes it even...