Details
Details
- Reviewers
mollekopf - Commits
- rK15f17f1a4193: Slim down number of wallet check jobs
Diff Detail
Diff Detail
- Repository
- rK kolab
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
This may not be a big win because non-owner users do have meet room entitlements in their wallets. Hopefully abusers never visited the Video chat page in dashboard, i.e. the entitlement wasn't created for them.
| src/app/Console/Commands/Wallet/ChargeCommand.php | ||
|---|---|---|
| 70 | This now results in one subquery per wallet, which could potentially make things worse? | |
| src/app/Console/Commands/Wallet/ChargeCommand.php | ||
|---|---|---|
| 70 | But it's not like a query in a loop. The engine should be able to treat it similar to JOIN. | |
| src/app/Console/Commands/Wallet/ChargeCommand.php | ||
|---|---|---|
| 70 | And BTW, this is a scheduled command, this query does not have to be fast. | |