Revert "MySqlDriver driver uses subqueries (#265)"

This reverts commit e89a34a.

"MySQL does not support LIMIT in subqueries for certain subquery operators" dev.mysql.com/doc/refman/8.0/en/subquery-restrictions.html

Author: dg

src/Database/Drivers/MySqlDriver.php

@@ -27,9 +27,6 @@ class MySqlDriver implements Nette\Database\Driver
 	/** @var Nette\Database\Connection */
 	private $connection;
 
-	/** @var string */
-	private $version;
-
 
 	/**
 	 * Driver options:
@@ -39,8 +36,8 @@ class MySqlDriver implements Nette\Database\Driver
 	public function initialize(Nette\Database\Connection $connection, array $options): void
 	{
 		$this->connection = $connection;
-		$this->version = $connection->getPdo()->getAttribute(\PDO::ATTR_SERVER_VERSION);
-		$charset = $options['charset'] ?? (version_compare($this->version, '5.5.3', '>=') ? 'utf8mb4' : 'utf8');
+		$charset = $options['charset']
+			?? (version_compare($connection->getPdo()->getAttribute(\PDO::ATTR_SERVER_VERSION), '5.5.3', '>=') ? 'utf8mb4' : 'utf8');
 		if ($charset) {
 			$connection->query('SET NAMES ?', $charset);
 		}
@@ -206,10 +203,7 @@ public function isSupported(string $item): bool
 		// MULTI_COLUMN_AS_OR_COND due to mysql bugs:
 		// - http://bugs.mysql.com/bug.php?id=31188
 		// - http://bugs.mysql.com/bug.php?id=35819
-		// SUPPORT_SUBSELECT is slow before 5.7
-		// - http://mysqlserverteam.com/derived-tables-in-mysql-5-7/
-		return $item === self::SUPPORT_SELECT_UNGROUPED_COLUMNS
-			|| $item === self::SUPPORT_MULTI_COLUMN_AS_OR_COND
-			|| ($item === self::SUPPORT_SUBSELECT && version_compare($this->version, '5.7', '>='));
+		// and more.
+		return $item === self::SUPPORT_SELECT_UNGROUPED_COLUMNS || $item === self::SUPPORT_MULTI_COLUMN_AS_OR_COND;
 	}
 }

tests/Database/Table/SqlBuilder.addWhere().phpt

@@ -38,7 +38,10 @@ test('?name', function () use ($explorer) {
 test('test Selection as a parameter', function () use ($explorer) {
 	$sqlBuilder = new SqlBuilder('book', $explorer);
 	$sqlBuilder->addWhere('id', $explorer->table('book'));
-	Assert::equal(reformat(['SELECT * FROM [book] WHERE ([id] IN (SELECT [id] FROM [book]))']), $sqlBuilder->buildSelectQuery());
+	Assert::equal(reformat([
+		'mysql' => 'SELECT * FROM `book` WHERE (`id` IN (?))',
+		'SELECT * FROM [book] WHERE ([id] IN (SELECT [id] FROM [book]))',
+	]), $sqlBuilder->buildSelectQuery());
 });
 
 
@@ -47,6 +50,7 @@ test('test more Selection as a parameter', function () use ($explorer) {
 	$sqlBuilder->addWhere('id', $explorer->table('book'));
 	$sqlBuilder->addWhere('id', $explorer->table('book_tag')->select('book_id'));
 	Assert::equal(reformat([
+		'mysql' => 'SELECT * FROM `book` WHERE (`id` IN (?)) AND (`id` IN (?))',
 		'SELECT * FROM [book] WHERE ([id] IN (SELECT [id] FROM [book])) AND ([id] IN (SELECT [book_id] FROM [book_tag]))',
 	]), $sqlBuilder->buildSelectQuery());
 });
@@ -56,6 +60,7 @@ test('test more Selection as one of more argument', function () use ($explorer)
 	$sqlBuilder = new SqlBuilder('book', $explorer);
 	$sqlBuilder->addWhere('id ? AND id ?', $explorer->table('book')->where('id', 2), $explorer->table('book_tag')->select('book_id'));
 	Assert::equal(reformat([
+		'mysql' => 'SELECT * FROM `book` WHERE (`id` IN (?) AND `id` IN (?))',
 		'SELECT * FROM [book] WHERE ([id] IN (SELECT [id] FROM [book] WHERE ([id] = ?)) AND [id] IN (SELECT [book_id] FROM [book_tag]))',
 	]), $sqlBuilder->buildSelectQuery());
 });
@@ -77,6 +82,7 @@ test('test Selection with parameters as a parameter', function () use ($explorer
 	$sqlBuilder->addWhere('id', $explorer->table('book')->having('COUNT(:book_tag.tag_id) >', 1));
 	$schemaSupported = $explorer->getConnection()->getDriver()->isSupported(Driver::SUPPORT_SCHEMA);
 	Assert::equal(reformat([
+		'mysql' => 'SELECT * FROM `book` WHERE (`id` IN (?))',
 		'SELECT * FROM [book] WHERE ([id] IN (SELECT [id] FROM [book] LEFT JOIN ' . ($schemaSupported ? '[public].[book_tag] ' : '') . '[book_tag] ON [book].[id] = [book_tag].[book_id] HAVING COUNT([book_tag].[tag_id]) > ?))',
 	]), $sqlBuilder->buildSelectQuery());
 	Assert::count(1, $sqlBuilder->getParameters());
@@ -87,6 +93,7 @@ test('test Selection with column as a parameter', function () use ($explorer) {
 	$sqlBuilder = new SqlBuilder('book', $explorer);
 	$sqlBuilder->addWhere('id', $explorer->table('book')->select('id'));
 	Assert::equal(reformat([
+		'mysql' => 'SELECT * FROM `book` WHERE (`id` IN (?))',
 		'SELECT * FROM [book] WHERE ([id] IN (SELECT [id] FROM [book]))',
 	]), $sqlBuilder->buildSelectQuery());
 });
@@ -96,6 +103,7 @@ test('test multiple placeholder parameter', function () use ($explorer) {
 	$sqlBuilder = new SqlBuilder('book', $explorer);
 	$sqlBuilder->addWhere('id ? OR id ?', null, $explorer->table('book'));
 	Assert::equal(reformat([
+		'mysql' => 'SELECT * FROM `book` WHERE (`id` IS NULL OR `id` IN (?))',
 		'SELECT * FROM [book] WHERE ([id] IS NULL OR [id] IN (SELECT [id] FROM [book]))',
 	]), $sqlBuilder->buildSelectQuery());
 });
@@ -168,6 +176,7 @@ test('tests NOT', function () use ($explorer) {
 	$sqlBuilder->addWhere('id NOT', null);
 	$sqlBuilder->addWhere('id NOT', $explorer->table('book')->select('id'));
 	Assert::equal(reformat([
+		'mysql' => 'SELECT * FROM `book` WHERE (`id` NOT IN (?)) AND (`id` IS NOT NULL) AND (`id` NOT IN (?))',
 		'SELECT * FROM [book] WHERE ([id] NOT IN (?)) AND ([id] IS NOT NULL) AND ([id] NOT IN (SELECT [id] FROM [book]))',
 	]), $sqlBuilder->buildSelectQuery());
 });