Mercurial > hg > orthanc-databases
annotate MySQL/Plugins/MySQLStorageArea.cpp @ 224:61c309e06797
added internal class DatabaseBackendAdapterV2::Adapter
author | Sebastien Jodogne <s.jodogne@gmail.com> |
---|---|
date | Thu, 01 Apr 2021 16:09:59 +0200 |
parents | ee5858d438dc |
children | a4918d57435c |
rev | line source |
---|---|
17 | 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 |
17 | 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 "MySQLStorageArea.h" | |
23 | |
24 #include "../../Framework/MySQL/MySQLDatabase.h" | |
25 #include "../../Framework/MySQL/MySQLTransaction.h" | |
137
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
26 #include "MySQLDefinitions.h" |
17 | 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> |
17 | 30 |
31 #include <boost/math/special_functions/round.hpp> | |
32 | |
33 | |
34 namespace OrthancDatabases | |
35 { | |
36 IDatabase* MySQLStorageArea::OpenInternal() | |
37 { | |
157
275e14f57f1e
replacing deprecated std::auto_ptr by std::unique_ptr
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
152
diff
changeset
|
38 std::unique_ptr<MySQLDatabase> db(new MySQLDatabase(parameters_)); |
17 | 39 |
40 db->Open(); | |
41 | |
42 { | |
137
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
43 MySQLDatabase::TransientAdvisoryLock lock(*db, MYSQL_LOCK_DATABASE_SETUP); |
217
ee5858d438dc
TransactionType given to MySQLTransaction constructor
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
193
diff
changeset
|
44 MySQLTransaction t(*db, TransactionType_ReadWrite); |
17 | 45 |
46 int64_t size; | |
47 if (db->LookupGlobalIntegerVariable(size, "max_allowed_packet")) | |
48 { | |
49 int mb = boost::math::iround(static_cast<double>(size) / | |
50 static_cast<double>(1024 * 1024)); | |
51 LOG(WARNING) << "Your MySQL server cannot " | |
52 << "store DICOM files larger than " << mb << "MB"; | |
53 LOG(WARNING) << " => Consider increasing \"max_allowed_packet\" " | |
54 << "in \"my.cnf\" if this limit is insufficient for your use"; | |
55 } | |
56 else | |
57 { | |
58 LOG(WARNING) << "Unable to auto-detect the maximum size of DICOM " | |
59 << "files that can be stored in this MySQL server"; | |
60 } | |
61 | |
62 if (clearAll_) | |
63 { | |
64 db->Execute("DROP TABLE IF EXISTS StorageArea", false); | |
65 } | |
66 | |
67 db->Execute("CREATE TABLE IF NOT EXISTS StorageArea(" | |
68 "uuid VARCHAR(64) NOT NULL PRIMARY KEY," | |
69 "content LONGBLOB NOT NULL," | |
70 "type INTEGER NOT NULL)", false); | |
71 | |
72 t.Commit(); | |
73 } | |
74 | |
137
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
75 /** |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
76 * WARNING: This lock must be acquired after |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
77 * "MYSQL_LOCK_DATABASE_SETUP" is released. Indeed, in MySQL < |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
78 * 5.7, it is impossible to acquire more than one lock at a time, |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
79 * as calling "SELECT GET_LOCK()" releases all the |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
80 * previously-acquired locks. |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
81 * https://dev.mysql.com/doc/refman/5.7/en/locking-functions.html |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
82 **/ |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
83 if (parameters_.HasLock()) |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
84 { |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
85 db->AdvisoryLock(MYSQL_LOCK_STORAGE); |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
86 } |
52b3859ee0b7
MySQL: acquiring named locks instead of numbers
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
135
diff
changeset
|
87 |
17 | 88 return db.release(); |
89 } | |
90 | |
91 | |
92 MySQLStorageArea::MySQLStorageArea(const MySQLParameters& parameters) : | |
93 StorageBackend(new Factory(*this)), | |
94 parameters_(parameters), | |
95 clearAll_(false) | |
96 { | |
97 } | |
98 } |