Uploaded image for project: 'Nuxeo Platform'
  1. Nuxeo Platform
  2. NXP-28914

Log failure exception in BackingServiceConfigurator

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 10.10
    • Fix Version/s: 11.3, 2021.0
    • Component/s: Runtime

      Description

      Currently only the failure message is displayed, which does not help to understand the problem most of the time.

      Displaying the complete exception would help a lot.

      Suggested change

      diff --git a/nuxeo-runtime/nuxeo-launcher-commons/src/main/java/org/nuxeo/launcher/config/BackingServiceConfigurator.java b/nuxeo-runtime/nuxeo-launcher-commons/src/main/java/org/nuxeo/launcher/config/BackingServiceConfigurator.java
      index f8da07b3bbe..d81d2a71696 100644
      --- a/nuxeo-runtime/nuxeo-launcher-commons/src/main/java/org/nuxeo/launcher/config/BackingServiceConfigurator.java
      +++ b/nuxeo-runtime/nuxeo-launcher-commons/src/main/java/org/nuxeo/launcher/config/BackingServiceConfigurator.java
      @@ -95,7 +95,7 @@ public class BackingServiceConfigurator {
                   if (checker.accepts(configurationGenerator)) {
                       try {
                           Failsafe.with(retryPolicy)
      -                            .onFailedAttempt(failure -> log.error(failure.getMessage())) //
      +                            .onFailedAttempt(failure -> log.error(failure.getMessage(), failure)) //
                                   .onRetry((c, f,
                                           ctx) -> log.warn(String.format("Failure %d. Retrying....", ctx.getExecutions()))) //
                                   .run(() -> checker.check(configurationGenerator)); //
      

       

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 5 minutes
                  5m