In this pull request, I propose changing the current QueryResult to MysqlResult. This change aligns with the recently introduced MysqlClient from #186, ensuring a consistent "Mysql" prefix throughout.
The name "QueryResult" is not incorrect in this case. However, when working with multiple database clients in one project, it can lead to confusion about which specific "query result" is being referred to. For instance, the https://github.com/clue/reactphp-sqlite project has a similar class named Result, which should also be renamed to something like SqliteResult in the future.
In this pull request, I propose changing the current
QueryResult
toMysqlResult
. This change aligns with the recently introducedMysqlClient
from #186, ensuring a consistent "Mysql" prefix throughout.The name "QueryResult" is not incorrect in this case. However, when working with multiple database clients in one project, it can lead to confusion about which specific "query result" is being referred to. For instance, the https://github.com/clue/reactphp-sqlite project has a similar class named
Result
, which should also be renamed to something likeSqliteResult
in the future.Furthermore, we applied a similar reasoning in https://github.com/reactphp/socket/pull/263 and https://github.com/reactphp/http/pull/417.
Refs #147 Builds on top of #186