[GitHub] [maven] michael-o commented on pull request #1092: [MNG-7767] Tone down plugin validator

["michael-o (via GitHub)" <gi...@apache.org>]

michael-o commented on PR #1092:
URL: https://github.com/apache/maven/pull/1092#issuecomment-1511379349

   > > I am really confused why `RepositorySystemSession` and `ConfigUtils` are used here. They feel a bit misplaced in plugin validation, don't they?
   > 
   > Well, this PR does not change anything about that. But, as validation is per session, the session is really the only thing that offers itself to store things in "natural" way. MavenSession does not have any means for this, so it is resolver session data that is used. It is done like it as this guarantees that validator does do what it needs to do in correct way in single-threaded and multi-threaded mvn and also in mvnd.
   
   I understand that, but still looks like because it is not the right storage for this. Maven session contains user and system properties.


-- 
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: issues-unsubscribe@maven.apache.org

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