Fix shutdown cleanup: stop Tomcat before MariaDB, disable reloadable (#115)

- Change shutdown hook order to stop Tomcat first, then MariaDB,
  ensuring Tomcat can cleanly finish context cleanup before the
  database is killed
- Set context reloadable=false since hot-reloading is unnecessary
  for a standalone distribution and causes extra file monitoring
- Call container.destroy() after container.stop() for proper
  Tomcat lifecycle cleanup

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: dkayiwa

src/main/java/org/openmrs/standalone/ApplicationController.java

@@ -234,17 +234,17 @@ private void init(boolean commandLineMode, boolean nonInteractive, DatabaseMode
 
 		userInterface.setVisible(true);
 
-		// add shutdown hook to stop server
+		// add shutdown hook to stop server (Tomcat first, then MariaDB)
 		Runtime.getRuntime().addShutdownHook(new Thread() {
 
 			public void run() {
+				stopServer();
 				try {
 					stopMariaDB();
 				} catch (ManagedProcessException e) {
 					System.out.println("Failed to stop MariaDB: " + e.getMessage());
 					e.printStackTrace();
 				}
-				stopServer();
 			}
 		});
 

src/main/java/org/openmrs/standalone/TomcatManager.java

@@ -74,7 +74,7 @@ public TomcatManager(String contextName, int port) {
 		String warPath = "tomcat/webapps/" + contextName + ".war";
 		File warFile = new File(warPath);
 		Context rootContext = container.addWebapp("/" + contextName, warFile.getAbsolutePath());
-		rootContext.setReloadable(true);
+		rootContext.setReloadable(false);
 
 		// create http connector
 		Connector httpConnector = new Connector();
@@ -101,6 +101,7 @@ public boolean stop() {
 		try {
 			if (container != null) {
 				container.stop();
+				container.destroy();
 				container = null;
 				stopMySql = true;
 			}