Github user peter-toth commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2630#discussion_r188609838
  
    --- Diff: 
nifi-nar-bundles/nifi-spark-bundle/nifi-livy-controller-service/src/main/java/org/apache/nifi/controller/livy/LivySessionController.java
 ---
    @@ -241,12 +241,14 @@ public void onConfigured(final ConfigurationContext 
context) {
                 while (enabled) {
                     try {
                         manageSessions();
    +                } catch (Exception e) {
    +                    getLogger().error("Livy Session Manager Thread run 
into an error, but continues to run", e);
    --- End diff --
    
    I agree, changed to what you suggested.


---

Reply via email to