Releasing DataSync 1.5.4 with some minor clean-up

socrata · Dec 31, 2014 · d3838e3 · d3838e3
1 parent cd42661
commit d3838e3
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 12 deletions.
diff --git a/README.md b/README.md
@@ -50,5 +50,5 @@ mvn clean compile -Dmaven.test.skip=true assembly:single
 This puts the JAR file into the "target" directory inside the repo.  So to open DataSync, simply:
 ```
 cd target
-java -jar DataSync-1.5.3-jar-with-dependencies.jar
+java -jar DataSync-1.5.4-jar-with-dependencies.jar
 ```
diff --git a/pom.xml b/pom.xml
@@ -3,8 +3,13 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>DataSync</groupId>
     <artifactId>DataSync</artifactId>
-    <version>1.5.3</version>
+    <version>1.5.4</version>
     <developers>
+        <developer>
+            <name>Ayn Leslie-Cook</name>
+            <email>ayn.leslie@socrata.com</email>
+            <organization>Socrata</organization>
+        </developer>
         <developer>
             <name>Adrian Laurenzi</name>
             <email>adrian.laurenzi@socrata.com</email>

diff --git a/src/main/java/com/socrata/datasync/config/controlfile/FileTypeControl.java b/src/main/java/com/socrata/datasync/config/controlfile/FileTypeControl.java
@@ -68,10 +68,12 @@ public boolean hasTimestampFormatting() {
     public Set<String> lookupTimestampFormatting() {
         Set<String> formats = new HashSet<>();
 
-        for (String format : floatingTimestampFormat)
-            formats.add(format);
-        for (String format : fixedTimestampFormat)
-            formats.add(format);
+        if (floatingTimestampFormat != null)
+            for (String format : floatingTimestampFormat)
+                formats.add(format);
+        if (fixedTimestampFormat != null)
+            for (String format : fixedTimestampFormat)
+                formats.add(format);
         if (overrides != null) {
             for (ColumnOverride override : overrides.values()) {
                 if (override.timestampFormat != null) {

diff --git a/src/main/java/com/socrata/datasync/validation/IntegrationJobValidity.java b/src/main/java/com/socrata/datasync/validation/IntegrationJobValidity.java
@@ -119,17 +119,17 @@ public static JobStatus validateJobParams(UserPreferences userPrefs, Integration
                     return noHeaders;
                 }
 
+                JobStatus controlHeaderAgreement = checkControlAgreement(fileControl, schema, rawHeaders,
+                        intermediateHeaders, publishFile.getName());
+                if (controlHeaderAgreement.isError())
+                    return controlHeaderAgreement;
+
                 Set<String> synthetics = new HashSet<>();
                 if (fileControl.hasSyntheticLocations()) synthetics = fileControl.syntheticLocations.keySet();
                 JobStatus csvDatasetAgreement = checkColumnAgreement(schema, finalHeaders, synthetics, publishFile.getName());
                 if (csvDatasetAgreement.isError())
                     return csvDatasetAgreement;
 
-                JobStatus controlHeaderAgreement = checkControlAgreement(fileControl, schema, rawHeaders,
-                        intermediateHeaders, publishFile.getName());
-                if (controlHeaderAgreement.isError())
-                    return controlHeaderAgreement;
-
             }
         } catch (Exception e) {
             // Not going to fail jobs on the validation check

diff --git a/src/test/java/com/socrata/datasync/VersionProviderTest.java b/src/test/java/com/socrata/datasync/VersionProviderTest.java
@@ -48,6 +48,6 @@ public void testIsLatestVersion() throws URISyntaxException {
 
     @Test
     public void testGetThisVersion() {
-        TestCase.assertEquals("1.5.3", VersionProvider.getThisVersion());
+        TestCase.assertEquals("1.5.4", VersionProvider.getThisVersion());
     }
 }