EpicsDAO / zapp

⚡Rust Serverless Framework
Apache License 2.0
11 stars 3 forks source link

Remove async/await for now #12

Closed JonasCir closed 1 year ago

JonasCir commented 2 years ago

Currently, async/await is used all over the place, however, it does not provide any advantage b/c we do not submit any task to an executor or used any concurrency.

I argue that we should selectively re-introduce it properly in parts where it actually makes sense.

You can refer to this resource to learn more.

POPPIN-FUMI commented 1 year ago

Yeah if it's not a big work it would be awesome. But so far, I have prioritized building scalable apps faster. Do you think you can implement this?

JonasCir commented 1 year ago

No big work. I totally agree what you said about scalable apps, but right now I tend to get a solid foundation and then see where async provides benefit. Right now, I had issues that I couldn't program some stuff the way I would like to b/c async interfering :)

13 is up

POPPIN-FUMI commented 1 year ago

oh that's so fast! yea true solid foundation at the beginning is important ! I will check them later. Thank you!