Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:128446
Date: Mon, 11 Aug 2025 21:35:01 +0200
To: internals@lists.php.net
Message-ID: <f7e65a8e68e68cc551c007dedd7e607e69f1c390@hey.com>
Subject: [PHP-DEV] PDO\Sqlite: Add transaction mode attribute
Precedence: list
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_689a45e61876b_8d2012015139a";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
From: samuel@archte.ch (=?UTF-8?B?U2FtdWVsIMWgdGFuY2w=?=)


----==_mimepart_689a45e61876b_8d2012015139a
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

Hi, I've opened a PR adding a new attribute to PDO\Sqlite:
https://github.com/php/php-src/pull/19317 <https://github.com/php/php-
src/pull/19317?notification_referrer_id=NT_kwDOAfgLhrQxNzkxMjcwMjYzOTozMzAzMzA5NA#issuecomment-
3176177745>

The change is explained in depth in the PR, but in short, it makes it
possible to use immediate/exclusive transaction modes, rather than the
default deferred mode, when using $pdo->beginTransaction(). Deferred
mode has this particular behavior where if your transaction first reads
before writing, it will only wait for other transactions to finish while
acquiring the initial shared lock. Then when it tries to upgrade to a
write lock it just immediately aborts with SQLITE_BUSY, regardless of
your busy_timeout setting.

This is a real pain point with using SQLite in production. I've
addressed it by adding a new attribute that changes how the PDO driver
begins the transaction.

Is my understanding correct that this doesn't need an RFC? And if that's
the case, is this PR not subject to the PHP 8.5 feature freeze happening
this week?

The changes are pretty basic so I'd appreciate any reviews that could
move this forward. Would love for this to land in PHP 8.5.

----==_mimepart_689a45e61876b_8d2012015139a
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<!DOCTYPE html>
<html>
  <head>
    <meta charset=3D"utf-8">
    <meta http-equiv=3D"Content-Type" content=3D"text/html; charset=3Dutf=
-8" />
    <meta name=3D"viewport" id=3D"viewport" content=3D"width=3Ddevice-wid=
th,minimum-scale=3D1.0,maximum-scale=3D10.0,initial-scale=3D1.0" />

    <style>
      html {
  -webkit-text-size-adjust: 100%;
  -ms-text-size-adjust: 100%;
}

h1 {
  font-size: 1.3em;
  line-height: 1.2;
  margin: 0;
}

ul,
ol {
  margin: 0;
  padding: 0;
}

ul li,
ol li,
li li {
  margin: 0 0 0 36px;
}

[dir=3Drtl] li {
  margin: 0 18px 0 0;
}

blockquote {
  border-color: #dfdee1; /* --color--border */
  border-style: solid;
  border-width: 0 0 0 1px;
  margin: 0;
  padding: 0 0 0 1em;
}

[dir=3Drtl] blockquote,
blockquote[dir=3Drtl] {
  border-width: 0 1px 0 0;
  padding: 0 1em 0 0;
}

pre {
  font-family: "SFMono-Regular", Consolas, "Liberation Mono", Menlo, Cour=
ier, monospace; /* --font-family--mono */
  font-size: 0.9em;
  margin: 0;
  padding: 1rem;
  background-color: #f3f1ef; /* --color-bg--surface */
  white-space: pre-wrap;
  word-wrap: break-word;
  overflow: visible;
}

.message-content {
  font-family: -apple-system, BlinkMacSystemFont, Aptos, Roboto, "Segoe U=
I", Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji", =
"Segoe UI Symbol"; /* --font-family */
  line-height: 1.4;
}

.attachment {
  display: inline-block;
  margin: 0;
  padding: 0;
}

.attachment__caption {
  padding: 0;
  text-align: center;
}

.attachment__caption a[href] {
  text-decoration: none;
  color: #333333;
}

.attachment--preview {
  width: 100%;
  text-align: center;
  margin: 0.625em 0;
}

