Fix issue with dates and RDSJanitorResourceTracker

Netflix · Jun 27, 2016 · 92297f7 · 92297f7
1 parent c3121f0
commit 92297f7
Showing 1 changed file with 11 additions and 10 deletions.
diff --git a/src/main/java/com/netflix/simianarmy/aws/janitor/RDSJanitorResourceTracker.java b/src/main/java/com/netflix/simianarmy/aws/janitor/RDSJanitorResourceTracker.java
@@ -264,18 +264,19 @@ private Resource mapResource(ResultSet rs) throws SQLException {
     		throw new SQLException(msg);
     	}    	
         return resource;
-    }             
-
-    private String millisToFormattedDate(String millisStr) {
-    	String datetime = null;
-    	try {
-    		long millis = Long.parseLong(millisStr);
-    		AWSResource.DATE_FORMATTER.print(millis);
-    	} catch(NumberFormatException nfe) {    		
-    	}
-    	return datetime;
     }
 
+	private String millisToFormattedDate(String millisStr) {
+		String datetime = null;
+		try {
+			long millis = Long.parseLong(millisStr);
+			datetime = AWSResource.DATE_FORMATTER.print(millis);
+		} catch(NumberFormatException nfe) {
+			LOGGER.error(String.format("Error parsing datetime %s when reading from RDS", millisStr));
+		}
+		return datetime;
+	}
+
     @Override
     public Resource getResource(String resourceId) {
         Validate.notEmpty(resourceId);