tbonelee opened a new pull request, #4969:
URL: https://github.com/apache/zeppelin/pull/4969

   ### What is this PR for?
   Currently, the server keeps running even when all web app contexts fail to 
initialize due to missing files or directories for web resources.
   
   In my opinion, if the web resource path for the default web app context does 
not exist, it would be better to shut down the server immediately, since the 
context initialization will fail anyway. In such cases, other essential 
features like REST APIs and WebSocket communication also won’t work properly, 
so keeping the server running doesn’t seem meaningful.
   
   The absence of non-default web resources, however, seems generally 
acceptable. So this PR ensures that we only fail fast when the default web app 
context is missing its required resources.
   
   ### What type of PR is it?
   Improvement
   
   ### What is the Jira issue?
   - https://issues.apache.org/jira/browse/ZEPPELIN-6225
   
   ### How should this be tested?
   - Start the server with the default web app directory intentionally missing.
   - Verify that the server fails to start and exit immediately.
   - Ensure that non-default apps can still be missing without preventing 
startup
   
   ### Questions:
   * Does the license files need to update? No
   * Is there breaking changes for older versions? No
   * Does this needs documentation? No


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to