Azure / Azure/azure-functions-java-worker

Should use getDeclaredMethods instead of getMethods

未关闭
#522 0 条评论 0 个 reaction 已指派 1 人 已被 @shreyas-gopalakrishna 认领 在 GitHub 查看
area:java-functions Enhancement needs-investigation Needs: Triage (Functions) reviewed
主要语言
Java
星标
103
派生
74
平均合并
4 天 8 小时
30 天内合并 PR
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 摘要。