Revert "Added handling of database resources after migration"#10406
Revert "Added handling of database resources after migration"#10406
Conversation
|
Issue reopened: DAT-664 When I delete all rows, total rows count goes to -10 |
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThis change removes StatsUsage usage from the codebase in two areas. In Appwrite\Event\StatsUsage, the public reset() method is deleted. In Appwrite\Platform\Workers\Migrations, all StatsUsage dependencies and metrics collection logic are removed: imports, constructor/parameters, method signatures, aggregated resource handling, per-resource stats emission, the processMigrationResourceStats helper, and the $source member. Migrations’ action() and processMigration() now omit StatsUsage and operate without stats aggregation, with the Transfer callback simplified to a no-argument closure. Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 PHPStan (2.1.17)Note: Using configuration file /phpstan.neon. If the excluded path can sometimes exist, append (?) parameters: 📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
Status, Documentation and Community
|
Security Scan Results for PRDocker Image Scan Results
Source Code Scan Results🎉 No vulnerabilities found! |
✨ Benchmark results
⚡ Benchmark Comparison
|
Reverts #10400