Closed edongashi closed 3 years ago
Standard tests have passed ❤️
Pull request can be merged 🎉
Hm, hang on, now the "waiting for database" is basically zero, right? This is going to fuck up the "slow queries" graph (see https://demo.aircloak.com/admin/queries/performance)
Hm, ok, let's remove the states we no longer need should we ever make another release.
Hm, hang on, now the "waiting for database" is basically zero, right? This is going to fuck up the "slow queries" graph (see https://demo.aircloak.com/admin/queries/performance)
Yeah, sadly this change will ripple in a lot of places. We risk breaking too many things if we start messing with the query lifecycle...
Yeah, created a separate issue to keep track of this.
I have ran queries through all possible channels and haven't encountered any bug.
This and #4882 contain the changes for the minor release 21.1.