Thomas Sieverding

Results 29 comments of Thomas Sieverding

> It sounds like you may be trying to paper over a code architecture issue. I disagree. As database queries can be grouped together via data loaders, there's a need...

@rex-remind101 I don't intend to support SQLX unless they make this change.

I think it's a bit much to introduce a feature flag for something so trivial; they should just make this change because it's easy and use cases exist

How do we feel about this change? I could update the PR to make the same change for other data structures as well

> * First, this is a breaking change. Actually it's not a breaking change; we can assume that as this currently returns a unit type, everywhere that currently use this...

It might also be worth taking a look at [Avacado](https://h2co3.github.io/avocado/) and [Diesel](http://diesel.rs/) for inspiration. Whereas with your crate queries use bson Document types, Avacado abstracts over this and provides Query/Aggregate...

> Yea, I've looked at Avacado before. Looks like most of their code was ripped off from my code here haha. Haha yea I noticed that as well, though I...

It would really be fantastic to make the aggregate pipeline more ergonomic and type safe as well. For instance, as aggregate can be used for updates but only with $addFields/$set,...

I already made timers that work when ran by commands. I was hoping for a lazy solution so that there didn't need to be a sidekick service that needed to...

Not sure if you saw this but the official driver has async support now.