Merge pull request #609 from utopia-php/primary-bigint

Primary bigint

Author: abnegate

src/Database/Adapter/MariaDB.php

@@ -155,7 +155,7 @@ public function createCollection(string $name, array $attributes = [], array $in
 
         $collection = "
 			CREATE TABLE {$this->getSQLTable($id)} (
-				_id INT(11) UNSIGNED NOT NULL AUTO_INCREMENT,
+				_id BIGINT UNSIGNED NOT NULL AUTO_INCREMENT,
 				_uid VARCHAR(255) NOT NULL,
 				_createdAt DATETIME(3) DEFAULT NULL,
 				_updatedAt DATETIME(3) DEFAULT NULL,
@@ -186,7 +186,7 @@ public function createCollection(string $name, array $attributes = [], array $in
 
         $permissions = "
             CREATE TABLE {$this->getSQLTable($id . '_perms')} (
-                _id int(11) UNSIGNED NOT NULL AUTO_INCREMENT,
+                _id BIGINT UNSIGNED NOT NULL AUTO_INCREMENT,
                 _type VARCHAR(12) NOT NULL,
                 _permission VARCHAR(255) NOT NULL,
                 _document VARCHAR(255) NOT NULL,

src/Database/Adapter/Postgres.php

@@ -232,14 +232,13 @@ public function createCollection(string $name, array $attributes = [], array $in
         $sqlTenant = $this->sharedTables ? '_tenant INTEGER DEFAULT NULL,' : '';
         $collection = "
             CREATE TABLE {$this->getSQLTable($id)} (
-                _id SERIAL NOT NULL,
+                _id BIGINT GENERATED BY DEFAULT AS IDENTITY PRIMARY KEY,
                 _uid VARCHAR(255) NOT NULL,
                 " . $sqlTenant . "
                 \"_createdAt\" TIMESTAMP(3) DEFAULT NULL,
                 \"_updatedAt\" TIMESTAMP(3) DEFAULT NULL,
-                _permissions TEXT DEFAULT NULL,
                 " . \implode(' ', $attributeStrings) . "
-                PRIMARY KEY (_id)
+                _permissions TEXT DEFAULT NULL
             );
         ";
 
@@ -262,13 +261,12 @@ public function createCollection(string $name, array $attributes = [], array $in
 
         $permissions = "
             CREATE TABLE {$this->getSQLTable($id . '_perms')} (
-                _id SERIAL NOT NULL,
+                _id BIGINT GENERATED BY DEFAULT AS IDENTITY PRIMARY KEY,
                 _tenant INTEGER DEFAULT NULL,
                 _type VARCHAR(12) NOT NULL,
                 _permission VARCHAR(255) NOT NULL,
-                _document VARCHAR(255) NOT NULL,
-                PRIMARY KEY (_id)
-            );   
+                _document VARCHAR(255) NOT NULL
+            );
         ";
 
         if ($this->sharedTables) {

tests/e2e/Adapter/Scopes/DocumentTests.php

@@ -22,6 +22,31 @@
 
 trait DocumentTests
 {
+    public function testBigintSequence(): void
+    {
+        /** @var Database $database */
+        $database = static::getDatabase();
+
+        $database->createCollection(__FUNCTION__);
+
+        $sequence = 5_000_000_000_000_000;
+
+        $document = $database->createDocument(__FUNCTION__, new Document([
+            '$sequence' => (string)$sequence,
+            '$permissions' => [
+                Permission::read(Role::any()),
+            ],
+        ]));
+
+        $this->assertEquals((string)$sequence, $document->getSequence());
+
+        $document = $database->getDocument(__FUNCTION__, $document->getId());
+        $this->assertEquals((string)$sequence, $document->getSequence());
+
+        $document = $database->findOne(__FUNCTION__, [Query::equal('$sequence', [(string)$sequence])]);
+        $this->assertEquals((string)$sequence, $document->getSequence());
+    }
+
     public function testCreateDocument(): Document
     {
         /** @var Database $database */