Azure / Azure/azure-functions-java-worker

Should use getDeclaredMethods instead of getMethods

Offen
#522 0 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @shreyas-gopalakrishna Auf GitHub ansehen
area:java-functions Enhancement needs-investigation Needs: Triage (Functions) reviewed
Vorherrschende Sprache
Java
Sterne
103
Forks
74
Ø Merge
4 T. 8 Std.
Gemergte PRs (30 T.)
2

Beschreibung

#### 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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.