Skip to content

Conversation

gcavanunez
Copy link

@gcavanunez gcavanunez commented Oct 8, 2025

Changes

Adds the pdo_sqlite & pdo_pgsql to the matrix release config since pdo_mysql was already included, however for sqlite and pgsql these extensions seemed to be missing.

Been really liking the project! it's super handy thanks for assembling this ya'll

@crazywhalecc
Copy link
Member

crazywhalecc commented Oct 12, 2025

Thank you for contributing this.

The common combination looks good to add pdo_pgsql, but it should also add pgsql extension at the same time.

And the bulk and gnu-bulk has swoole-hook-pgsql and swoole-hook-sqlite has included, it provides pdo_pgsql and pdo_sqlite same functions.

@gcavanunez
Copy link
Author

gcavanunez commented Oct 16, 2025

Added the pgsql to the common, but when checking the extensions loaded was when I noticed that the pdo_sqlite and pod_pgsql weren't being loaded, though it sounds like perhaps that because of the swoole-hook-* or would this be perhaps a bug?

Loaded Extensions

``` echo var_dump(get_loaded_extensions()); ``` ``` array(62) { [0]=> string(4) "Core" [1]=> string(4) "date" [2]=> string(6) "libxml" [3]=> string(7) "openssl" [4]=> string(4) "pcre" [5]=> string(7) "sqlite3" [6]=> string(4) "zlib" [7]=> string(4) "apcu" [8]=> string(6) "bcmath" [9]=> string(3) "bz2" [10]=> string(8) "calendar" [11]=> string(5) "ctype" [12]=> string(4) "curl" [13]=> string(3) "dba" [14]=> string(3) "dom" [15]=> string(7) "sockets" [16]=> string(5) "event" [17]=> string(4) "hash" [18]=> string(8) "fileinfo" [19]=> string(6) "filter" [20]=> string(3) "ftp" [21]=> string(2) "gd" [22]=> string(3) "gmp" [23]=> string(4) "json" [24]=> string(5) "iconv" [25]=> string(3) "SPL" [26]=> string(7) "session" [27]=> string(4) "intl" [28]=> string(8) "standard" [29]=> string(8) "mbstring" [30]=> string(7) "mysqlnd" [31]=> string(6) "mysqli" [32]=> string(13) "opentelemetry" [33]=> string(5) "pcntl" [34]=> string(4) "exif" [35]=> string(4) "imap" [36]=> string(3) "PDO" [37]=> string(9) "pdo_mysql" [38]=> string(5) "pgsql" [39]=> string(4) "Phar" [40]=> string(5) "posix" [41]=> string(8) "protobuf" [42]=> string(6) "random" [43]=> string(8) "readline" [44]=> string(5) "redis" [45]=> string(10) "Reflection" [46]=> string(7) "imagick" [47]=> string(5) "shmop" [48]=> string(9) "SimpleXML" [49]=> string(4) "soap" [50]=> string(6) "sodium" [51]=> string(6) "swoole" [52]=> string(7) "sysvmsg" [53]=> string(7) "sysvsem" [54]=> string(7) "sysvshm" [55]=> string(9) "tokenizer" [56]=> string(3) "xml" [57]=> string(9) "xmlreader" [58]=> string(9) "xmlwriter" [59]=> string(3) "xsl" [60]=> string(3) "zip" [61]=> string(12) "Zend OPcache" } ```

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants