#96 Adjusted logging to use "non-REST modules" instead of "asset modules"

Author: rjrudin

src/main/java/com/marklogic/client/ext/modulesloader/impl/BaseModulesFinder.java

@@ -63,11 +63,11 @@ protected void addAssetDirectories(Modules modules, String baseDir) {
 		List<String> recognizedPaths = getRecognizedPaths();
 
 		// classpath needs the trailing / to find child dirs
-		findResources("asset module directories", baseDir, "*", "*/").stream().forEach(resource -> {
+		findResources("non-REST module directories", baseDir, "*", "*/").stream().forEach(resource -> {
 			try {
 				String resourceFile = resource.getURL().getFile();
 				if (logger.isDebugEnabled()) {
-					logger.debug("Checking resource to see if it's a valid asset directory: " + resourceFile);
+					logger.debug("Checking resource to see if it's a valid non-REST module directory: " + resourceFile);
 				}
 
 				resourceFile = decodeAssetDirectoryResource(resourceFile);
@@ -107,7 +107,7 @@ protected String decodeAssetDirectoryResource(String resourceFile) {
 	    if (resourceFile.contains("%20")) {
 		    resourceFile = resourceFile.replaceAll("%20", " ");
 		    if (logger.isDebugEnabled()) {
-			    logger.debug("Replaced occurrences of %20 with a space in potential asset directory: " + resourceFile);
+			    logger.debug("Replaced occurrences of %20 with a space in potential non-REST module directory: " + resourceFile);
 		    }
 	    }
 	    return resourceFile;

src/main/java/com/marklogic/client/ext/modulesloader/impl/DefaultModulesLoader.java

@@ -94,7 +94,7 @@ public DefaultModulesLoader(AssetFileLoader assetFileLoader) {
 	}
 
 	/**
-	 * Use this when you don't need to load asset modules.
+	 * Use this when you don't need to load asset modules - i.e. only need to load REST modules.
 	 */
 	public DefaultModulesLoader() {
 		this.extensionMetadataProvider = new DefaultExtensionMetadataProvider();
@@ -124,8 +124,8 @@ public void initializeDefaultTaskExecutor() {
 	/**
 	 * Load modules from the given base directory, selecting modules via the given ModulesFinder, and loading them via
 	 * the given DatabaseClient. Note that asset modules will not be loaded by the DatabaseClient that's passed in here,
-	 * because the /v1/ext endpoint is so slow - load assets instead via a RestApiAssetLoader or an XccAssetLoader
-	 * passed into a constructor for this class.
+	 * because the /v1/ext endpoint is too slow when there are e.g. hundreds of modules or more - asset modules are
+	 * instead loaded via AssetFileLoader.
 	 */
 	public Set<Resource> loadModules(String baseDir, ModulesFinder modulesFinder, DatabaseClient client) {
 		if (logger.isDebugEnabled()) {
@@ -330,7 +330,7 @@ protected void loadAssets(Modules modules, Set<Resource> loadedModules) {
 		}
 
 		if (logger.isDebugEnabled()) {
-			logger.debug("Loading asset modules from paths: " + Arrays.asList(paths));
+			logger.debug("Loading non-REST modules from paths: " + Arrays.asList(paths));
 		}
 
 		if (includeFilenamePattern != null) {