[nifi-registry] 04/43: NIFIREG-317 Improve logging for errors that occur when creating providers

[aboda]

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

aboda pushed a commit to branch NIFIREG-371-RC1
in repository https://gitbox.apache.org/repos/asf/nifi-registry.git

commit 1bc729d63a8c90a80d7471111931f0cef7c0e6b2
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);
             }