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

Made connection holder usable in multithreaded contexts #1054

Merged
merged 3 commits into from
Jun 16, 2022

Conversation

trueqbit
Copy link
Collaborator

Addresses #1053.

dev/connection_holder.h Outdated Show resolved Hide resolved
@fnc12 fnc12 merged commit c01132b into fnc12:dev Jun 16, 2022
@fnc12
Copy link
Owner

fnc12 commented Jun 16, 2022

thanks @trueqbit

@trueqbit trueqbit deleted the atomic_connref branch January 4, 2023 19:18
tserong added a commit to tserong/ceph that referenced this pull request Sep 19, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which I need for thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
storage.sync_schema() breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Sep 19, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which I need for thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
storage.sync_schema() breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Sep 20, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which I need for thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
storage.sync_schema() breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Sep 20, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which I need for thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
storage.sync_schema() breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Sep 25, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which I need for thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
storage.sync_schema() breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Oct 14, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which I need for thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
storage.sync_schema() breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Oct 17, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which I need for thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
storage.sync_schema() breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Oct 17, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which we want to ensure thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
`storage.sync_schema()` breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Oct 18, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which we want to ensure thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
`storage.sync_schema()` breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Oct 19, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which we want to ensure thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
`storage.sync_schema()` breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Oct 19, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which we want to ensure thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
`storage.sync_schema()` breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
tserong added a commit to tserong/ceph that referenced this pull request Oct 25, 2023
sqlite_orm v1.8.2 has fnc12/sqlite_orm#1054,
which we want to ensure thread safety.  Unfortunately it's missing
fnc12/sqlite_orm#1169, without which
`storage.sync_schema()` breaks (or at least gives us nasty errors)
due to incorrect quotes.  In order to pick up the latter, I've
forked sqlite_orm into the aquarist-labs org, and made a branch
"v1.8.2-s3gw", which is upstream v1.8.2 plus a cherry pick of that
additional fix.

Signed-off-by: Tim Serong <[email protected]>
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.

4 participants