remove instanceof checks and use reflection for admin cmd invocations
- 主要言語
- Java
- スター
- 3.1k
- フォーク
- 1.4k
- 平均マージ
- 6日 19時間
- マージ済み PR(30日)
- 32
説明
##### ISSUE TYPE
* cleanup
##### COMPONENT NAME
~~~
API
~~~
##### CLOUDSTACK VERSION
~~~
4.17
~~~
##### CONFIGURATION
##### OS / ENVIRONMENT
##### SUMMARY
the API classes have instances that are for root admin only, these are checked for using `instanceof` in the service layer. This should be cleaned up in favour for reflection invocations as in #6951. It would be better to do all in one go but the fix in #6951 really needs it and is for a point release, so postponing.
##### STEPS TO REPRODUCE
~~~
~~~
##### EXPECTED RESULTS
~~~
~~~
##### ACTUAL RESULTS
~~~
~~~
コントリビューションガイド
調査の方向性
admin コマンドの呼び出しを処理する API クラスとサービス層から始め、次に #6951 で説明されている reflection アプローチと比較します。root admin 専用のインスタンスに対する既存の instanceof チェックを追跡し、影響を受ける呼び出し経路を特定します。これらのチェックが意図された reflection アプローチを使用し、admin 専用の動作を維持できれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- java
- 領域
- api, cloud
- issue の種類
- リファクタリング
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 静か
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100