added default control field type and autop-generated value:true calle…

…d columnStatistics. The new default ensures that we calculate cardinality on ingress. changed location where computeStatistics is set, since no need to set it if we are deleting.
socrata · Oct 26, 2015 · 5808dfe · 5808dfe
1 parent 26db0e0
commit 5808dfe
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/main/java/com/socrata/datasync/config/controlfile/ControlFile.java b/src/main/java/com/socrata/datasync/config/controlfile/ControlFile.java
@@ -104,7 +104,8 @@ public static ControlFile generateControlFile(final String fileToPublish,
                .useSocrataGeocoding(useSocrataGeocoding)
                .trimWhitespace(true)
                .trimServerWhitespace(true)
-               .overrides(new HashMap<String, ColumnOverride>());
+               .overrides(new HashMap<String, ColumnOverride>())
+               .columnStatistics(true);
         }
 
         if (isCsv) {

diff --git a/src/main/java/com/socrata/datasync/config/controlfile/FileTypeControl.java b/src/main/java/com/socrata/datasync/config/controlfile/FileTypeControl.java
@@ -40,6 +40,7 @@ public class FileTypeControl {
     public Map<String, LocationColumn> syntheticPoints;
     public Boolean useSocrataGeocoding;
     public String action;
+    public Boolean columnStatistics;
 
     public FileTypeControl() {}
 
@@ -142,6 +143,7 @@ public Set<String> lookupTimestampFormatting() {
     public FileTypeControl hasHeaderRow(boolean h) { hasHeaderRow = h; return this;}
 
     public FileTypeControl action(String a) { action = a; return this;}
-}
+
+    public FileTypeControl columnStatistics(boolean u) { columnStatistics = u; return this; }