Add debug logging to migrator. (#7381)

Author: michaelabuckley

hapi-fhir-sql-migrate/src/main/java/ca/uhn/fhir/jpa/migrate/HapiMigrator.java

@@ -246,6 +246,7 @@ private void recordTaskAsCompletedIfNotDryRun(BaseTask theNext, long theExecutio
 
 	public void addTasks(Iterable<BaseTask> theMigrationTasks) {
 		if (HapiSystemProperties.isUnitTestModeEnabled()) {
+			ourLog.info("Skipping tasks because unit test mode is enabled");
 			// Tests only need to initialize the schemas. No need to run all the migrations for every test.
 			for (BaseTask task : theMigrationTasks) {
 				if (task instanceof InitializeSchemaTask) {

hapi-fhir-sql-migrate/src/main/java/ca/uhn/fhir/jpa/migrate/taskdef/BaseTask.java

@@ -217,8 +217,9 @@ private int doExecuteSql(@Language("SQL") String theSql, Object... theArguments)
 		} catch (DataAccessException e) {
 			if (myFlags.contains(TaskFlagEnum.FAILURE_ALLOWED)) {
 				ourLog.info(
-						"Task {} did not exit successfully on doExecuteSql(), but task is allowed to fail",
-						getMigrationVersion());
+						"Task {} running sql \"{}\"did not exit successfully on doExecuteSql(), but task is allowed to fail",
+						getMigrationVersion(),
+						theSql);
 				ourLog.debug("Error was: {}", e.getMessage(), e);
 				myExecutionResult = MigrationTaskExecutionResultEnum.NOT_APPLIED_ALLOWED_FAILURE;
 				return 0;
@@ -334,7 +335,7 @@ public final int hashCode() {
 
 	@Override
 	public final boolean equals(Object theObject) {
-		if (theObject == null || getClass().equals(theObject.getClass()) == false) {
+		if (theObject == null || !getClass().equals(theObject.getClass())) {
 			return false;
 		}
 		BaseTask otherObject = (BaseTask) theObject;