[ 
https://issues.apache.org/jira/browse/MNG-7630?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17678875#comment-17678875
 ] 

ASF GitHub Bot commented on MNG-7630:
-------------------------------------

rmannibucau commented on code in PR #913:
URL: https://github.com/apache/maven/pull/913#discussion_r1081859598


##########
maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/MavenWorkspaceReader.java:
##########
@@ -28,4 +29,20 @@
 public interface MavenWorkspaceReader extends WorkspaceReader {
 
     Model findModel(Artifact artifact);
+
+    /**
+     * List all available artifacts this workspace repository manages.
+     *
+     * @return a stream of artifacts in no particular order
+     * @since 3.9.0
+     */
+    Stream<Artifact> listArtifacts();

Review Comment:
   > It does not, implementors are free to cache items internally
   
   of reactor reader? you want to override it?
   
   About the collection pitfalls, I guess it is mainly about being consistent 
with the track on the API and internals we started so I'd say:
   
   1. make it immutable and in the javadoc,
   2. solved by 1,
   3. solved by 1,
   4. solved by 1.
   
   Strictly speaking streams have the exact same pitfall but you move this 
issue "when you materialize" the view so this is not really a point except the 
fact the view should be immutable as all the work done in the rest of the API 
probably.





> Support listing of workspace models and implement upcomming interface methods
> -----------------------------------------------------------------------------
>
>                 Key: MNG-7630
>                 URL: https://issues.apache.org/jira/browse/MNG-7630
>             Project: Maven
>          Issue Type: Improvement
>            Reporter: Christoph Läubrich
>            Priority: Major
>             Fix For: 3.9.0, 4.0.0-alpha-4, 4.0.0
>
>
> This is the Maven counterpart of MRESOLVER-307.
> As this will be most likely be released in resolver 2.0  only maven can still 
> already have an implementation for the new methods to:
> - show how they would look like
> - if upgrading source is already prepared
> - Maven has an own extension of that interface that should have similar 
> behaviour



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to