joerghoh commented on PR #2358: URL: https://github.com/apache/jackrabbit-oak/pull/2358#issuecomment-3116867892
@reschke I don't understand your concerns. From what I see, the field ```documentPrefixMap``` is consumed within the TargetImportHandler just once: in the method ```currentNamePathMapper()```. Also the method ```createCurrentPrefixMap()``` does not have any side effects and consistently rebuilds the prefixMap from scratch. Based on these 2 observations I think it's fine to call ```createCurrentPrefixMap``` only when this map is required, and not at (arbitrary) intermediate steps. -- 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: oak-dev-unsubscr...@jackrabbit.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
