[GitHub] [pulsar] jerrypeng commented on issue #4176: [pulsar function] refactor function runtime

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

jerrypeng commented on issue #4176: [pulsar function] refactor function runtime
URL: https://github.com/apache/pulsar/issues/4176#issuecomment-488052148
 
 
   Yup I agree some refactoring can and should be done to make runtimes more modular and that the code between all of these runtimes is not sprinkled everywhere in the code.
   
   I can definitely seen the confusion here with the overloaded term "runtime".  We have a interface call Runtime and we have an enum called Runtime but they refer to different things.  The former refers more to a Runtime Environment and the later refers to a Runtime language.  We can discuss how we can rename to eliminate this confusion.  Maybe we can just change the Runtime interface to be called something like RuntimeEnv 

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