Use native IBM DB2 11 syntax for Top-N queries

doctrine · Dec 31, 2021 · 25e964f · 25e964f
1 parent c5154dc
commit 25e964f
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 52 deletions.
diff --git a/src/Platforms/DB2Platform.php b/src/Platforms/DB2Platform.php
@@ -641,30 +641,6 @@ public function getTemporaryTableName(string $tableName): string
         return 'SESSION.' . $tableName;
     }
 
-    protected function doModifyLimitQuery(string $query, ?int $limit, int $offset): string
-    {
-        $where = [];
-
-        if ($offset > 0) {
-            $where[] = sprintf('db22.DC_ROWNUM >= %d', $offset + 1);
-        }
-
-        if ($limit !== null) {
-            $where[] = sprintf('db22.DC_ROWNUM <= %d', $offset + $limit);
-        }
-
-        if (empty($where)) {
-            return $query;
-        }
-
-        // Todo OVER() needs ORDER BY data!
-        return sprintf(
-            'SELECT db22.* FROM (SELECT db21.*, ROW_NUMBER() OVER() AS DC_ROWNUM FROM (%s) db21) db22 WHERE %s',
-            $query,
-            implode(' AND ', $where)
-        );
-    }
-
     public function getLocateExpression(string $string, string $substring, ?string $start = null): string
     {
         if ($start === null) {

diff --git a/tests/Platforms/DB2PlatformTest.php b/tests/Platforms/DB2PlatformTest.php
@@ -392,34 +392,6 @@ public function testGeneratesSQLSnippets(): void
         self::assertEquals('SUBSTR(column, 5, 2)', $this->platform->getSubstringExpression('column', '5', '2'));
     }
 
-    public function testModifiesLimitQuery(): void
-    {
-        self::assertEquals(
-            'SELECT * FROM user',
-            $this->platform->modifyLimitQuery('SELECT * FROM user', null, 0)
-        );
-
-        self::assertEquals(
-            'SELECT db22.* FROM ('
-                . 'SELECT db21.*, ROW_NUMBER() OVER() AS DC_ROWNUM FROM (SELECT * FROM user) db21'
-                . ') db22 WHERE db22.DC_ROWNUM <= 10',
-            $this->platform->modifyLimitQuery('SELECT * FROM user', 10)
-        );
-
-        self::assertEquals(
-            'SELECT db22.* FROM (SELECT db21.*, ROW_NUMBER() OVER() AS DC_ROWNUM FROM ('
-                . 'SELECT * FROM user) db21'
-                . ') db22 WHERE db22.DC_ROWNUM >= 6 AND db22.DC_ROWNUM <= 15',
-            $this->platform->modifyLimitQuery('SELECT * FROM user', 10, 5)
-        );
-        self::assertEquals(
-            'SELECT db22.* FROM ('
-                . 'SELECT db21.*, ROW_NUMBER() OVER() AS DC_ROWNUM FROM (SELECT * FROM user) db21'
-                . ') db22 WHERE db22.DC_ROWNUM >= 6 AND db22.DC_ROWNUM <= 5',
-            $this->platform->modifyLimitQuery('SELECT * FROM user', 0, 5)
-        );
-    }
-
     public function testSupportsIdentityColumns(): void
     {
         self::assertTrue($this->platform->supportsIdentityColumns());