[nifi-registry] branch master updated: NIFIREG-317 Improve logging for errors that occur when creating providers

[kdoran]

This is an automated email from the ASF dual-hosted git repository.

kdoran pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nifi-registry.git


The following commit(s) were added to refs/heads/master by this push:
     new f530195  NIFIREG-317 Improve logging for errors that occur when 
creating providers
f530195 is described below

commit f530195d543d1fcef682048f727b4fb0ad600218
Author: Bryan Bende <bbe...@apache.org>
AuthorDate: Wed Sep 4 14:46:42 2019 -0400

    NIFIREG-317 Improve logging for errors that occur when creating providers
---
 .../org/apache/nifi/registry/provider/StandardProviderFactory.java    | 4 ++++
 1 file changed, 4 insertions(+)

diff --git 
a/nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/provider/StandardProviderFactory.java
 
b/nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/provider/StandardProviderFactory.java
index c9eb9f5..3d14d67 100644
--- 
a/nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/provider/StandardProviderFactory.java
+++ 
b/nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/provider/StandardProviderFactory.java
@@ -121,6 +121,7 @@ public class StandardProviderFactory implements 
ProviderFactory, DisposableBean
                     final JAXBElement<Providers> element = 
unmarshaller.unmarshal(new StreamSource(providersConfigFile), Providers.class);
                     providersHolder.set(element.getValue());
                 } catch (SAXException | JAXBException e) {
+                    LOGGER.error(e.getMessage(), e);
                     throw new ProviderFactoryException("Unable to load the 
providers configuration file at: " + providersConfigFile.getAbsolutePath(), e);
                 }
             } else {
@@ -157,6 +158,7 @@ public class StandardProviderFactory implements 
ProviderFactory, DisposableBean
 
                 LOGGER.info("Instantiated FlowPersistenceProvider with class 
name {}", new Object[]{flowProviderClassName});
             } catch (Exception e) {
+                LOGGER.error(e.getMessage(), e);
                 throw new ProviderFactoryException("Error creating 
FlowPersistenceProvider with class name: " + flowProviderClassName, e);
             }
 
@@ -207,6 +209,7 @@ public class StandardProviderFactory implements 
ProviderFactory, DisposableBean
 
                     LOGGER.info("Instantiated EventHookProvider with class 
name {}", new Object[] {hookProviderClassName});
                 } catch (Exception e) {
+                    LOGGER.error(e.getMessage(), e);
                     throw new ProviderFactoryException("Error creating 
EventHookProvider with class name: " + hookProviderClassName, e);
                 }
 
@@ -250,6 +253,7 @@ public class StandardProviderFactory implements 
ProviderFactory, DisposableBean
 
                 LOGGER.info("Instantiated BundlePersistenceProvider with class 
name {}", new Object[] {extensionBundleProviderClassName});
             } catch (Exception e) {
+                LOGGER.error(e.getMessage(), e);
                 throw new ProviderFactoryException("Error creating 
BundlePersistenceProvider with class name: " + 
extensionBundleProviderClassName, e);
             }