Merge pull request #567 from pietroalbini/overrides-bigint

limits: allow overriding memory limits to more than 4GB

Author: pietroalbini

src/db/migrate.rs

@@ -269,6 +269,17 @@ fn migrate_inner(version: Option<Version>, conn: &Connection, apply_mode: ApplyM
             // downgrade query
             "DROP TABLE blacklisted_crates;"
         ),
+        migration!(
+            context,
+            // version
+            7,
+            // description
+            "Allow memory limits of more than 4 GB",
+            // upgrade query
+            "ALTER TABLE sandbox_overrides ALTER COLUMN max_memory_bytes TYPE BIGINT;",
+            // downgrade query
+            "ALTER TABLE sandbox_overrides ALTER COLUMN max_memory_bytes TYPE INTEGER;"
+        ),
     ];
 
     for migration in migrations {

src/docbuilder/limits.rs

@@ -31,7 +31,7 @@ impl Limits {
         )?;
         if !res.is_empty() {
             let row = res.get(0);
-            if let Some(memory) = row.get::<_, Option<i32>>("max_memory_bytes") {
+            if let Some(memory) = row.get::<_, Option<i64>>("max_memory_bytes") {
                 limits.memory = memory as usize;
             }
             if let Some(timeout) = row.get::<_, Option<i32>>("timeout_seconds") {