[
https://issues.apache.org/jira/browse/METRON-1887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16693649#comment-16693649
]
ASF GitHub Bot commented on METRON-1887:
----------------------------------------
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?
> Add logging to the ClasspathFunctionResolver
> --------------------------------------------
>
> Key: METRON-1887
> URL: https://issues.apache.org/jira/browse/METRON-1887
> Project: Metron
> Issue Type: Improvement
> Reporter: Michael Miklavcic
> Assignee: Michael Miklavcic
> Priority: Major
>
> We had a user reporting non-deterministic NPE's here -
> [https://github.com/apache/metron/blob/master/metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java#L250.]
> The purpose of this ticket is to more gracefully handle the NPE and add some
> additional debugging information to better track down the source of this
> issue should it come up again.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
