Narcasserun commented on issue #7623:
URL: 
https://github.com/apache/dolphinscheduler/issues/7623#issuecomment-1003023982


   > > This is a big change, a new way of loading plugins has been reintroduced 
again.
   > > I am not clear whether there are requirements of dynamic load the jar, 
it may need to discuss.
   > 
   > I totally agree with @ruanwenjun about this point `whether there are 
requirements of dynamic load the jar`, in my humble opinion dynamically loading 
jars is less needed than you might thought in DolphinScheduler, although 
loading jars with some fancy technology might be cool at the first place, but 
they may be overused at most cases because do we really need this kind of 
stuff? It's better to discuss first whether we really need this considering 
this is big change and we just got rid of classloader things not too long ago.
   
   I think we should pay more attention to what value this feature can bring to 
DS. It is wrong to blindly introduce new technologies and build cars behind 
closed doors. This feature is to isolate the configuration loading related to 
Hadoop, hive and Kerberos, which has nothing to do with the plug-in loading 
method of alarm, task and registry. Maybe it's limited from my own point of 
view @kezhenxu94 @ruanwenjun 


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to