Prosím o otestování Nette Database 4.0 RC

@kminekmatej Podle kódu NDB verze 3.2 to vypadá, že reflection: bylo v průběhu času nahrazeno za conventions: a drženo jen z důvodu zpětné kompatibility – viz kód.
Pokud čtu kód správně, mohlo by stačit nahradit reflection: conventional za conventions: Static.

DB driver

My vlastní DB driver používáme – vypadá nějak takto:

<?php

declare(strict_types=1);

namespace App\Infra\Database\Drivers;

use Nette\Database\Driver;
use Nette\Database\Connection;
use Nette\Database\Drivers\MySqlDriver;

final class AcmeMysqlDbDriver extends MySqlDriver {

	/** @var bool Does this driver support setting isolation level for transaction? */
	public const SUPPORTS_ISOLATION_LEVEL = true;

	/** @var int Max execution time for each query (in seconds) within a DB connection. */
	private const MAX_STATEMENT_TIME = 60 * 10;

	/**
	 * @param array{
	 *     charset?: string,
	 *     sqlmode?: string,
	 *     supportBooleans?: bool,
	 *   } $options
	 */
	public function initialize(Connection $connection, array $options): void {

		parent::initialize($connection, $options);
		$connection->query(sprintf(
			'SET SESSION max_statement_time=%d;',
			self::MAX_STATEMENT_TIME,
		));

		// https://jira.mariadb.org/browse/MDEV-20083
		// https://mariadb.com/kb/en/conversion-of-big-in-predicates-into-subqueries/
		// Disable this "optimization", because our queries are MUCH faster (in
		// another words: they finish and do not hang.)
		$connection->query('SET SESSION in_predicate_conversion_threshold=0;');

		// NOTE: Cannot use PHP8,1 first-class callable syntax as we sometimes
		// need to serialize DB connection - e.g. in tests - where PHP
		// would complain about serializing a closure.
		$connection->setRowNormalizer(
			[BothWaysDbRowKeysNormalizer::class, 'normalizeRow'],
		);

	}

	public function isSupported(string $item): bool {

		return in_array($item, [
			Driver::SupportSelectUngroupedColumns,
			Driver::SupportMultiColumnAsOrCond,
			Driver::SupportSubselect,
		], true);

	}

}

Jak je vidět, využíváme inicializaci driveru pro nastavení potřebných věcí pro dané spojení – doufám, že tahle možnost bude v 4.0 zachovaná. V https://github.com/…Explorer.php#L35 vidím možný list $onConnect callbacků, kam by se toto dalo strčit, ale jelikož není zřejmé, jak (kdy přesně) se v rámci Nette DI bude připojení ve finále inicializovat, tak netuším, zda budu mít možnost nějaké $onConnect[] = ... udělat ještě před tím, než se zavolá connect() – snad ano.

Row normalizer

Pokud koukám správně, tak v aktuálním masteru také vidím, že je „row normalized“ deprecated: [...] is deprecated, configure 'convert*' options instead., viz https://github.com/…Explorer.php#L202

Row normalizer používáme ke modifikování dat array (slovníku) daného řádku – v našem případě hodnotu pod klíčem (sloupcem) this_is_some_column chceme mít k dispozici i pod klíčem thisIsSomeColumn, takže potřebujeme duplikovat klíče se změněným case (což bychom btw. nemuseli duplikovat, kdyby se kdysi nezavrhly věci typu https://github.com/…se/issues/90). Čili jelikož potřebujeme modifikovat celý array, tak nějaké:

class Explorer
{
	...
	private const TypeConverterOptions = ['convertBoolean', 'convertDateTime', 'convertDecimal', 'newDateTime'];
	...
}

… je nám docela k ničemu. Bylo by fajn udržet možnost si připravit data řádku tak, jak my potřebujeme. Pokud se možnost toto dělat někde v 4.0 schovává, tak jsem to nebyl schopen rozpoznat – a kdyby to nešlo, tak to bude upgrade na Nette Database 4.0 silně blokovat.

Zmizelé getDsn()

V naší appce databázi shardujeme – máme hlavní DB + zcela separátně jiné, menší databáze. Pro spojení k tomuhle všemu používáme Nette Database/Explorer a kvůli tomu musíme v kódu managovat vytváření instancí spojení a Explorerů pro každý shard.

K tomu se nám náramně hodí getDsn(), které je aktuálně k dispozici (např. jako klíč pro kešování zmiňovaných instancí, ale i další věci, které jsou složitější k vysvětlení). Hlasoval bych pro zachování možnosti získat z PDO driveru zpět DSN – které tam, jestli koukám správně, stejně bude moct být: https://github.com/…O/Driver.php#L24

Editoval smuuf (9. 10. 2024 10:13)

Jo, getDsn() bych rovněž potřeboval