Mercurial > hg > orthanc-databases
annotate Framework/PostgreSQL/PostgreSQLTransaction.cpp @ 522:c49136b34891 large-queries tip
use a prepared statement for InsertOrUpdateMetadata
| author | Alain Mazy <am@orthanc.team> |
|---|---|
| date | Fri, 05 Jul 2024 09:15:54 +0200 |
| parents | 54d518dcd74a |
| children |
| rev | line source |
|---|---|
| 0 | 1 /** |
| 2 * Orthanc - A Lightweight, RESTful DICOM Store | |
| 3 * Copyright (C) 2012-2016 Sebastien Jodogne, Medical Physics | |
| 4 * Department, University Hospital of Liege, Belgium | |
|
507
54d518dcd74a
updated copyright, as Orthanc Team now replaces Osimis
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
497
diff
changeset
|
5 * Copyright (C) 2017-2023 Osimis S.A., Belgium |
|
54d518dcd74a
updated copyright, as Orthanc Team now replaces Osimis
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
497
diff
changeset
|
6 * Copyright (C) 2024-2024 Orthanc Team SRL, Belgium |
|
459
ecd0b719cff5
update year to 2024
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
389
diff
changeset
|
7 * Copyright (C) 2021-2024 Sebastien Jodogne, ICTEAM UCLouvain, Belgium |
| 0 | 8 * |
| 9 * This program is free software: you can redistribute it and/or | |
| 10 * modify it under the terms of the GNU Affero General Public License | |
| 11 * as published by the Free Software Foundation, either version 3 of | |
| 12 * the License, or (at your option) any later version. | |
| 13 * | |
| 14 * This program is distributed in the hope that it will be useful, but | |
| 15 * WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 16 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
| 17 * Affero General Public License for more details. | |
| 18 * | |
| 19 * You should have received a copy of the GNU Affero General Public License | |
| 20 * along with this program. If not, see <http://www.gnu.org/licenses/>. | |
| 21 **/ | |
| 22 | |
| 23 | |
| 24 #include "PostgreSQLTransaction.h" | |
| 25 | |
| 26 #include "PostgreSQLStatement.h" | |
| 27 | |
|
157
275e14f57f1e
replacing deprecated std::auto_ptr by std::unique_ptr
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
152
diff
changeset
|
28 #include <Compatibility.h> // For std::unique_ptr<> |
| 152 | 29 #include <Logging.h> |
| 30 #include <OrthancException.h> | |
| 0 | 31 |
| 32 namespace OrthancDatabases | |
| 33 { | |
|
216
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
34 PostgreSQLTransaction::PostgreSQLTransaction(PostgreSQLDatabase& database, |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
35 TransactionType type) : |
| 0 | 36 database_(database), |
|
214
ab96698c73a3
removed useless information about read-only in ITransaction and IPrecompiledStatement
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
193
diff
changeset
|
37 isOpen_(false) |
| 0 | 38 { |
|
216
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
39 Begin(type); |
| 0 | 40 } |
| 41 | |
| 42 | |
| 43 PostgreSQLTransaction::~PostgreSQLTransaction() | |
| 44 { | |
| 45 if (isOpen_) | |
| 46 { | |
|
234
d1b124d116c1
PostgreSQL index plugin handles retries for collisions between multiple writers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
216
diff
changeset
|
47 LOG(INFO) << "PostgreSQL: An active PostgreSQL transaction was dismissed"; |
| 0 | 48 |
| 49 try | |
| 50 { | |
|
237
35598014f140
refactoring to remove GlobalProperties.cpp
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
234
diff
changeset
|
51 database_.ExecuteMultiLines("ABORT"); |
| 0 | 52 } |
| 53 catch (Orthanc::OrthancException&) | |
| 54 { | |
|
46
6a574d810b98
Compatibility with MySQL 8.0
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
55 // Ignore possible exceptions due to connection loss |
| 0 | 56 } |
| 57 } | |
| 58 } | |
| 59 | |
| 60 | |
|
216
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
61 void PostgreSQLTransaction::Begin(TransactionType type) |
| 0 | 62 { |
| 63 if (isOpen_) | |
| 64 { | |
| 65 LOG(ERROR) << "PostgreSQL: Beginning a transaction twice!"; | |
| 66 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); | |
| 67 } | |
| 68 | |
|
237
35598014f140
refactoring to remove GlobalProperties.cpp
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
234
diff
changeset
|
69 database_.ExecuteMultiLines("BEGIN"); |
|
216
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
70 |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
71 switch (type) |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
72 { |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
73 case TransactionType_ReadWrite: |
|
370
d2b5d9c92214
PG: test feature: configurable transaction isolation level
Alain Mazy <am@osimis.io>
parents:
359
diff
changeset
|
74 { |
| 497 | 75 std::string statement = database_.GetReadWriteTransactionStatement(); |
|
370
d2b5d9c92214
PG: test feature: configurable transaction isolation level
Alain Mazy <am@osimis.io>
parents:
359
diff
changeset
|
76 if (!statement.empty()) // if not defined, will use the default DB transaction isolation level |
|
d2b5d9c92214
PG: test feature: configurable transaction isolation level
Alain Mazy <am@osimis.io>
parents:
359
diff
changeset
|
77 { |
|
d2b5d9c92214
PG: test feature: configurable transaction isolation level
Alain Mazy <am@osimis.io>
parents:
359
diff
changeset
|
78 database_.ExecuteMultiLines(statement); |
|
d2b5d9c92214
PG: test feature: configurable transaction isolation level
Alain Mazy <am@osimis.io>
parents:
359
diff
changeset
|
79 } |
| 497 | 80 |
| 81 break; | |
| 82 } | |
|
216
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
83 |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
84 case TransactionType_ReadOnly: |
|
370
d2b5d9c92214
PG: test feature: configurable transaction isolation level
Alain Mazy <am@osimis.io>
parents:
359
diff
changeset
|
85 { |
| 497 | 86 std::string statement = database_.GetReadOnlyTransactionStatement(); |
|
370
d2b5d9c92214
PG: test feature: configurable transaction isolation level
Alain Mazy <am@osimis.io>
parents:
359
diff
changeset
|
87 if (!statement.empty()) // if not defined, will use the default DB transaction isolation level |
|
d2b5d9c92214
PG: test feature: configurable transaction isolation level
Alain Mazy <am@osimis.io>
parents:
359
diff
changeset
|
88 { |
|
d2b5d9c92214
PG: test feature: configurable transaction isolation level
Alain Mazy <am@osimis.io>
parents:
359
diff
changeset
|
89 database_.ExecuteMultiLines(statement); |
|
d2b5d9c92214
PG: test feature: configurable transaction isolation level
Alain Mazy <am@osimis.io>
parents:
359
diff
changeset
|
90 } |
| 497 | 91 |
| 92 break; | |
| 93 } | |
|
216
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
94 |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
95 default: |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
96 throw Orthanc::OrthancException(Orthanc::ErrorCode_ParameterOutOfRange); |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
97 } |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
98 |
| 0 | 99 isOpen_ = true; |
| 100 } | |
| 101 | |
| 102 | |
| 103 void PostgreSQLTransaction::Rollback() | |
| 104 { | |
| 105 if (!isOpen_) | |
| 106 { | |
| 107 LOG(ERROR) << "PostgreSQL: Attempting to rollback a nonexistent transaction. " | |
| 108 << "Did you remember to call Begin()?"; | |
| 109 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); | |
| 110 } | |
| 111 | |
|
237
35598014f140
refactoring to remove GlobalProperties.cpp
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
234
diff
changeset
|
112 database_.ExecuteMultiLines("ABORT"); |
| 0 | 113 isOpen_ = false; |
| 114 } | |
| 115 | |
| 116 | |
| 117 void PostgreSQLTransaction::Commit() | |
| 118 { | |
| 119 if (!isOpen_) | |
| 120 { | |
| 121 LOG(ERROR) << "PostgreSQL: Attempting to roll back a nonexistent transaction. " | |
| 122 << "Did you remember to call Begin()?"; | |
| 123 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); | |
| 124 } | |
| 125 | |
|
237
35598014f140
refactoring to remove GlobalProperties.cpp
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
234
diff
changeset
|
126 database_.ExecuteMultiLines("COMMIT"); |
| 0 | 127 isOpen_ = false; |
| 128 } | |
| 129 | |
| 130 | |
| 131 IResult* PostgreSQLTransaction::Execute(IPrecompiledStatement& statement, | |
| 132 const Dictionary& parameters) | |
| 133 { | |
|
214
ab96698c73a3
removed useless information about read-only in ITransaction and IPrecompiledStatement
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
193
diff
changeset
|
134 return dynamic_cast<PostgreSQLStatement&>(statement).Execute(*this, parameters); |
| 0 | 135 } |
| 136 | |
| 137 | |
| 138 void PostgreSQLTransaction::ExecuteWithoutResult(IPrecompiledStatement& statement, | |
| 139 const Dictionary& parameters) | |
| 140 { | |
| 141 dynamic_cast<PostgreSQLStatement&>(statement).ExecuteWithoutResult(*this, parameters); | |
| 142 } | |
| 143 } |