[pull] main from immich-app:main - #966
Merged
Merged
Conversation
… is not visible (#30947)
…30278) Co-authored-by: priyanshuANDcoad <priyanshu23154050@akgec.ac.in>
…le locks (#30851) * fix(server): kill the child process when a spawned duplex stream is destroyed A failed database backup left pg_dump running. The stream was destroyed but the child was not, so the dump sat idle in transaction holding AccessShareLock on every table until someone terminated it by hand. One report had it survive five days and block DDL from another application on the same Postgres. Destroying the duplex now kills the child, which also covers pipeline() tearing every stream down when one of them fails. The backup service additionally holds references to the streams it spawns so the error path can destroy them. createWriteStream can throw before pipeline() takes ownership, which is how the reported ENOENT on an unmounted backup volume left nothing referencing the running pg_dump. * chore(server): drop the explanatory comments from the backup teardown fix Requested in review. The reasoning lives in the commit message and the PR body, which is where it belongs.
* Added the proper toast for each scenario in the if/else statement instead of only having toastManager.primary() for all scenarios. * Created a test file for album.service.ts. * chore: cleanup --------- Co-authored-by: Daniel Dietzler <mail@ddietzler.dev>
* fix(web): map without set start/end dates * fix(web): invalid timeline map options
Update remote-machine-learning.md To include the default URL, should someone need it (as the default entry is not easily recreated otherwise).
Fix external library statistics for excluded assets
* Update sorting Updates the sorting to sort 1. First by timeCreatedAt, but then 2. By filename. * Push tsts * Sync SQL * Formatting fixes --------- Co-authored-by: Daniel Dietzler <36593685+danieldietzler@users.noreply.github.com>
github-actions
Bot
added
documentation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
12 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )