Improve write speed (requery#119)

* Add recommended compile flags
* Update SQLiteGlobal constants to recommended (from android bindings)
* Improve sqlite statement detection speed
* Remove DQS=0 compile options, because it can break backward support
* Remove unnecessary toUpperCase() when detecting sqlite statement type

Author: alexoro

sqlite-android/src/main/java/io/requery/android/database/sqlite/SQLiteGlobal.java

@@ -97,20 +97,20 @@ public static String getDefaultSyncMode() {
      * Gets the database synchronization mode when in WAL mode.
      */
     public static String getWALSyncMode() {
-        return "FULL";
+        return "normal";
     }
 
     /**
      * Gets the WAL auto-checkpoint integer in database pages.
      */
     public static int getWALAutoCheckpoint() {
-        return 100;
+        return 1000;
     }
 
     /**
      * Gets the connection pool size when in WAL mode.
      */
     public static int getWALConnectionPoolSize() {
-        return 4;
+        return 10;
     }
 }

sqlite-android/src/main/java/io/requery/android/database/sqlite/SQLiteStatementType.java

@@ -64,35 +64,43 @@ public static int getSqlStatementType(String sql) {
         if (sql.length() < 3) {
             return STATEMENT_OTHER;
         }
-        String prefixSql = sql.substring(0, 3).toUpperCase(Locale.US);
-        switch (prefixSql) {
-            case "SEL":
-                return STATEMENT_SELECT;
-            case "INS":
-            case "UPD":
-            case "REP":
-            case "DEL":
-                return STATEMENT_UPDATE;
-            case "ATT":
-                return STATEMENT_ATTACH;
-            case "COM":
-                return STATEMENT_COMMIT;
-            case "END":
-                return STATEMENT_COMMIT;
-            case "ROL":
-                return STATEMENT_ABORT;
-            case "BEG":
-                return STATEMENT_BEGIN;
-            case "PRA":
-                return STATEMENT_PRAGMA;
-            case "CRE":
-            case "DRO":
-            case "ALT":
-                return STATEMENT_DDL;
-            case "ANA":
-            case "DET":
-                return STATEMENT_UNPREPARED;
+        String prefixSql = sql.substring(0, 3);
+
+        if (prefixSql.equalsIgnoreCase("SEL")) {
+            return STATEMENT_SELECT;
+        }
+        if (prefixSql.equalsIgnoreCase("INS")
+                || prefixSql.equalsIgnoreCase("UPD")
+                || prefixSql.equalsIgnoreCase("REP")
+                || prefixSql.equalsIgnoreCase("DEL")) {
+            return STATEMENT_UPDATE;
+        }
+        if (prefixSql.equalsIgnoreCase("ATT")) {
+            return STATEMENT_ATTACH;
+        }
+        if (prefixSql.equalsIgnoreCase("COM")
+                || prefixSql.equalsIgnoreCase("END")) {
+            return STATEMENT_COMMIT;
+        }
+        if (prefixSql.equalsIgnoreCase("ROL")) {
+            return STATEMENT_ABORT;
+        }
+        if (prefixSql.equalsIgnoreCase("BEG")) {
+            return STATEMENT_BEGIN;
         }
+        if (prefixSql.equalsIgnoreCase("PRA")) {
+            return STATEMENT_PRAGMA;
+        }
+        if (prefixSql.equalsIgnoreCase("CRE")
+                || prefixSql.equalsIgnoreCase("DRO")
+                || prefixSql.equalsIgnoreCase("ALT")) {
+            return STATEMENT_DDL;
+        }
+
+        if (prefixSql.equalsIgnoreCase("ANA") || prefixSql.equalsIgnoreCase("DET")) {
+            return STATEMENT_UNPREPARED;
+        }
+
         return STATEMENT_OTHER;
     }
 }

sqlite-android/src/main/jni/sqlite/Android.mk

@@ -26,7 +26,12 @@ sqlite_flags := \
 	-DSQLITE_ENABLE_RTREE=1 \
 	-DSQLITE_OMIT_BUILTIN_TEST \
 	-DSQLITE_OMIT_COMPILEOPTION_DIAGS \
-	-DSQLITE_DEFAULT_FILE_PERMISSIONS=0600
+	-DSQLITE_DEFAULT_FILE_PERMISSIONS=0600 \
+    -DSQLITE_DEFAULT_MEMSTATUS=0 \
+    -DSQLITE_MAX_EXPR_DEPTH=0 \
+    -DSQLITE_USE_ALLOCA \
+    -DSQLITE_ENABLE_BATCH_ATOMIC_WRITE \
+    -O3
 
 LOCAL_CFLAGS += $(sqlite_flags)
 LOCAL_CFLAGS += -Wno-unused-parameter -Wno-int-to-pointer-cast