Azure / Azure/azure-functions-java-worker

Should use getDeclaredMethods instead of getMethods

オープン
#522 コメント 0 件 リアクション 0 件 担当者 1 名 @shreyas-gopalakrishna が担当を希望しています GitHub で見る
area:java-functions Enhancement needs-investigation Needs: Triage (Functions) reviewed
主要言語
Java
スター
103
フォーク
74
平均マージ
4日 8時間
マージ済み PR(30日)
2

説明

#### Investigative information

Please provide the following:

- Timestamp:
- Function App name:
- Function name(s) (as appropriate):
- Invocation ID:
- Region:

#### Repro steps

Provide the steps required to reproduce the problem:

#### Expected behavior

Provide a description of the expected behavior.

#### Actual behavior

Provide a description of the actual behavior observed.

#### Known workarounds

Provide a description of any known workarounds.

#### Related information

Provide any related information

* Programming language used
* Links to source
* Bindings used

see the code here it should use getDeclaredMethods to get rid of loop unnecessary method from base class
https://github.com/Azure/azure-functions-java-worker/blob/d7819f42d8230a4e1b76f0b25cab11168b80c144/src/main/java/com/microsoft/azure/functions/worker/broker/EnhancedJavaMethodExecutorImpl.java#L25
https://github.com/Azure/azure-functions-java-worker/blob/d7819f42d8230a4e1b76f0b25cab11168b80c144/src/main/java/com/microsoft/azure/functions/worker/broker/JavaMethodExecutorImpl.java#L26

Also if only compare method name then we cannot do method reload, we should also compare method signature.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

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

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