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

Bugfix/1168 #1169

Merged
merged 1 commit into from
Mar 26, 2023
Merged

Bugfix/1168 #1169

merged 1 commit into from
Mar 26, 2023

Conversation

fnc12
Copy link
Owner

@fnc12 fnc12 commented Mar 26, 2023

Fixing #1168

@fnc12 fnc12 changed the base branch from master to dev March 26, 2023 13:40
@fnc12 fnc12 requested a review from trueqbit March 26, 2023 13:40
Copy link
Collaborator

@trueqbit trueqbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Argh, stupid mistake!

I suggest we use the same C string literal "SELECT name FROM sqlite_master WHERE type='table'" as in storage_base::table_names() below in order to benefit from string pooling.

@fnc12
Copy link
Owner Author

fnc12 commented Mar 26, 2023

@trueqbit well it is not yours it is mine cause all you did is refactored existing quotes

@trueqbit
Copy link
Collaborator

@trueqbit well it is not yours it is mine cause all you did is refactored existing quotes

I have to disappoint you - it was my mistake 🤣, see #d470ca12b062

@fnc12
Copy link
Owner Author

fnc12 commented Mar 26, 2023

@trueqbit well it is not yours it is mine cause all you did is refactored existing quotes

I have to disappoint you - it was my mistake 🤣, see #d470ca12b062

it shows me 404. Even if it is made by you - no problem.

@fnc12 fnc12 merged commit 24a7ea2 into dev Mar 26, 2023
@trueqbit trueqbit linked an issue Mar 26, 2023 that may be closed by this pull request
@trueqbit trueqbit deleted the bugfix/1168 branch April 12, 2023 09:05
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]>
@trueqbit trueqbit linked an issue May 17, 2024 that may be closed by this pull request
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.

SQLITE_DQS Compile Option Can we just use single quote?
3 participants