Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Only check first row if no changes required #552

Merged
merged 3 commits into from
Jun 19, 2024

Conversation

BobdenOs
Copy link
Contributor

When post processing blobs in SQLite stop walking the data as early as possible. If the first row shows that there are not columns to be changed return false and skip all following rows.

@BobdenOs BobdenOs added the next release pr to be checked for next release label Jun 7, 2024
@johannes-vogel johannes-vogel enabled auto-merge (squash) June 19, 2024 08:24
@johannes-vogel johannes-vogel merged commit 39b0b85 into main Jun 19, 2024
4 of 6 checks passed
@johannes-vogel johannes-vogel deleted the perf/change-stream branch June 19, 2024 08:26
@cap-bots cap-bots mentioned this pull request Jun 18, 2024
johannes-vogel added a commit that referenced this pull request Jun 19, 2024
🤖 I have created a release *beep* *boop*
---


<details><summary>db-service: 1.10.1</summary>

##
[1.10.1](db-service-v1.10.0...db-service-v1.10.1)
(2024-06-19)


### Fixed

* Only check first row if no changes required
([#552](#552))
([39b0b85](39b0b85))
</details>

<details><summary>sqlite: 1.7.2</summary>

##
[1.7.2](sqlite-v1.7.1...sqlite-v1.7.2)
(2024-06-19)


### Fixed

* **deps:** update dependency better-sqlite3 to v11
([#669](#669))
([7167ec5](7167ec5))
</details>

<details><summary>hana: 1.0.0</summary>

##
[1.0.0](hana-v0.5.0...hana-v1.0.0)
(2024-06-19)


### Fixed

* Binary columns now return as Buffer for HANAService
([#689](#689))
([179bd92](179bd92))
* Improve error message for disconnected connections
([#678](#678))
([eb4ef37](eb4ef37))
* insertion of arrayed elements
([#677](#677))
([92cf307](92cf307))
* insertion of large decimals
([#686](#686))
([ae8abff](ae8abff))
* Support static conditions inside unmanaged associations
([#682](#682))
([e17ddfd](e17ddfd))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: Johannes Vogel <[email protected]>
@cap-bots cap-bots mentioned this pull request Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next release pr to be checked for next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants