[ https://issues.apache.org/jira/browse/MRESOLVER-445?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17792918#comment-17792918 ]
ASF GitHub Bot commented on MRESOLVER-445: ------------------------------------------ cstamas commented on code in PR #383: URL: https://github.com/apache/maven-resolver/pull/383#discussion_r1414099149 ########## maven-resolver-api/src/main/java/org/eclipse/aether/RepositorySystemSession.java: ########## @@ -391,20 +378,20 @@ interface SessionBuilder { SessionBuilder setData(SessionData data); /** - * Sets the custom session data supplier associated with this session. + * Sets the cache the repository system may use to save data for future reuse during the session. * - * @param dataSupplier The session data supplier, may not be {@code null}. + * @param cache The repository cache, may be {@code null} if none. * @return This session for chaining, never {@code null}. */ - SessionBuilder setSessionDataSupplier(Supplier<SessionData> dataSupplier); + SessionBuilder setCache(RepositoryCache cache); /** - * Sets the cache the repository system may use to save data for future reuse during the session. + * Sets the custom session data supplier associated with this session. * - * @param cache The repository cache, may be {@code null} if none. + * @param dataSupplier The session data supplier, may not be {@code null}. * @return This session for chaining, never {@code null}. */ - SessionBuilder setCache(RepositoryCache cache); + SessionBuilder setSessionDataSupplier(Supplier<SessionData> dataSupplier); Review Comment: Will add a warning about this above to setCache/setData methods, and will merge this PR unless some other concern crops up. > Simplify session handling, move out logic from session builder > -------------------------------------------------------------- > > Key: MRESOLVER-445 > URL: https://issues.apache.org/jira/browse/MRESOLVER-445 > Project: Maven Resolver > Issue Type: Task > Reporter: Tamas Cservenak > Assignee: Tamas Cservenak > Priority: Major > Fix For: 2.0.0 > > > Simplify session handling (copy is gone), also move out logic from session > builder to make it reusable. -- This message was sent by Atlassian Jira (v8.20.10#820010)