Mercurial > hg > orthanc-databases
annotate Framework/Common/DatabaseManager.cpp @ 23:b2ff1cd2907a
handling of implicit transactions in DatabaseManager
| author | Sebastien Jodogne <s.jodogne@gmail.com> |
|---|---|
| date | Thu, 12 Jul 2018 10:44:17 +0200 |
| parents | c7c54993a92e |
| children | 2fb9cd42af14 |
| 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 | |
| 5 * Copyright (C) 2017-2018 Osimis S.A., Belgium | |
| 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 "DatabaseManager.h" | |
| 23 | |
| 24 #include <Core/Logging.h> | |
| 25 #include <Core/OrthancException.h> | |
| 26 | |
| 27 #include <boost/thread.hpp> | |
| 28 | |
| 29 namespace OrthancDatabases | |
| 30 { | |
| 31 IDatabase& DatabaseManager::GetDatabase() | |
| 32 { | |
| 33 static const unsigned int MAX_CONNECTION_ATTEMPTS = 10; // TODO: Parameter | |
| 34 | |
| 35 unsigned int count = 0; | |
| 36 | |
| 37 while (database_.get() == NULL) | |
| 38 { | |
| 39 transaction_.reset(NULL); | |
| 40 | |
| 41 try | |
| 42 { | |
| 43 database_.reset(factory_->Open()); | |
| 44 } | |
| 45 catch (Orthanc::OrthancException& e) | |
| 46 { | |
| 47 if (e.GetErrorCode() == Orthanc::ErrorCode_DatabaseUnavailable) | |
| 48 { | |
| 49 count ++; | |
| 50 | |
| 51 if (count <= MAX_CONNECTION_ATTEMPTS) | |
| 52 { | |
| 53 LOG(WARNING) << "Database is currently unavailable, retrying..."; | |
| 54 boost::this_thread::sleep(boost::posix_time::seconds(1)); | |
| 55 continue; | |
| 56 } | |
| 57 else | |
| 58 { | |
| 59 LOG(ERROR) << "Timeout when connecting to the database, giving up"; | |
| 60 } | |
| 61 } | |
| 62 | |
| 63 throw; | |
| 64 } | |
| 65 } | |
| 66 | |
| 67 if (database_.get() == NULL || | |
| 68 database_->GetDialect() != dialect_) | |
| 69 { | |
| 70 throw Orthanc::OrthancException(Orthanc::ErrorCode_InternalError); | |
| 71 } | |
| 72 else | |
| 73 { | |
| 74 return *database_; | |
| 75 } | |
| 76 } | |
| 77 | |
| 78 | |
| 79 void DatabaseManager::Close() | |
| 80 { | |
| 81 LOG(TRACE) << "Closing the connection to the database"; | |
| 82 | |
| 83 // Rollback active transaction, if any | |
| 84 transaction_.reset(NULL); | |
| 85 | |
| 86 // Delete all the cached statements (must occur before closing | |
| 87 // the database) | |
| 88 for (CachedStatements::iterator it = cachedStatements_.begin(); | |
| 89 it != cachedStatements_.end(); ++it) | |
| 90 { | |
| 91 assert(it->second != NULL); | |
| 92 delete it->second; | |
| 93 } | |
| 94 | |
| 95 cachedStatements_.clear(); | |
| 96 | |
| 97 // Close the database | |
| 98 database_.reset(NULL); | |
| 99 | |
| 100 LOG(TRACE) << "Connection to the database is closed"; | |
| 101 } | |
| 102 | |
| 103 | |
| 104 void DatabaseManager::CloseIfUnavailable(Orthanc::ErrorCode e) | |
| 105 { | |
| 106 if (e != Orthanc::ErrorCode_Success) | |
| 107 { | |
| 108 transaction_.reset(NULL); | |
| 109 } | |
| 110 | |
| 111 if (e == Orthanc::ErrorCode_DatabaseUnavailable) | |
| 112 { | |
| 113 LOG(ERROR) << "The database is not available, closing the connection"; | |
| 114 Close(); | |
| 115 } | |
| 116 } | |
| 117 | |
| 118 | |
| 119 IPrecompiledStatement* DatabaseManager::LookupCachedStatement(const StatementLocation& location) const | |
| 120 { | |
| 121 CachedStatements::const_iterator found = cachedStatements_.find(location); | |
| 122 | |
| 123 if (found == cachedStatements_.end()) | |
| 124 { | |
| 125 return NULL; | |
| 126 } | |
| 127 else | |
| 128 { | |
| 129 assert(found->second != NULL); | |
| 130 return found->second; | |
| 131 } | |
| 132 } | |
| 133 | |
| 134 | |
| 135 IPrecompiledStatement& DatabaseManager::CacheStatement(const StatementLocation& location, | |
| 136 const Query& query) | |
| 137 { | |
| 138 LOG(TRACE) << "Caching statement from " << location.GetFile() << ":" << location.GetLine(); | |
| 139 | |
| 140 std::auto_ptr<IPrecompiledStatement> statement(GetDatabase().Compile(query)); | |
| 141 | |
| 142 IPrecompiledStatement* tmp = statement.get(); | |
| 143 if (tmp == NULL) | |
| 144 { | |
| 145 throw Orthanc::OrthancException(Orthanc::ErrorCode_InternalError); | |
| 146 } | |
| 147 | |
| 148 assert(cachedStatements_.find(location) == cachedStatements_.end()); | |
| 149 cachedStatements_[location] = statement.release(); | |
| 150 | |
| 151 return *tmp; | |
| 152 } | |
| 153 | |
| 154 | |
| 155 ITransaction& DatabaseManager::GetTransaction() | |
| 156 { | |
| 157 if (transaction_.get() == NULL) | |
| 158 { | |
|
23
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
159 LOG(TRACE) << "Automatically creating an implicit database transaction"; |
| 0 | 160 |
| 161 try | |
| 162 { | |
|
23
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
163 transaction_.reset(GetDatabase().CreateTransaction(true)); |
| 0 | 164 } |
| 165 catch (Orthanc::OrthancException& e) | |
| 166 { | |
| 167 CloseIfUnavailable(e.GetErrorCode()); | |
| 168 throw; | |
| 169 } | |
| 170 } | |
| 171 | |
| 172 assert(transaction_.get() != NULL); | |
| 173 return *transaction_; | |
| 174 } | |
| 175 | |
|
23
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
176 |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
177 void DatabaseManager::ReleaseImplicitTransaction() |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
178 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
179 if (transaction_.get() != NULL && |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
180 transaction_->IsImplicit()) |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
181 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
182 LOG(TRACE) << "Committing an implicit database transaction"; |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
183 |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
184 try |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
185 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
186 transaction_->Commit(); |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
187 transaction_.reset(NULL); |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
188 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
189 catch (Orthanc::OrthancException& e) |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
190 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
191 // Don't throw the exception, as we are in CachedStatement destructor |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
192 LOG(ERROR) << "Error while committing an implicit database transaction: " << e.What(); |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
193 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
194 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
195 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
196 |
| 0 | 197 |
| 198 DatabaseManager::DatabaseManager(IDatabaseFactory* factory) : // Takes ownership | |
| 199 factory_(factory) | |
| 200 { | |
| 201 if (factory == NULL) | |
| 202 { | |
| 203 throw Orthanc::OrthancException(Orthanc::ErrorCode_NullPointer); | |
| 204 } | |
| 205 | |
| 206 dialect_ = factory->GetDialect(); | |
| 207 } | |
| 208 | |
| 209 | |
| 210 void DatabaseManager::StartTransaction() | |
| 211 { | |
| 212 boost::recursive_mutex::scoped_lock lock(mutex_); | |
| 213 | |
| 214 try | |
| 215 { | |
| 216 if (transaction_.get() != NULL) | |
| 217 { | |
|
23
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
218 LOG(ERROR) << "Cannot start another transaction while there is an uncommitted transaction"; |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
219 throw Orthanc::OrthancException(Orthanc::ErrorCode_Database); |
| 0 | 220 } |
| 221 | |
|
23
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
222 transaction_.reset(GetDatabase().CreateTransaction(false)); |
| 0 | 223 } |
| 224 catch (Orthanc::OrthancException& e) | |
| 225 { | |
| 226 CloseIfUnavailable(e.GetErrorCode()); | |
| 227 throw; | |
| 228 } | |
| 229 } | |
| 230 | |
| 231 | |
| 232 void DatabaseManager::CommitTransaction() | |
| 233 { | |
| 234 boost::recursive_mutex::scoped_lock lock(mutex_); | |
| 235 | |
| 236 if (transaction_.get() == NULL) | |
| 237 { | |
| 238 LOG(ERROR) << "Cannot commit a non-existing transaction"; | |
| 239 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); | |
| 240 } | |
| 241 else | |
| 242 { | |
| 243 try | |
| 244 { | |
| 245 transaction_->Commit(); | |
| 246 transaction_.reset(NULL); | |
| 247 } | |
| 248 catch (Orthanc::OrthancException& e) | |
| 249 { | |
| 250 CloseIfUnavailable(e.GetErrorCode()); | |
| 251 throw; | |
| 252 } | |
| 253 } | |
| 254 } | |
| 255 | |
| 256 | |
| 257 void DatabaseManager::RollbackTransaction() | |
| 258 { | |
| 259 boost::recursive_mutex::scoped_lock lock(mutex_); | |
| 260 | |
| 261 if (transaction_.get() == NULL) | |
| 262 { | |
| 263 LOG(ERROR) << "Cannot rollback a non-existing transaction"; | |
| 264 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); | |
| 265 } | |
| 266 else | |
| 267 { | |
| 268 try | |
| 269 { | |
| 270 transaction_->Rollback(); | |
| 271 transaction_.reset(NULL); | |
| 272 } | |
| 273 catch (Orthanc::OrthancException& e) | |
| 274 { | |
| 275 CloseIfUnavailable(e.GetErrorCode()); | |
| 276 throw; | |
| 277 } | |
| 278 } | |
| 279 } | |
| 280 | |
| 281 | |
| 282 IResult& DatabaseManager::CachedStatement::GetResult() const | |
| 283 { | |
| 284 if (result_.get() == NULL) | |
| 285 { | |
| 286 LOG(ERROR) << "Accessing the results of a statement without having executed it"; | |
| 287 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); | |
| 288 } | |
| 289 | |
| 290 return *result_; | |
| 291 } | |
| 292 | |
| 293 | |
|
12
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
294 void DatabaseManager::CachedStatement::Setup(const char* sql) |
| 0 | 295 { |
|
12
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
296 statement_ = manager_.LookupCachedStatement(location_); |
| 0 | 297 |
| 298 if (statement_ == NULL) | |
| 299 { | |
| 300 query_.reset(new Query(sql)); | |
| 301 } | |
| 302 else | |
| 303 { | |
| 304 LOG(TRACE) << "Reusing cached statement from " | |
|
12
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
305 << location_.GetFile() << ":" << location_.GetLine(); |
| 0 | 306 } |
| 307 } | |
| 308 | |
|
12
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
309 |
|
23
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
310 DatabaseManager::Transaction::Transaction(DatabaseManager& manager) : |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
311 lock_(manager.mutex_), |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
312 manager_(manager), |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
313 database_(manager.GetDatabase()), |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
314 committed_(false) |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
315 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
316 manager_.StartTransaction(); |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
317 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
318 |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
319 |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
320 DatabaseManager::Transaction::~Transaction() |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
321 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
322 if (!committed_) |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
323 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
324 try |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
325 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
326 manager_.RollbackTransaction(); |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
327 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
328 catch (Orthanc::OrthancException& e) |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
329 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
330 // Don't rethrow the exception as we are in a destructor |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
331 LOG(ERROR) << "Uncatched error during some transaction rollback: " << e.What(); |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
332 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
333 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
334 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
335 |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
336 |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
337 void DatabaseManager::Transaction::Commit() |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
338 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
339 if (committed_) |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
340 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
341 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
342 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
343 else |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
344 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
345 manager_.CommitTransaction(); |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
346 committed_ = true; |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
347 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
348 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
349 |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
350 |
|
12
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
351 DatabaseManager::CachedStatement::CachedStatement(const StatementLocation& location, |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
352 DatabaseManager& manager, |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
353 const char* sql) : |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
354 lock_(manager.mutex_), |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
355 manager_(manager), |
| 18 | 356 database_(manager_.GetDatabase()), |
|
12
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
357 location_(location), |
| 18 | 358 transaction_(manager_.GetTransaction()) |
|
12
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
359 { |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
360 Setup(sql); |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
361 } |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
362 |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
363 |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
364 DatabaseManager::CachedStatement::CachedStatement(const StatementLocation& location, |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
365 Transaction& transaction, |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
366 const char* sql) : |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
367 lock_(manager_.mutex_), |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
368 manager_(transaction.GetManager()), |
| 18 | 369 database_(manager_.GetDatabase()), |
|
12
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
370 location_(location), |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
371 transaction_(manager_.GetTransaction()) |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
372 { |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
373 Setup(sql); |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
374 } |
|
41543239072d
transactions for storage area
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
0
diff
changeset
|
375 |
|
23
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
376 |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
377 DatabaseManager::CachedStatement::~CachedStatement() |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
378 { |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
379 manager_.ReleaseImplicitTransaction(); |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
380 } |
|
b2ff1cd2907a
handling of implicit transactions in DatabaseManager
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
18
diff
changeset
|
381 |
| 0 | 382 |
| 383 void DatabaseManager::CachedStatement::SetReadOnly(bool readOnly) | |
| 384 { | |
| 385 if (query_.get() != NULL) | |
| 386 { | |
| 387 query_->SetReadOnly(readOnly); | |
| 388 } | |
| 389 } | |
| 390 | |
| 391 | |
| 392 void DatabaseManager::CachedStatement::SetParameterType(const std::string& parameter, | |
| 393 ValueType type) | |
| 394 { | |
| 395 if (query_.get() != NULL) | |
| 396 { | |
| 397 query_->SetType(parameter, type); | |
| 398 } | |
| 399 } | |
| 400 | |
| 401 | |
| 402 void DatabaseManager::CachedStatement::Execute() | |
| 403 { | |
| 404 Dictionary parameters; | |
| 405 Execute(parameters); | |
| 406 } | |
| 407 | |
| 408 | |
| 409 void DatabaseManager::CachedStatement::Execute(const Dictionary& parameters) | |
| 410 { | |
| 411 if (result_.get() != NULL) | |
| 412 { | |
| 413 LOG(ERROR) << "Cannot execute twice a statement"; | |
| 414 throw Orthanc::OrthancException(Orthanc::ErrorCode_BadSequenceOfCalls); | |
| 415 } | |
| 416 | |
| 417 try | |
| 418 { | |
| 419 if (query_.get() != NULL) | |
| 420 { | |
| 421 // Register the newly-created statement | |
| 422 assert(statement_ == NULL); | |
| 423 statement_ = &manager_.CacheStatement(location_, *query_); | |
| 424 query_.reset(NULL); | |
| 425 } | |
| 426 | |
| 427 assert(statement_ != NULL); | |
| 428 result_.reset(transaction_.Execute(*statement_, parameters)); | |
| 429 } | |
| 430 catch (Orthanc::OrthancException& e) | |
| 431 { | |
| 432 manager_.CloseIfUnavailable(e.GetErrorCode()); | |
| 433 throw; | |
| 434 } | |
| 435 } | |
| 436 | |
| 437 | |
| 438 bool DatabaseManager::CachedStatement::IsDone() const | |
| 439 { | |
| 440 try | |
| 441 { | |
| 442 return GetResult().IsDone(); | |
| 443 } | |
| 444 catch (Orthanc::OrthancException& e) | |
| 445 { | |
| 446 manager_.CloseIfUnavailable(e.GetErrorCode()); | |
| 447 throw; | |
| 448 } | |
| 449 } | |
| 450 | |
| 451 | |
| 452 void DatabaseManager::CachedStatement::Next() | |
| 453 { | |
| 454 try | |
| 455 { | |
| 456 GetResult().Next(); | |
| 457 } | |
| 458 catch (Orthanc::OrthancException& e) | |
| 459 { | |
| 460 manager_.CloseIfUnavailable(e.GetErrorCode()); | |
| 461 throw; | |
| 462 } | |
| 463 } | |
| 464 | |
| 465 | |
| 466 size_t DatabaseManager::CachedStatement::GetResultFieldsCount() const | |
| 467 { | |
| 468 try | |
| 469 { | |
| 470 return GetResult().GetFieldsCount(); | |
| 471 } | |
| 472 catch (Orthanc::OrthancException& e) | |
| 473 { | |
| 474 manager_.CloseIfUnavailable(e.GetErrorCode()); | |
| 475 throw; | |
| 476 } | |
| 477 } | |
| 478 | |
| 479 | |
| 480 void DatabaseManager::CachedStatement::SetResultFieldType(size_t field, | |
| 481 ValueType type) | |
| 482 { | |
| 483 try | |
| 484 { | |
| 485 if (!GetResult().IsDone()) | |
| 486 { | |
| 487 GetResult().SetExpectedType(field, type); | |
| 488 } | |
| 489 } | |
| 490 catch (Orthanc::OrthancException& e) | |
| 491 { | |
| 492 manager_.CloseIfUnavailable(e.GetErrorCode()); | |
| 493 throw; | |
| 494 } | |
| 495 } | |
| 496 | |
| 497 | |
| 498 const IValue& DatabaseManager::CachedStatement::GetResultField(size_t index) const | |
| 499 { | |
| 500 try | |
| 501 { | |
| 502 return GetResult().GetField(index); | |
| 503 } | |
| 504 catch (Orthanc::OrthancException& e) | |
| 505 { | |
| 506 manager_.CloseIfUnavailable(e.GetErrorCode()); | |
| 507 throw; | |
| 508 } | |
| 509 } | |
| 510 } |