[GitHub] [samza] cameronlee314 opened a new pull request #1047: SAMZA-2210: Initial majority migration for injecting classloader when doing reflection

[GitBox <gi...@apache.org>]

cameronlee314 opened a new pull request #1047: SAMZA-2210: Initial majority migration for injecting classloader when doing reflection 
URL: https://github.com/apache/samza/pull/1047
 
 
   When doing reflection, we will now explicitly specify the classloader to use. This will allow us to inject a custom classloader in the future, in order to do things like isolate Samza infrastructure from application code.
   From a functionality perspective, this should not change anything. Ultimately, for this PR, the classloader that is passed through is the same classloader that is already being used. The purpose of this PR is to do the wiring to allow easier evolution in a follow-up PR.
   This does not cover all reflection cases. There are several other follow-up PRs which will handle other reflection cases. This PR is already very significantly large, so trying to keep it from being even larger.
   
   This PR includes https://github.com/apache/samza/pull/1044.

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


With regards,
Apache Git Services