change ClickHouseConnection constructor for usage driverOptions

Author: argayash

README.md

@@ -25,7 +25,14 @@ $connectionParams = [
     'password' => '',
     'dbname' => 'default',
     'driverClass' => 'FOD\DBALClickHouse\Driver',
-    'wrapperClass' => 'FOD\DBALClickHouse\Connection'
+    'wrapperClass' => 'FOD\DBALClickHouse\Connection',
+    'driverOptions' => [
+        'extremes' => false,
+        'readonly' => true,
+        'max_execution_time' => 30,
+        'enable_http_compression' => 0,
+        'https' => false,
+    ],
 ];
 $conn = \Doctrine\DBAL\DriverManager::getConnection($connectionParams, new \Doctrine\DBAL\Configuration());
 ```

src/ClickHouseConnection.php

@@ -11,7 +11,6 @@
 
 namespace FOD\DBALClickHouse;
 
-use Doctrine\DBAL\ConnectionException;
 use ClickHouseDB\Client as Smi2CHClient;
 use Doctrine\DBAL\ParameterType;
 use Doctrine\DBAL\Platforms\AbstractPlatform;
@@ -36,27 +35,25 @@ class ClickHouseConnection implements \Doctrine\DBAL\Driver\Connection
     /**
      * Connection constructor
      *
+     * @param array $params
      * @param string $username The username to use when connecting.
      * @param string $password The password to use when connecting.
-     * @param string $host
-     * @param int $port
-     * @param string $database
      * @param AbstractPlatform $platform
      */
     public function __construct(
-        $username = 'default',
-        $password = '',
-        $host = 'localhost',
-        $port = 8123,
-        $database = 'default',
+        array $params,
+        $username,
+        $password,
         AbstractPlatform $platform
     ) {
         $this->smi2CHClient = new Smi2CHClient([
-            'host' => $host,
-            'port' => $port,
+            'host' => $params['host'] ?? 'localhost',
+            'port' => $params['port'] ?? 8123,
             'username' => $username,
             'password' => $password,
-            'settings' => ['database' => $database]
+            'settings' => array_merge([
+                'database' => $params['dbname'] ?? 'default',
+            ], $params['driverOptions'] ?? []),
         ]);
 
         $this->platform = $platform;

src/Driver.php

@@ -51,8 +51,7 @@ public function connect(array $params, $user = null, $password = null, array $dr
             throw new ClickHouseException('Connection parameter `dbname` is required');
         }
 
-        return new ClickHouseConnection($user, $password, $params['host'], $params['port'], $params['dbname'],
-            $this->getDatabasePlatform());
+        return new ClickHouseConnection($params, $user, $password, $this->getDatabasePlatform());
     }
 
     /**

tests/CreateConnectionTest.php

@@ -48,6 +48,13 @@ public static function createConnection($params = null)
                 'dbname' => phpunit_ch_dbname,
                 'driverClass' => phpunit_ch_driver_class,
                 'wrapperClass' => phpunit_ch_wrapper_class,
+                'driverOptions' => [
+                    'extremes'                => false,
+                    'readonly'                => true,
+                    'max_execution_time'      => 30,
+                    'enable_http_compression' => 0,
+                    'https'                   => false
+                ],
             ];
         }
         return \Doctrine\DBAL\DriverManager::getConnection($params, new \Doctrine\DBAL\Configuration());