You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@helix.apache.org by GitBox <gi...@apache.org> on 2020/09/02 04:30:23 UTC

[GitHub] [helix] pkuwm opened a new issue #1337: PreFetch annotation parsing may be wrong with getMethods()[0]

pkuwm opened a new issue #1337:
URL: https://github.com/apache/helix/issues/1337


   There are two methods in IZkDataListener. It seems this getMethods()[0] is to get `handleDataChange()`. But actually getMethods() is not guaranteed to be ordered. If an annotation(enabled = false) is applied to `handleDataChange()` in users code, but the result is enabled because JVM `getMethods()[0]` returns `handleDataDeleted()`
   
   ```
   Method callbackMethod = IZkDataListener.class.getMethods()[0];
   ```


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org