search

macressler / alfresco-bulk-filesystem-import

Automatically exported from code.google.com/p/alfresco-bulk-filesystem-import
GNU Lesser General Public License v3.0
1 stars 0 forks source link

DirectoryAnalyserImpl should use a Map, not a List, for intermediate processing #91

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Currently the DirectoryAnalyserImpl class uses a List<ImportableItem> during 
the analysis process.  From profiling I can see that a large part of the time 
the analyser spends is scanning this list repeatedly to find out whether a 
given file is already in there or not.  Using a Map<String,ImportableItem> 
would almost certainly be a better option, with the key being the name of the 
"parent" object.

Original issue reported on code.google.com by pmo...@gmail.com on 9 Oct 2011 at 4:54

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision 518e8cc688e6.

Original comment by pmo...@gmail.com on 9 Oct 2011 at 10:33