Skip to content

Conversation

@jayy-lmao
Copy link
Contributor

Does your PR solve an issue?
Part 5 of fixing #166
(Breaking apart #3449)

@jayy-lmao jayy-lmao changed the title feat: add postgres geometry path feat(postgres): add geometry path Jan 30, 2025
@abonander
Copy link
Collaborator

@jayy-lmao can you resolve the merge conflict?

jthacker and others added 22 commits March 2, 2025 09:15
The readme previously stated that the uuid feature is only for postres but it actually also gates the functionality in mysql and sqlite.
…aunchbadge#3724)

* Add flag to disable automatic loading of .env files

* Update sqlx-cli/src/opt.rs

Co-authored-by: Austin Bonander <austin.bonander@gmail.com>

---------

Co-authored-by: Austin Bonander <austin.bonander@gmail.com>
…y set of tuples (launchbadge#3734)

* throw a warning in tracing so that the empty tuples would be noticed

* use debug assertion to throw a panic in debug mode
…(postgres). (launchbadge#3753)

* feat(sqlx-postgres): use advisory lock to avoid setup race condition

* fix(sqlx-postgres): numeric hex constants not supported before postgres 16
When sqlx-core/src/from_row.rs was updated to implement FromRow for tuples of up to 16 values, a comment was left stating that it was implemented up to tuples of 9 values.
…aunchbadge#3762)

Both actix_web::web::Data and sqlx::PgPool internally wrap an Arc. Thus,
using Data<PgPool> as an extractor in an actix-web route handler results
in two Arcs wrapping the data of interest, which isn't ideal.

Actix-web 4.9.0 introduced a new web::ThinData extractor for cases like
this, where the data is already wrapped in an `Arc` (or is otherwise
similarly cheap and sensible to simply clone), which doesn't wrap the
inner value in a (second) Arc.

Since the new extractor is better suited to the task, suggest it in
place of web::Data when giving an example on how to share a pool.
@abonander abonander merged commit 5c573e1 into launchbadge:main Mar 2, 2025
81 checks passed
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.