jira-importer opened a new issue, #668:
URL: https://github.com/apache/maven-indexer/issues/668

   **[Emeric 
Werner](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=JIRAUSER305687)**
 opened 
**[MINDEXER-225](https://issues.apache.org/jira/browse/MINDEXER-225?redirect=false)**
 and commented
   
   The indexer-reader RecordCompactor and RecordExpander classes separate 
individual entries of the `classNames` entry using the field separator `|` (see 
https://github.com/apache/maven-indexer/blob/87627e0e797c1b3cc5bb5f007b94b0371b97f7e0/indexer-reader/src/main/java/org/apache/maven/index/reader/RecordExpander.java#L212
 ). However, the JarFileContentsIndexCreator class in indexer-core uses 
newlines (`\n`) as a separator between classes as seen in 
https://github.com/apache/maven-indexer/blob/87627e0e797c1b3cc5bb5f007b94b0371b97f7e0/indexer-core/src/main/java/org/apache/maven/index/creator/JarFileContentsIndexCreator.java#L166
 . The behaviour of indexer-core with it's JarFileContentsIndexCreator is the 
behaviour I am empirically seeing in the wild with sample 
nexus-maven-repository-index.gz files.
   
   Of course, one can still pass all the contents in a String whose elements 
were joined with the newline character ahead of time - but at that point it 
makes little sense why a String array needs to be passed as Record's value as 
it will almost always be a String-array of size 1 anyways (on a similar note, 
the documentation states that the `classNames` entry stores a value of type 
java.util.List\<String>, but that is incorrect - it is a String[]).
   
   
   ---
   
   **Affects:** 7.1.3
   


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