Mercurial > hg > orthanc-databases
annotate Framework/PostgreSQL/PostgreSQLTransaction.cpp @ 234:d1b124d116c1
PostgreSQL index plugin handles retries for collisions between multiple writers
| author | Sebastien Jodogne <s.jodogne@gmail.com> |
|---|---|
| date | Thu, 08 Apr 2021 10:50:01 +0200 |
| parents | fbb52129158a |
| children | 35598014f140 |
| 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 | |
|
193
3236894320d6
upgrade to year 2021
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
157
diff
changeset
|
5 * Copyright (C) 2017-2021 Osimis S.A., Belgium |
| 0 | 6 * |
| 7 * This program is free software: you can redistribute it and/or | |
| 8 * modify it under the terms of the GNU Affero General Public License | |
| 9 * as published by the Free Software Foundation, either version 3 of | |
| 10 * the License, or (at your option) any later version. | |
| 11 * | |
| 12 * This program is distributed in the hope that it will be useful, but | |
| 13 * WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 14 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
| 15 * Affero General Public License for more details. | |
| 16 * | |
| 17 * You should have received a copy of the GNU Affero General Public License | |
| 18 * along with this program. If not, see <http://www.gnu.org/licenses/>. | |
| 19 **/ | |
| 20 | |
| 21 | |
| 22 #include "PostgreSQLTransaction.h" | |
| 23 | |
| 24 #include "PostgreSQLStatement.h" | |
| 25 | |
|
157
275e14f57f1e
replacing deprecated std::auto_ptr by std::unique_ptr
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
152
diff
changeset
|
26 #include <Compatibility.h> // For std::unique_ptr<> |
| 152 | 27 #include <Logging.h> |
| 28 #include <OrthancException.h> | |
| 0 | 29 |
| 30 namespace OrthancDatabases | |
| 31 { | |
|
216
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
32 PostgreSQLTransaction::PostgreSQLTransaction(PostgreSQLDatabase& database, |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
33 TransactionType type) : |
| 0 | 34 database_(database), |
|
214
ab96698c73a3
removed useless information about read-only in ITransaction and IPrecompiledStatement
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
193
diff
changeset
|
35 isOpen_(false) |
| 0 | 36 { |
|
216
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
37 Begin(type); |
| 0 | 38 } |
| 39 | |
| 40 | |
| 41 PostgreSQLTransaction::~PostgreSQLTransaction() | |
| 42 { | |
| 43 if (isOpen_) | |
| 44 { | |
|
234
d1b124d116c1
PostgreSQL index plugin handles retries for collisions between multiple writers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
216
diff
changeset
|
45 LOG(INFO) << "PostgreSQL: An active PostgreSQL transaction was dismissed"; |
| 0 | 46 |
| 47 try | |
| 48 { | |
| 49 database_.Execute("ABORT"); | |
| 50 } | |
| 51 catch (Orthanc::OrthancException&) | |
| 52 { | |
|
46
6a574d810b98
Compatibility with MySQL 8.0
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
53 // Ignore possible exceptions due to connection loss |
| 0 | 54 } |
| 55 } | |
| 56 } | |
| 57 | |
| 58 | |
|
216
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
59 void PostgreSQLTransaction::Begin(TransactionType type) |
| 0 | 60 { |
| 61 if (isOpen_) | |
| 62 { | |
| 63 LOG(ERROR) << "PostgreSQL: Beginning a transaction twice!"; | |
| 64 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); | |
| 65 } | |
| 66 | |
| 67 database_.Execute("BEGIN"); | |
|
216
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
68 |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
69 switch (type) |
|
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 case TransactionType_ReadWrite: |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
72 database_.Execute("SET TRANSACTION ISOLATION LEVEL SERIALIZABLE READ WRITE"); |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
73 break; |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
74 |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
75 case TransactionType_ReadOnly: |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
76 database_.Execute("SET TRANSACTION ISOLATION LEVEL SERIALIZABLE READ ONLY"); |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
77 break; |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
78 |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
79 default: |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
80 throw Orthanc::OrthancException(Orthanc::ErrorCode_ParameterOutOfRange); |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
81 } |
|
fbb52129158a
TransactionType given to PostgreSQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
214
diff
changeset
|
82 |
| 0 | 83 isOpen_ = true; |
| 84 } | |
| 85 | |
| 86 | |
| 87 void PostgreSQLTransaction::Rollback() | |
| 88 { | |
| 89 if (!isOpen_) | |
| 90 { | |
| 91 LOG(ERROR) << "PostgreSQL: Attempting to rollback a nonexistent transaction. " | |
| 92 << "Did you remember to call Begin()?"; | |
| 93 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); | |
| 94 } | |
| 95 | |
| 96 database_.Execute("ABORT"); | |
| 97 isOpen_ = false; | |
| 98 } | |
| 99 | |
| 100 | |
| 101 void PostgreSQLTransaction::Commit() | |
| 102 { | |
| 103 if (!isOpen_) | |
| 104 { | |
| 105 LOG(ERROR) << "PostgreSQL: Attempting to roll back a nonexistent transaction. " | |
| 106 << "Did you remember to call Begin()?"; | |
| 107 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); | |
| 108 } | |
| 109 | |
| 110 database_.Execute("COMMIT"); | |
| 111 isOpen_ = false; | |
| 112 } | |
| 113 | |
| 114 | |
| 115 IResult* PostgreSQLTransaction::Execute(IPrecompiledStatement& statement, | |
| 116 const Dictionary& parameters) | |
| 117 { | |
|
214
ab96698c73a3
removed useless information about read-only in ITransaction and IPrecompiledStatement
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
193
diff
changeset
|
118 return dynamic_cast<PostgreSQLStatement&>(statement).Execute(*this, parameters); |
| 0 | 119 } |
| 120 | |
| 121 | |
| 122 void PostgreSQLTransaction::ExecuteWithoutResult(IPrecompiledStatement& statement, | |
| 123 const Dictionary& parameters) | |
| 124 { | |
| 125 dynamic_cast<PostgreSQLStatement&>(statement).ExecuteWithoutResult(*this, parameters); | |
| 126 } | |
| 127 } |