You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add new API endpoint for getting instance overview statistics
- Implement GetInstanceOverviewStatistics function in statistic controller
- Update router to include new endpoint
- Add placeholder implementation for community edition
- Add new API endpoint: GET /v1/statistic/instances/resource_overview_statistics
- This endpoint provides statistics on instance average score, high priority SQL count, and pending workflow count
- Update API documentation in docs.go, swagger.json, and swagger.yaml
Why: The suggestion addresses a security concern by adding a middleware for authorization on the new endpoint, aligning it with the other endpoints in the file. The improvement is both relevant and accurate, though its impact is moderate since it may depend on the intended accessibility of the endpoint.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
User description
关联的 issue
https://github.com/actiontech/sqle-ee/issues/2321
描述你的变更
确认项（pr提交后操作）
Tip
请在指定复审人之前，确认并完成以下事项，完成后✅
not_compatibleneed_update_docDescription
新增GET /statistic/instances/resource_overview_statistics接口
添加实例概览统计请求和响应结构体
增加社区版不支持统计的桩函数
更新API文档及Swagger配置文件
Changes walkthrough 📝
app.go
新增接口路由调用sqle/api/app.go
statistic.go
新增统计接口实现sqle/api/controller/v1/statistic.go
statistic_ce.go
新增统计接口桩函数sqle/api/controller/v1/statistic_ce.go
docs.go
更新文档添加新接口定义sqle/docs/docs.go
swagger.json
更新swagger.json添加新接口sqle/docs/swagger.json
swagger.yaml
更新swagger.yaml添加新接口sqle/docs/swagger.yaml