Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/1274#discussion_r235121753
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/utils/VFSClassloaderUtil.java
---
@@ -112,14 +112,18 @@ public static FileSystemManager generateVfs() throws
FileSystemException {
* @throws FileSystemException
*/
public static Optional<ClassLoader> configureClassloader(String paths)
throws FileSystemException {
+ LOG.debug("Configuring class loader with paths = {}", paths);
if(paths.trim().isEmpty()) {
--- End diff --
Should we add a null check here, just in case? Or use StringUtils.isBlank?
---