You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@yetus.apache.org by GitBox <gi...@apache.org> on 2020/12/01 14:58:47 UTC

[GitHub] [yetus] aw-was-here commented on pull request #210: YETUS-1094. Load the personality file before loading plugins.

aw-was-here commented on pull request #210:
URL: https://github.com/apache/yetus/pull/210#issuecomment-736606780


   The order picked in YETUS-1072 was very much intentional.  It doesn't make much sense to load the personality first when it should have "override" capabilities of everything loaded before it.  
   
    While this issue was an unintended side-effect, I'm more inclined to kill off `personality_plugins`.  Long-term, it isn't really the correct model since the personality really has no idea what plug-ins are actually present. Eventually, personalities are pretty much going to get ripped apart and YETUS-1072 was just the first phase.


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