apache / apache/cloudstack

remove instanceof checks and use reflection for admin cmd invocations

オープン
#6,966 コメント 5 件 リアクション 0 件 担当者 0 名 GitHub で見る
no-issue-activity status:needs-work status:stale type:technical-debt
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。