.attachment--preview img {
  border: 1px solid #dfdee1; /* --color--border */
  vertical-align: middle;
  width: auto;
  max-width: 100%;
  max-height: 640px;
}

.attachment--preview .attachment__caption {
  color: #716d7b;  /* --color-txt--subtle */
  font-size: 0.85em;
  margin-top: 0.625em;
}

.attachment--file {
  color: #282138; /* --color-txt */
  line-height: 1;
  margin: 0 2px 2px 0;
  padding: 0.4em 1em;
  border: 1px solid #dfdee1; /* --color--border */
  border-radius: 5px;
}

.permalink {
  color: inherit;
}

.txt--subtle {
  color: #716d7b;  /* --color-txt--subtle */
}

.txt--xx-small {
  font-size: 14px;
}

.flush {
  margin: 0;
  padding: 0;
}

.push--bottom {
  margin-bottom: 8px;
}

.border--top {
  border-top: 1px solid #ECE9E6; /* --color-border--solid */
}

.btn {
  padding: 0.2em 0.4em;
  font-weight: 500;
  text-decoration: none;
  border-radius: 3rem;
  white-space: nowrap;
  background: #5522FA; /* --color-tertiary */
  border-color: #5522FA;
  color: #ffffff;
}

.btn--email {
  display: inline-block;
  text-align: center;
  font-weight: 500;
  font-size: 1em;
  text-decoration: none;
  border-radius: 2em;
  white-space: nowrap;
  background: #5522FA; /* --color-tertiary */
  border-color: #5522FA;
  color: #ffffff;
  border-top: 0.3em solid #5522FA;
  border-left: 1em solid #5522FA;
  border-bottom: 0.3em solid #5522FA;
  border-right: 1em solid #5522FA;
}

.shaded {
  padding: 1em;
  border-radius: 4px;
  background-color: #f6f5f3; /* --color-bg--surface */
  border: 1px solid #dfdee1; /* --color--border */
}

.shaded--blue {
  background-color: rgba(80, 162, 255, 0.2); /* --rgb-blue 0.2 */
}

.shaded--red {
  background-color: rgba(255, 120, 120, 0.2); /* --rgb-red 0.2 */
}

.strikethrough {
  text-decoration: line-through;
}

    </style>
  </head>

  <body>
    <div class=3D"message-content">
      <div class=3D"trix-content">
  <div>Hi, I&#39;ve opened a PR adding a new attribute to PDO\Sqlite: <a =
href=3D"https://github.com/php/php-src/pull/19317?notification_referrer_i=
d=3DNT_kwDOAfgLhrQxNzkxMjcwMjYzOTozMzAzMzA5NA#issuecomment-3176177745">ht=
tps://github.com/php/php-src/pull/19317</a><br><br>The change is explaine=
d in depth in the PR, but in short, it makes it possible to use immediate=
/exclusive transaction modes, rather than the default deferred mode, when=
 using $pdo-&gt;beginTransaction(). Deferred mode has this particular beh=
avior where if your transaction first reads before writing, it will only =
wait for other transactions to finish while acquiring the initial shared =
lock. Then when it tries to upgrade to a write lock it just immediately a=
borts with SQLITE_BUSY, regardless of your busy_timeout setting.<br><br>T=
his is a real pain point with using SQLite in production. I&#39;ve addres=
sed it by adding a new attribute that changes how the PDO driver begins t=
he transaction.<br><br>Is my understanding correct that this doesn&#39;t =
need an RFC? And if that&#39;s the case, is this PR not subject to the PH=
P 8.5 feature freeze happening this week?<br><br>The changes are pretty b=
asic so I&#39;d appreciate any reviews that could move this forward. Woul=
d love for this to land in PHP 8.5.</div>
</div>


    </div>
  </body>
</html>

----==_mimepart_689a45e61876b_8d2012015139a--