annotate UnitTestsSources/MultiThreadingTests.cpp @ 2559:9b7680dee75d jobs

cont
author Sebastien Jodogne <s.jodogne@gmail.com>
date Thu, 03 May 2018 18:45:47 +0200
parents 57f81b988713
children 1e66fe3ddf9f
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
827
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
1 /**
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
2 * Orthanc - A Lightweight, RESTful DICOM Store
1900
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1681
diff changeset
3 * Copyright (C) 2012-2016 Sebastien Jodogne, Medical Physics
1288
6e7e5ed91c2d upgrade to year 2015
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1009
diff changeset
4 * Department, University Hospital of Liege, Belgium
2447
878b59270859 upgrade to year 2018
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2382
diff changeset
5 * Copyright (C) 2017-2018 Osimis S.A., Belgium
827
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
6 *
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
7 * This program is free software: you can redistribute it and/or
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
8 * modify it under the terms of the GNU General Public License as
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
9 * published by the Free Software Foundation, either version 3 of the
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
10 * License, or (at your option) any later version.
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
11 *
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
12 * In addition, as a special exception, the copyright holders of this
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
13 * program give permission to link the code of its release with the
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
14 * OpenSSL project's "OpenSSL" library (or with modified versions of it
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
15 * that use the same license as the "OpenSSL" library), and distribute
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
16 * the linked executables. You must obey the GNU General Public License
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
17 * in all respects for all of the code used other than "OpenSSL". If you
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
18 * modify file(s) with this exception, you may extend this exception to
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
19 * your version of the file(s), but you are not obligated to do so. If
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
20 * you do not wish to do so, delete this exception statement from your
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
21 * version. If you delete this exception statement from all source files
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
22 * in the program, then also delete it here.
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
23 *
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
24 * This program is distributed in the hope that it will be useful, but
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
25 * WITHOUT ANY WARRANTY; without even the implied warranty of
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
26 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
27 * General Public License for more details.
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
28 *
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
29 * You should have received a copy of the GNU General Public License
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
30 * along with this program. If not, see <http://www.gnu.org/licenses/>.
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
31 **/
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
32
3d6f9b7d0add precompiled headers in unit tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 776
diff changeset
33
831
84513f2ee1f3 pch for unit tests and server
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 827
diff changeset
34 #include "PrecompiledHeadersUnitTests.h"
723
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
35 #include "gtest/gtest.h"
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
36
781
f0ac3a53ccf2 scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 779
diff changeset
37 #include "../OrthancServer/Scheduler/ServerScheduler.h"
723
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
38 #include "../Core/OrthancException.h"
2143
fd5875662670 creation of namespace SystemToolbox
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2140
diff changeset
39 #include "../Core/SystemToolbox.h"
723
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
40 #include "../Core/Toolbox.h"
760
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
41 #include "../Core/MultiThreading/Locker.h"
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
42 #include "../Core/MultiThreading/Mutex.h"
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
43 #include "../Core/MultiThreading/ReaderWriterLock.h"
723
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
44
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
45 using namespace Orthanc;
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
46
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
47 namespace
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
48 {
1396
ac4efabeb80c Migration of the orthanc-client as a separate project
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1304
diff changeset
49 class DynamicInteger : public IDynamicObject
723
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
50 {
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
51 private:
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
52 int value_;
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
53 std::set<int>& target_;
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
54
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
55 public:
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
56 DynamicInteger(int value, std::set<int>& target) :
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
57 value_(value), target_(target)
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
58 {
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
59 }
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
60
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
61 int GetValue() const
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
62 {
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
63 return value_;
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
64 }
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
65 };
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
66 }
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
67
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
68
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
69 TEST(MultiThreading, SharedMessageQueueBasic)
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
70 {
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
71 std::set<int> s;
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
72
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
73 SharedMessageQueue q;
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
74 ASSERT_TRUE(q.WaitEmpty(0));
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
75 q.Enqueue(new DynamicInteger(10, s));
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
76 ASSERT_FALSE(q.WaitEmpty(1));
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
77 q.Enqueue(new DynamicInteger(20, s));
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
78 q.Enqueue(new DynamicInteger(30, s));
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
79 q.Enqueue(new DynamicInteger(40, s));
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
80
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
81 std::auto_ptr<DynamicInteger> i;
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
82 i.reset(dynamic_cast<DynamicInteger*>(q.Dequeue(1))); ASSERT_EQ(10, i->GetValue());
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
83 i.reset(dynamic_cast<DynamicInteger*>(q.Dequeue(1))); ASSERT_EQ(20, i->GetValue());
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
84 i.reset(dynamic_cast<DynamicInteger*>(q.Dequeue(1))); ASSERT_EQ(30, i->GetValue());
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
85 ASSERT_FALSE(q.WaitEmpty(1));
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
86 i.reset(dynamic_cast<DynamicInteger*>(q.Dequeue(1))); ASSERT_EQ(40, i->GetValue());
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
87 ASSERT_TRUE(q.WaitEmpty(0));
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
88 ASSERT_EQ(NULL, q.Dequeue(1));
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
89 }
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
90
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
91
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
92 TEST(MultiThreading, SharedMessageQueueClean)
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
93 {
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
94 std::set<int> s;
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
95
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
96 try
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
97 {
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
98 SharedMessageQueue q;
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
99 q.Enqueue(new DynamicInteger(10, s));
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
100 q.Enqueue(new DynamicInteger(20, s));
1583
9ea3d082b064 got rid of custom exceptions
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1582
diff changeset
101 throw OrthancException(ErrorCode_InternalError);
723
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
102 }
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
103 catch (OrthancException&)
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
104 {
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
105 }
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
106 }
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
107
0da078f3affc multithreading tests
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
diff changeset
108
760
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
109 TEST(MultiThreading, Mutex)
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
110 {
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
111 Mutex mutex;
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
112 Locker locker(mutex);
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
113 }
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
114
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
115
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
116 TEST(MultiThreading, ReaderWriterLock)
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
117 {
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
118 ReaderWriterLock lock;
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
119
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
120 {
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
121 Locker locker1(lock.ForReader());
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
122 Locker locker2(lock.ForReader());
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
123 }
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
124
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
125 {
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
126 Locker locker3(lock.ForWriter());
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
127 }
b2a62f22fbe8 lock abstraction
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 723
diff changeset
128 }
769
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
129
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
130
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
131
2382
7284093111b0 big reorganization to cleanly separate framework vs. server
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2244
diff changeset
132 #include "../Core/DicomNetworking/ReusableDicomUserConnection.h"
769
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
133
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
134 TEST(ReusableDicomUserConnection, DISABLED_Basic)
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
135 {
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
136 ReusableDicomUserConnection c;
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
137 c.SetMillisecondsBeforeClose(200);
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
138 printf("START\n"); fflush(stdout);
775
d3ba35466225 integration mainline -> lua-scripting
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 770 769
diff changeset
139
769
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
140 {
1427
d710ea64f0fd Custom setting of the local AET during C-Store SCU (both in Lua and in the REST API)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1396
diff changeset
141 RemoteModalityParameters remote("STORESCP", "localhost", 2000, ModalityManufacturer_Generic);
d710ea64f0fd Custom setting of the local AET during C-Store SCU (both in Lua and in the REST API)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1396
diff changeset
142 ReusableDicomUserConnection::Locker lock(c, "ORTHANC", remote);
2222
21713ce8717b Fix handling of Move Originator AET and ID in C-MOVE SCP
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2143
diff changeset
143 lock.GetConnection().StoreFile("/home/jodogne/DICOM/Cardiac/MR.X.1.2.276.0.7230010.3.1.4.2831157719.2256.1336386844.676281");
769
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
144 }
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
145
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
146 printf("**\n"); fflush(stdout);
2242
4e8e0ad2001c move USleep() in SystemToolbox
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2222
diff changeset
147 SystemToolbox::USleep(1000000);
769
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
148 printf("**\n"); fflush(stdout);
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
149
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
150 {
1427
d710ea64f0fd Custom setting of the local AET during C-Store SCU (both in Lua and in the REST API)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1396
diff changeset
151 RemoteModalityParameters remote("STORESCP", "localhost", 2000, ModalityManufacturer_Generic);
d710ea64f0fd Custom setting of the local AET during C-Store SCU (both in Lua and in the REST API)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1396
diff changeset
152 ReusableDicomUserConnection::Locker lock(c, "ORTHANC", remote);
2222
21713ce8717b Fix handling of Move Originator AET and ID in C-MOVE SCP
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2143
diff changeset
153 lock.GetConnection().StoreFile("/home/jodogne/DICOM/Cardiac/MR.X.1.2.276.0.7230010.3.1.4.2831157719.2256.1336386844.676277");
769
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
154 }
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
155
2140
aa4b8895cd23 reorganization
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1907
diff changeset
156 SystemToolbox::ServerBarrier();
769
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
157 printf("DONE\n"); fflush(stdout);
3f946e5c3802 ReusableDicomUserConnection
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
158 }
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
159
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
160
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
161
1000
13e230bbd882 rename filter to command
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 995
diff changeset
162 class Tutu : public IServerCommand
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
163 {
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
164 private:
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
165 int factor_;
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
166
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
167 public:
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
168 Tutu(int f) : factor_(f)
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
169 {
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
170 }
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
171
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
172 virtual bool Apply(ListOfStrings& outputs,
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
173 const ListOfStrings& inputs)
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
174 {
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
175 for (ListOfStrings::const_iterator
1304
7b6925b0890d cppcheck
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1288
diff changeset
176 it = inputs.begin(); it != inputs.end(); ++it)
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
177 {
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
178 int a = boost::lexical_cast<int>(*it);
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
179 int b = factor_ * a;
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
180
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
181 printf("%d * %d = %d\n", a, factor_, b);
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
182
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
183 //if (a == 84) { printf("BREAK\n"); return false; }
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
184
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
185 outputs.push_back(boost::lexical_cast<std::string>(b));
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
186 }
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
187
2242
4e8e0ad2001c move USleep() in SystemToolbox
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2222
diff changeset
188 SystemToolbox::USleep(30000);
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
189
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
190 return true;
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
191 }
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
192 };
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
193
768
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
194
770
a64ca424e0e2 fix test
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 768
diff changeset
195 static void Tata(ServerScheduler* s, ServerJob* j, bool* done)
768
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
196 {
1000
13e230bbd882 rename filter to command
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 995
diff changeset
197 typedef IServerCommand::ListOfStrings ListOfStrings;
779
76eb563f08f0 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 777
diff changeset
198
770
a64ca424e0e2 fix test
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 768
diff changeset
199 while (!(*done))
768
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
200 {
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
201 ListOfStrings l;
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
202 s->GetListOfJobs(l);
1304
7b6925b0890d cppcheck
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1288
diff changeset
203 for (ListOfStrings::iterator it = l.begin(); it != l.end(); ++it)
7b6925b0890d cppcheck
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1288
diff changeset
204 {
7b6925b0890d cppcheck
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1288
diff changeset
205 printf(">> %s: %0.1f\n", it->c_str(), 100.0f * s->GetProgress(*it));
7b6925b0890d cppcheck
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1288
diff changeset
206 }
2242
4e8e0ad2001c move USleep() in SystemToolbox
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2222
diff changeset
207 SystemToolbox::USleep(3000);
768
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
208 }
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
209 }
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
210
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
211
1009
26642cecd36d clearer job interface
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1005
diff changeset
212 TEST(MultiThreading, ServerScheduler)
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
213 {
995
8c67382f44a7 limit number of jobs in the scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 994
diff changeset
214 ServerScheduler scheduler(10);
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
215
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
216 ServerJob job;
1000
13e230bbd882 rename filter to command
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 995
diff changeset
217 ServerCommandInstance& f2 = job.AddCommand(new Tutu(2));
13e230bbd882 rename filter to command
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 995
diff changeset
218 ServerCommandInstance& f3 = job.AddCommand(new Tutu(3));
13e230bbd882 rename filter to command
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 995
diff changeset
219 ServerCommandInstance& f4 = job.AddCommand(new Tutu(4));
13e230bbd882 rename filter to command
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 995
diff changeset
220 ServerCommandInstance& f5 = job.AddCommand(new Tutu(5));
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
221 f2.AddInput(boost::lexical_cast<std::string>(42));
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
222 //f3.AddInput(boost::lexical_cast<std::string>(42));
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
223 //f4.AddInput(boost::lexical_cast<std::string>(42));
1009
26642cecd36d clearer job interface
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1005
diff changeset
224 f2.ConnectOutput(f3);
26642cecd36d clearer job interface
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1005
diff changeset
225 f3.ConnectOutput(f4);
26642cecd36d clearer job interface
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1005
diff changeset
226 f4.ConnectOutput(f5);
26642cecd36d clearer job interface
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1005
diff changeset
227
26642cecd36d clearer job interface
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1005
diff changeset
228 f3.SetConnectedToSink(true);
26642cecd36d clearer job interface
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1005
diff changeset
229 f5.SetConnectedToSink(true);
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
230
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
231 job.SetDescription("tutu");
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
232
770
a64ca424e0e2 fix test
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 768
diff changeset
233 bool done = false;
a64ca424e0e2 fix test
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 768
diff changeset
234 boost::thread t(Tata, &scheduler, &job, &done);
768
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
235
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
236
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
237 //scheduler.Submit(job);
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
238
1000
13e230bbd882 rename filter to command
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 995
diff changeset
239 IServerCommand::ListOfStrings l;
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
240 scheduler.SubmitAndWait(l, job);
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
241
1492
9f66a12eb8fc fix warnings for gcc 5
jodogne
parents: 1486
diff changeset
242 ASSERT_EQ(2u, l.size());
1009
26642cecd36d clearer job interface
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1005
diff changeset
243 ASSERT_EQ(42 * 2 * 3, boost::lexical_cast<int>(l.front()));
26642cecd36d clearer job interface
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1005
diff changeset
244 ASSERT_EQ(42 * 2 * 3 * 4 * 5, boost::lexical_cast<int>(l.back()));
26642cecd36d clearer job interface
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1005
diff changeset
245
1000
13e230bbd882 rename filter to command
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 995
diff changeset
246 for (IServerCommand::ListOfStrings::iterator i = l.begin(); i != l.end(); i++)
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
247 {
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
248 printf("** %s\n", i->c_str());
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
249 }
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
250
2140
aa4b8895cd23 reorganization
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1907
diff changeset
251 //SystemToolbox::ServerBarrier();
2242
4e8e0ad2001c move USleep() in SystemToolbox
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2222
diff changeset
252 //SystemToolbox::USleep(3000000);
768
476febedc516 improvements
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 765
diff changeset
253
1453
c0bdc47165ef code to warn about possible threading problems
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1427
diff changeset
254 scheduler.Stop();
c0bdc47165ef code to warn about possible threading problems
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1427
diff changeset
255
770
a64ca424e0e2 fix test
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 768
diff changeset
256 done = true;
1453
c0bdc47165ef code to warn about possible threading problems
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1427
diff changeset
257 if (t.joinable())
c0bdc47165ef code to warn about possible threading problems
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1427
diff changeset
258 {
c0bdc47165ef code to warn about possible threading problems
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1427
diff changeset
259 t.join();
c0bdc47165ef code to warn about possible threading problems
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 1427
diff changeset
260 }
765
fc97f762834c scheduler
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 760
diff changeset
261 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
262
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
263
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
264
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
265
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
266
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
267 #if !defined(ORTHANC_SANDBOXED)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
268 # error The macro ORTHANC_SANDBOXED must be defined
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
269 #endif
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
270
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
271 #if ORTHANC_SANDBOXED == 1
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
272 # error The job engine cannot be used in sandboxed environments
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
273 #endif
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
274
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
275 #include "../Core/Logging.h"
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
276
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
277 #include <boost/math/special_functions/round.hpp>
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
278 #include <boost/date_time/posix_time/posix_time.hpp>
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
279 #include <queue>
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
280
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
281 namespace Orthanc
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
282 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
283 enum JobState
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
284 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
285 JobState_Pending,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
286 JobState_Running,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
287 JobState_Success,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
288 JobState_Failure,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
289 JobState_Paused,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
290 JobState_Retry
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
291 };
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
292
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
293 enum JobStepStatus
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
294 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
295 JobStepStatus_Success,
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
296 JobStepStatus_Failure,
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
297 JobStepStatus_Continue,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
298 JobStepStatus_Retry
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
299 };
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
300
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
301
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
302 class JobStepResult
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
303 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
304 private:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
305 JobStepStatus status_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
306
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
307 public:
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
308 explicit JobStepResult(JobStepStatus status) :
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
309 status_(status)
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
310 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
311 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
312
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
313 virtual ~JobStepResult()
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
314 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
315 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
316
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
317 JobStepStatus GetStatus() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
318 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
319 return status_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
320 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
321 };
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
322
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
323
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
324 class RetryResult : public JobStepResult
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
325 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
326 private:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
327 unsigned int timeout_; // Retry after "timeout_" milliseconds
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
328
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
329 public:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
330 RetryResult(unsigned int timeout) :
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
331 JobStepResult(JobStepStatus_Retry),
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
332 timeout_(timeout)
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
333 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
334 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
335
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
336 unsigned int GetTimeout() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
337 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
338 return timeout_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
339 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
340 };
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
341
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
342
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
343 class IJob : public boost::noncopyable
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
344 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
345 public:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
346 virtual ~IJob()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
347 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
348 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
349
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
350 virtual JobStepResult* ExecuteStep() = 0;
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
351
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
352 virtual void ReleaseResources() = 0; // For pausing jobs
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
353
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
354 virtual float GetProgress() = 0;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
355
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
356 virtual void FormatStatus(Json::Value& value) = 0;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
357 };
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
358
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
359
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
360 class JobInfo
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
361 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
362 private:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
363 std::string id_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
364 int priority_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
365 ErrorCode errorCode_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
366 JobState state_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
367 float progress_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
368 boost::posix_time::ptime infoTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
369 boost::posix_time::ptime creationTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
370 boost::posix_time::time_duration runtime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
371 boost::posix_time::ptime eta_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
372 Json::Value status_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
373
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
374 public:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
375 JobInfo(const std::string& id,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
376 int priority,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
377 ErrorCode errorCode,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
378 JobState state,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
379 float progress,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
380 const boost::posix_time::ptime& creationTime,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
381 const boost::posix_time::time_duration& runtime) :
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
382 id_(id),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
383 priority_(priority),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
384 errorCode_(errorCode),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
385 state_(state),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
386 progress_(progress),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
387 infoTime_(boost::posix_time::microsec_clock::universal_time()),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
388 creationTime_(creationTime),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
389 runtime_(runtime)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
390 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
391 if (progress < 0 ||
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
392 progress > 1)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
393 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
394 throw OrthancException(ErrorCode_ParameterOutOfRange);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
395 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
396
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
397 float r = static_cast<float>(runtime_.total_milliseconds());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
398
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
399 eta_ = infoTime_ + boost::posix_time::milliseconds(boost::math::llround(1.0f - progress) * r);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
400 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
401
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
402 const std::string& GetIdentifier() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
403 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
404 return id_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
405 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
406
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
407 int GetPriority() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
408 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
409 return priority_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
410 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
411
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
412 ErrorCode GetErrorCode() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
413 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
414 return errorCode_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
415 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
416
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
417 JobState GetState() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
418 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
419 return state_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
420 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
421
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
422 float GetProgress() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
423 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
424 return progress_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
425 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
426
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
427 const boost::posix_time::ptime& GetInfoTime() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
428 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
429 return infoTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
430 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
431
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
432 const boost::posix_time::ptime& GetCreationTime() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
433 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
434 return creationTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
435 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
436
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
437 const boost::posix_time::time_duration& GetRuntime() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
438 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
439 return runtime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
440 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
441
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
442 const boost::posix_time::ptime& GetEstimatedTimeOfArrival() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
443 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
444 return eta_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
445 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
446
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
447 const Json::Value& GetStatus() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
448 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
449 return status_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
450 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
451
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
452 Json::Value& GetStatus()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
453 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
454 return status_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
455 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
456 };
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
457
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
458
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
459 class JobHandler : public boost::noncopyable
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
460 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
461 private:
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
462 std::string id_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
463 JobState state_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
464 boost::mutex jobMutex_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
465 std::auto_ptr<IJob> job_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
466 int priority_; // "+inf()" means highest priority
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
467 boost::posix_time::ptime creationTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
468 boost::posix_time::ptime lastStateChangeTime;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
469 boost::posix_time::ptime retryTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
470 boost::posix_time::time_duration runtime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
471 bool pauseScheduled_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
472 ErrorCode lastErrorCode_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
473 float lastProgress_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
474 Json::Value lastStatus_;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
475
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
476 void SetStateInternal(JobState state)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
477 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
478 const boost::posix_time::ptime now = boost::posix_time::microsec_clock::universal_time();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
479
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
480 if (state_ == JobState_Running)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
481 {
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
482 runtime_ += (now - lastStateChangeTime);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
483 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
484
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
485 state_ = state;
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
486 lastStateChangeTime = now;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
487 pauseScheduled_ = false;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
488 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
489
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
490 public:
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
491 JobHandler(IJob* job,
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
492 int priority) :
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
493 id_(Toolbox::GenerateUuid()),
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
494 state_(JobState_Pending),
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
495 job_(job),
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
496 priority_(priority),
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
497 creationTime_(boost::posix_time::microsec_clock::universal_time()),
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
498 lastStateChangeTime(creationTime_),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
499 runtime_(boost::posix_time::milliseconds(0)),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
500 pauseScheduled_(false),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
501 lastErrorCode_(ErrorCode_Success),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
502 lastProgress_(0)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
503 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
504 if (job == NULL)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
505 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
506 throw OrthancException(ErrorCode_NullPointer);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
507 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
508 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
509
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
510 const std::string& GetId() const
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
511 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
512 return id_;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
513 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
514
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
515 void SetPriority(int priority)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
516 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
517 priority_ = priority;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
518 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
519
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
520 int GetPriority() const
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
521 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
522 return priority_;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
523 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
524
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
525 JobState GetState() const
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
526 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
527 return state_;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
528 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
529
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
530 void SetState(JobState state)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
531 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
532 if (state == JobState_Retry)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
533 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
534 // Use "SetRetryState()"
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
535 throw OrthancException(ErrorCode_BadSequenceOfCalls);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
536 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
537 else
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
538 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
539 SetStateInternal(state);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
540 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
541 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
542
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
543 void SetRetryState(unsigned int timeout)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
544 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
545 if (state_ == JobState_Running)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
546 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
547 SetStateInternal(JobState_Retry);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
548 retryTime_ = (boost::posix_time::microsec_clock::universal_time() +
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
549 boost::posix_time::milliseconds(timeout));
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
550 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
551 else
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
552 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
553 // Only valid for running jobs
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
554 throw OrthancException(ErrorCode_BadSequenceOfCalls);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
555 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
556 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
558 void SchedulePause()
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
559 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
560 if (state_ == JobState_Running)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
561 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
562 pauseScheduled_ = true;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
563 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
564 else
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
565 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
566 // Only valid for running jobs
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
567 throw OrthancException(ErrorCode_BadSequenceOfCalls);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
568 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
569 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
570
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
571 bool IsPauseScheduled()
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
572 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
573 return pauseScheduled_;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
574 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
575
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
576 bool IsRetryReady(const boost::posix_time::ptime& now) const
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
577 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
578 if (state_ != JobState_Retry)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
579 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
580 throw OrthancException(ErrorCode_BadSequenceOfCalls);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
581 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
582 else
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
583 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
584 return retryTime_ <= now;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
585 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
586 }
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
587
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
588 class JobLock
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
589 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
590 private:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
591 boost::mutex::scoped_lock lock_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
592 JobHandler& handler_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
593
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
594 public:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
595 JobLock(JobHandler& handler) :
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
596 lock_(handler.jobMutex_),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
597 handler_(handler)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
598 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
599 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
600
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
601 IJob& GetJob()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
602 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
603 return *handler_.job_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
604 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
605
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
606 void UpdateStatus()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
607 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
608 handler_.lastProgress_ = handler_.job_->GetProgress();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
609 handler_.job_->FormatStatus(handler_.lastStatus_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
610 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
611
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
612 void SetLastErrorCode(ErrorCode code)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
613 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
614 handler_.lastErrorCode_ = code;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
615 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
616 };
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
617 };
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
618
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
619
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
620 class JobsRegistry : public boost::noncopyable
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
621 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
622 private:
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
623 struct PriorityComparator
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
624 {
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
625 bool operator() (JobHandler*& a,
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
626 JobHandler*& b) const
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
627 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
628 return a->GetPriority() < b->GetPriority();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
629 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
630 };
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
631
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
632 typedef std::map<std::string, JobHandler*> JobsIndex;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
633 typedef std::list<const JobHandler*> CompletedJobs;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
634 typedef std::set<JobHandler*> RetryJobs;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
635 typedef std::priority_queue<JobHandler*,
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
636 std::vector<JobHandler*>, // Could be a "std::deque"
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
637 PriorityComparator> PendingJobs;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
638
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
639 boost::mutex mutex_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
640 JobsIndex jobsIndex_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
641 PendingJobs pendingJobs_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
642 CompletedJobs completedJobs_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
643 RetryJobs retryJobs_;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
644
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
645 boost::condition_variable pendingJobAvailable_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
646 size_t maxCompletedJobs_;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
647
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
648
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
649 #ifndef NDEBUG
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
650 bool IsPendingJob(const JobHandler& job) const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
651 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
652 PendingJobs copy = pendingJobs_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
653 while (!copy.empty())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
654 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
655 if (copy.top() == &job)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
656 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
657 return true;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
658 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
659
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
660 copy.pop();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
661 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
662
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
663 return false;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
664 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
665
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
666 bool IsCompletedJob(const JobHandler& job) const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
667 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
668 for (CompletedJobs::const_iterator it = completedJobs_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
669 it != completedJobs_.end(); ++it)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
670 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
671 if (*it == &job)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
672 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
673 return true;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
674 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
675 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
676
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
677 return false;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
678 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
679
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
680 bool IsRetryJob(JobHandler& job) const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
681 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
682 return retryJobs_.find(&job) != retryJobs_.end();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
683 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
684 #endif
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
685
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
686
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
687 void CheckInvariants()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
688 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
689 #ifndef NDEBUG
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
690 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
691 PendingJobs copy = pendingJobs_;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
692 while (!copy.empty())
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
693 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
694 assert(copy.top()->GetState() == JobState_Pending);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
695 copy.pop();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
696 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
697 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
698
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
699 assert(completedJobs_.size() <= maxCompletedJobs_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
700
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
701 for (CompletedJobs::const_iterator it = completedJobs_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
702 it != completedJobs_.end(); ++it)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
703 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
704 assert((*it)->GetState() == JobState_Success ||
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
705 (*it)->GetState() == JobState_Failure);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
706 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
707
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
708 for (RetryJobs::const_iterator it = retryJobs_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
709 it != retryJobs_.end(); ++it)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
710 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
711 assert((*it)->GetState() == JobState_Retry);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
712 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
713
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
714 for (JobsIndex::iterator it = jobsIndex_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
715 it != jobsIndex_.end(); ++it)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
716 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
717 JobHandler& job = *it->second;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
718
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
719 assert(job.GetId() == it->first);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
720
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
721 switch (job.GetState())
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
722 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
723 case JobState_Pending:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
724 assert(!IsRetryJob(job) && IsPendingJob(job) && !IsCompletedJob(job));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
725 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
726
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
727 case JobState_Success:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
728 case JobState_Failure:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
729 assert(!IsRetryJob(job) && !IsPendingJob(job) && IsCompletedJob(job));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
730 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
731
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
732 case JobState_Retry:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
733 assert(IsRetryJob(job) && !IsPendingJob(job) && !IsCompletedJob(job));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
734 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
735
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
736 case JobState_Running:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
737 case JobState_Paused:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
738 assert(!IsRetryJob(job) && !IsPendingJob(job) && !IsCompletedJob(job));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
739 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
740
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
741 default:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
742 throw OrthancException(ErrorCode_InternalError);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
743 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
744 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
745 #endif
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
746 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
747
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
748
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
749 void ForgetOldCompletedJobs()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
750 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
751 if (maxCompletedJobs_ != 0)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
752 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
753 while (completedJobs_.size() > maxCompletedJobs_)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
754 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
755 assert(completedJobs_.front() != NULL);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
756
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
757 std::string id = completedJobs_.front()->GetId();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
758 assert(jobsIndex_.find(id) != jobsIndex_.end());
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
759
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
760 jobsIndex_.erase(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
761 delete(completedJobs_.front());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
762 completedJobs_.pop_front();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
763 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
764 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
765 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
766
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
767
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
768 void MarkRunningAsCompleted(JobHandler& job,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
769 bool success)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
770 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
771 LOG(INFO) << "Job has completed with " << (success ? "success" : "failure")
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
772 << ": " << job.GetId();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
773
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
774 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
775 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
776 assert(job.GetState() == JobState_Running);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
777
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
778 job.SetState(success ? JobState_Success : JobState_Failure);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
779
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
780 completedJobs_.push_back(&job);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
781 ForgetOldCompletedJobs();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
782
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
783 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
784 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
785
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
786
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
787 void MarkRunningAsRetry(JobHandler& job,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
788 unsigned int timeout)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
789 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
790 LOG(INFO) << "Job scheduled for retry in " << timeout << "ms: " << job.GetId();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
791
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
792 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
793 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
794
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
795 assert(job.GetState() == JobState_Running &&
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
796 retryJobs_.find(&job) == retryJobs_.end());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
797
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
798 retryJobs_.insert(&job);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
799 job.SetRetryState(timeout);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
800
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
801 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
802 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
803
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
804
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
805 void MarkRunningAsPaused(JobHandler& job)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
806 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
807 LOG(INFO) << "Job paused: " << job.GetId();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
808
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
809 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
810 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
811 assert(job.GetState() == JobState_Running);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
812
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
813 job.SetState(JobState_Paused);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
814
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
815 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
816 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
817
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
818
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
819 JobHandler* WaitPendingJob(unsigned int timeout)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
820 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
821 boost::mutex::scoped_lock lock(mutex_);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
822
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
823 while (pendingJobs_.empty())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
824 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
825 if (timeout == 0)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
826 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
827 pendingJobAvailable_.wait(lock);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
828 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
829 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
830 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
831 bool success = pendingJobAvailable_.timed_wait
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
832 (lock, boost::posix_time::milliseconds(timeout));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
833 if (!success)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
834 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
835 return NULL;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
836 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
837 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
838 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
839
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
840 JobHandler* job = pendingJobs_.top();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
841 pendingJobs_.pop();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
842
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
843 job->SetState(JobState_Running);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
844 return job;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
845 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
846
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
847
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
848 public:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
849 JobsRegistry() :
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
850 maxCompletedJobs_(10)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
851 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
852 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
853
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
854
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
855 ~JobsRegistry()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
856 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
857 for (JobsIndex::iterator it = jobsIndex_.begin(); it != jobsIndex_.end(); ++it)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
858 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
859 assert(it->second != NULL);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
860 delete it->second;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
861 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
862 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
863
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
864
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
865 void SetMaxCompletedJobs(size_t i)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
866 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
867 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
868 CheckInvariants();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
869
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
870 maxCompletedJobs_ = i;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
871 ForgetOldCompletedJobs();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
872
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
873 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
874 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
875
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
876
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
877 void ListJobs(std::set<std::string>& target)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
878 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
879 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
880 CheckInvariants();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
881
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
882 for (JobsIndex::const_iterator it = jobsIndex_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
883 it != jobsIndex_.end(); ++it)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
884 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
885 target.insert(it->first);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
886 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
887 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
888
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
889
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
890 void Submit(std::string& id,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
891 IJob* job, // Takes ownership
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
892 int priority)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
893 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
894 std::auto_ptr<JobHandler> handler(new JobHandler(job, priority));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
895
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
896 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
897 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
898
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
899 id = handler->GetId();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
900
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
901 pendingJobs_.push(handler.get());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
902 pendingJobAvailable_.notify_one();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
903
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
904 jobsIndex_.insert(std::make_pair(id, handler.release()));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
905
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
906 LOG(INFO) << "New job submitted: " << id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
907
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
908 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
909 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
910
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
911
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
912 void Submit(IJob* job, // Takes ownership
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
913 int priority)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
914 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
915 std::string id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
916 Submit(id, job, priority);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
917 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
918
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
919
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
920 void SetPriority(const std::string& id,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
921 int priority)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
922 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
923 LOG(INFO) << "Changing priority to " << priority << " for job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
924
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
925 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
926 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
927
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
928 JobsIndex::iterator found = jobsIndex_.find(id);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
929
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
930 if (found == jobsIndex_.end())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
931 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
932 LOG(WARNING) << "Unknown job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
933 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
934 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
935 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
936 found->second->SetPriority(priority);
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
937
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
938 if (found->second->GetState() == JobState_Pending)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
939 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
940 // If the job is pending, we need to reconstruct the
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
941 // priority queue, as the heap condition has changed
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
942
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
943 PendingJobs copy;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
944 std::swap(copy, pendingJobs_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
945
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
946 assert(pendingJobs_.empty());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
947 while (!copy.empty())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
948 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
949 pendingJobs_.push(copy.top());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
950 copy.pop();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
951 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
952 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
953 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
954
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
955 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
956 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
957
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
958
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
959 void Pause(const std::string& id)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
960 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
961 LOG(INFO) << "Pausing job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
962
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
963 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
964 CheckInvariants();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
965
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
966 JobsIndex::iterator found = jobsIndex_.find(id);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
967
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
968 if (found == jobsIndex_.end())
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
969 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
970 LOG(WARNING) << "Unknown job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
971 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
972 else
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
973 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
974 switch (found->second->GetState())
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
975 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
976 case JobState_Pending:
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
977 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
978 // If the job is pending, we need to reconstruct the
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
979 // priority queue to remove it
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
980 PendingJobs copy;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
981 std::swap(copy, pendingJobs_);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
982
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
983 assert(pendingJobs_.empty());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
984 while (!copy.empty())
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
985 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
986 if (copy.top()->GetId() != id)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
987 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
988 pendingJobs_.push(copy.top());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
989 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
990
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
991 copy.pop();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
992 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
993
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
994 found->second->SetState(JobState_Paused);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
995
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
996 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
997 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
998
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
999 case JobState_Retry:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1000 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1001 RetryJobs::iterator item = retryJobs_.find(found->second);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1002 assert(item != retryJobs_.end());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1003 retryJobs_.erase(item);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1004
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1005 found->second->SetState(JobState_Paused);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1006
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1007 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1008 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1009
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1010 case JobState_Paused:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1011 case JobState_Success:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1012 case JobState_Failure:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1013 // Nothing to be done
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1014 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1015
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1016 case JobState_Running:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1017 found->second->SchedulePause();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1018 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1019
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1020 default:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1021 throw OrthancException(ErrorCode_InternalError);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1022 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1023 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1024
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1025 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1026 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1027
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1028
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1029 void Resume(const std::string& id)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1030 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1031 LOG(INFO) << "Resuming job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1032
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1033 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1034 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1035
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1036 JobsIndex::iterator found = jobsIndex_.find(id);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1037
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1038 if (found == jobsIndex_.end())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1039 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1040 LOG(WARNING) << "Unknown job: " << id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1041 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1042 else if (found->second->GetState() != JobState_Paused)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1043 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1044 LOG(WARNING) << "Cannot resume a job that is not paused: " << id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1045 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1046 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1047 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1048 found->second->SetState(JobState_Pending);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1049 pendingJobs_.push(found->second);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1050 pendingJobAvailable_.notify_one();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1051 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1052
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1053 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1054 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1055
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1056
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1057 void Resubmit(const std::string& id)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1058 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1059 LOG(INFO) << "Resubmitting failed job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1060
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1061 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1062 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1063
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1064 JobsIndex::iterator found = jobsIndex_.find(id);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1065
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1066 if (found == jobsIndex_.end())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1067 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1068 LOG(WARNING) << "Unknown job: " << id;
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1069 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1070 else if (found->second->GetState() != JobState_Failure)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1071 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1072 LOG(WARNING) << "Cannot resubmit a job that has not failed: " << id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1073 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1074 else
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1075 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1076 bool ok = false;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1077 for (CompletedJobs::iterator it = completedJobs_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1078 it != completedJobs_.end(); ++it)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1079 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1080 if (*it == found->second)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1081 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1082 ok = true;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1083 completedJobs_.erase(it);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1084 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1085 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1086 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1087
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1088 assert(ok);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1089
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1090 found->second->SetState(JobState_Pending);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1091 pendingJobs_.push(found->second);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1092 pendingJobAvailable_.notify_one();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1093 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1094
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1095 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1096 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1097
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1098
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1099 void ScheduleRetries()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1100 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1101 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1102 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1103
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1104 RetryJobs copy;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1105 std::swap(copy, retryJobs_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1106
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1107 const boost::posix_time::ptime now = boost::posix_time::microsec_clock::universal_time();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1108
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1109 assert(retryJobs_.empty());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1110 for (RetryJobs::iterator it = copy.begin(); it != copy.end(); ++it)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1111 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1112 if ((*it)->IsRetryReady(now))
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1113 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1114 LOG(INFO) << "Retrying job: " << (*it)->GetId();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1115 (*it)->SetState(JobState_Pending);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1116 pendingJobs_.push(*it);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1117 pendingJobAvailable_.notify_one();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1118 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1119 else
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1120 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1121 retryJobs_.insert(*it);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1122 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1123 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1124
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1125 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1126 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1127
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1128
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1129 bool GetState(JobState& state,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1130 const std::string& id)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1131 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1132 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1133 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1134
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1135 JobsIndex::const_iterator it = jobsIndex_.find(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1136 if (it == jobsIndex_.end())
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1137 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1138 return false;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1139 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1140 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1141 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1142 state = it->second->GetState();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1143 return true;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1144 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1145 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1146
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1147
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1148 class RunningJob : public boost::noncopyable
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1149 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1150 private:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1151 JobsRegistry& that_;
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1152 JobHandler* handler_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1153 JobState targetState_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1154 unsigned int retryTimeout_;
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1155
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1156 public:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1157 RunningJob(JobsRegistry& that,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1158 unsigned int timeout) :
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1159 that_(that),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1160 handler_(NULL),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1161 targetState_(JobState_Failure),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1162 retryTimeout_(0)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1163 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1164 handler_ = that_.WaitPendingJob(timeout);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1165 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1166
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1167 ~RunningJob()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1168 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1169 if (IsValid())
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1170 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1171 switch (targetState_)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1172 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1173 case JobState_Failure:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1174 that_.MarkRunningAsCompleted(*handler_, false);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1175 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1176
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1177 case JobState_Success:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1178 that_.MarkRunningAsCompleted(*handler_, true);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1179 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1180
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1181 case JobState_Paused:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1182 that_.MarkRunningAsPaused(*handler_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1183 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1184
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1185 case JobState_Retry:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1186 that_.MarkRunningAsRetry(*handler_, retryTimeout_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1187 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1188
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1189 default:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1190 assert(0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1191 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1192 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1193 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1194
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1195 bool IsValid() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1196 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1197 return handler_ != NULL;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1198 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1199
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1200 const std::string& GetId() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1201 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1202 if (IsValid())
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1203 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1204 return handler_->GetId();
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1205 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1206 else
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1207 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1208 throw OrthancException(ErrorCode_BadSequenceOfCalls);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1209 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1210 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1211
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1212 int GetPriority() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1213 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1214 if (IsValid())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1215 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1216 return handler_->GetPriority();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1217 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1218 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1219 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1220 throw OrthancException(ErrorCode_BadSequenceOfCalls);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1221 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1222 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1223
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1224 bool IsPauseScheduled()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1225 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1226 if (!IsValid())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1227 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1228 throw OrthancException(ErrorCode_BadSequenceOfCalls);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1229 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1230
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1231 boost::mutex::scoped_lock lock(that_.mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1232 that_.CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1233 assert(handler_->GetState() == JobState_Running);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1234
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1235 return handler_->IsPauseScheduled();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1236 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1237
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1238 void MarkSuccess()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1239 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1240 if (!IsValid())
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1241 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1242 throw OrthancException(ErrorCode_BadSequenceOfCalls);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1243 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1244
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1245 targetState_ = JobState_Success;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1246 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1247
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1248 void MarkFailure()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1249 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1250 if (!IsValid())
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1251 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1252 throw OrthancException(ErrorCode_BadSequenceOfCalls);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1253 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1254
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1255 targetState_ = JobState_Failure;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1256 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1257
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1258 void SchedulePause()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1259 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1260 if (!IsValid())
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1261 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1262 throw OrthancException(ErrorCode_BadSequenceOfCalls);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1263 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1264
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1265 targetState_ = JobState_Paused;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1266 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1267
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1268 void MarkRetry(unsigned int timeout)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1269 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1270 if (!IsValid())
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1271 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1272 throw OrthancException(ErrorCode_BadSequenceOfCalls);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1273 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1274
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1275 targetState_ = JobState_Retry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1276 retryTimeout_ = timeout;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1277 }
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1278
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1279 void ExecuteStep()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1280 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1281 if (!IsValid())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1282 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1283 throw OrthancException(ErrorCode_BadSequenceOfCalls);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1284 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1285
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1286 if (handler_->IsPauseScheduled())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1287 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1288 targetState_ = JobState_Paused;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1289 return;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1290 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1291
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1292 std::auto_ptr<JobStepResult> result;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1293
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1294 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1295 JobHandler::JobLock lock(*handler_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1296
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1297 bool ok = false;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1298
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1299 try
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1300 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1301 result.reset(lock.GetJob().ExecuteStep());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1302 lock.UpdateStatus();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1303 ok = true;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1304 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1305 catch (OrthancException& e)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1306 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1307 lock.SetLastErrorCode(e.GetErrorCode());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1308 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1309 catch (boost::bad_lexical_cast&)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1310 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1311 lock.SetLastErrorCode(ErrorCode_BadFileFormat);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1312 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1313 catch (...)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1314 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1315 lock.SetLastErrorCode(ErrorCode_InternalError);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1316 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1317
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1318 if (ok)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1319 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1320 lock.SetLastErrorCode(ErrorCode_Success);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1321 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1322 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1323 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1324 result.reset(new JobStepResult(JobStepStatus_Failure));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1325 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1326 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1327
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1328 switch (result->GetStatus())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1329 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1330 case JobStepStatus_Success:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1331 targetState_ = JobState_Success;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1332 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1333
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1334 case JobStepStatus_Failure:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1335 targetState_ = JobState_Failure;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1336 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1337
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1338 case JobStepStatus_Continue:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1339 targetState_ = JobState_Running;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1340 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1341
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1342 case JobStepStatus_Retry:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1343 targetState_ = JobState_Retry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1344 retryTimeout_ = dynamic_cast<RetryResult&>(*result).GetTimeout();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1345 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1346
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1347 default:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1348 throw OrthancException(ErrorCode_InternalError);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1349 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1350 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1351 };
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1352 };
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1353 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1354
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1355
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1356
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1357 class DummyJob : public Orthanc::IJob
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1358 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1359 private:
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1360 JobStepResult result_;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1361
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1362 public:
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1363 DummyJob() :
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1364 result_(Orthanc::JobStepStatus_Success)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1365 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1366 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1367
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1368 explicit DummyJob(JobStepResult result) :
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1369 result_(result)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1370 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1371 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1372
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1373 virtual JobStepResult* ExecuteStep()
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1374 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1375 return new JobStepResult(result_);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1376 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1377
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1378 virtual void ReleaseResources()
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1379 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1380 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1381
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1382 virtual float GetProgress()
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1383 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1384 return 0;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1385 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1386
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1387 virtual void FormatStatus(Json::Value& value)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1388 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1389 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1390 };
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1391
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1392
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1393 static bool CheckState(Orthanc::JobsRegistry& registry,
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1394 const std::string& id,
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1395 Orthanc::JobState state)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1396 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1397 Orthanc::JobState s;
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1398 if (registry.GetState(s, id))
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1399 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1400 return state == s;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1401 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1402 else
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1403 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1404 return false;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1405 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1406 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1407
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1408
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1409 TEST(JobsRegistry, Priority)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1410 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1411 JobsRegistry registry;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1412
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1413 std::string i1, i2, i3, i4;
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1414 registry.Submit(i1, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1415 registry.Submit(i2, new DummyJob(), 30);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1416 registry.Submit(i3, new DummyJob(), 20);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1417 registry.Submit(i4, new DummyJob(), 5);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1418
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1419 registry.SetMaxCompletedJobs(2);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1420
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1421 std::set<std::string> id;
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1422 registry.ListJobs(id);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1423
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1424 ASSERT_EQ(4u, id.size());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1425 ASSERT_TRUE(id.find(i1) != id.end());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1426 ASSERT_TRUE(id.find(i2) != id.end());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1427 ASSERT_TRUE(id.find(i3) != id.end());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1428 ASSERT_TRUE(id.find(i4) != id.end());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1429
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1430 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1431
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1432 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1433 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1434 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1435 ASSERT_EQ(30, job.GetPriority());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1436 ASSERT_EQ(i2, job.GetId());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1437
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1438 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Running));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1439 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1440
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1441 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Failure));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1442 ASSERT_TRUE(CheckState(registry, i3, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1443
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1444 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1445 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1446 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1447 ASSERT_EQ(20, job.GetPriority());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1448 ASSERT_EQ(i3, job.GetId());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1449
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1450 job.MarkSuccess();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1451
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1452 ASSERT_TRUE(CheckState(registry, i3, Orthanc::JobState_Running));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1453 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1454
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1455 ASSERT_TRUE(CheckState(registry, i3, Orthanc::JobState_Success));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1456
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1457 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1458 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1459 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1460 ASSERT_EQ(10, job.GetPriority());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1461 ASSERT_EQ(i1, job.GetId());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1462 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1463
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1464 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1465 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1466 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1467 ASSERT_EQ(5, job.GetPriority());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1468 ASSERT_EQ(i4, job.GetId());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1469 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1470
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1471 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1472 JobsRegistry::RunningJob job(registry, 1);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1473 ASSERT_FALSE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1474 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1475
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1476 Orthanc::JobState s;
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1477 ASSERT_TRUE(registry.GetState(s, i1));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1478 ASSERT_FALSE(registry.GetState(s, i2)); // Removed because oldest
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1479 ASSERT_FALSE(registry.GetState(s, i3)); // Removed because second oldest
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1480 ASSERT_TRUE(registry.GetState(s, i4));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1481
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1482 registry.SetMaxCompletedJobs(1); // (*)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1483 ASSERT_FALSE(registry.GetState(s, i1)); // Just discarded by (*)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1484 ASSERT_TRUE(registry.GetState(s, i4));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1485 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1486
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1487
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1488 TEST(JobsRegistry, Simultaneous)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1489 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1490 JobsRegistry registry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1491
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1492 std::string i1, i2;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1493 registry.Submit(i1, new DummyJob(), 20);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1494 registry.Submit(i2, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1495
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1496 ASSERT_TRUE(CheckState(registry, i1, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1497 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1498
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1499 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1500 JobsRegistry::RunningJob job1(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1501 JobsRegistry::RunningJob job2(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1502
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1503 ASSERT_TRUE(job1.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1504 ASSERT_TRUE(job2.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1505
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1506 job1.MarkFailure();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1507 job2.MarkSuccess();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1508
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1509 ASSERT_TRUE(CheckState(registry, i1, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1510 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1511 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1512
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1513 ASSERT_TRUE(CheckState(registry, i1, Orthanc::JobState_Failure));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1514 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1515 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1516
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1517
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1518 TEST(JobsRegistry, Resubmit)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1519 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1520 JobsRegistry registry;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1521
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1522 std::string id;
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1523 registry.Submit(id, new DummyJob(), 10);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1524
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1525 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1526
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1527 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1528 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1529
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1530 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1531 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1532 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1533 job.MarkFailure();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1534
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1535 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1536
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1537 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1538 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1539 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1540
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1541 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Failure));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1542
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1543 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1544 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1545
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1546 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1547 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1548 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1549 ASSERT_EQ(id, job.GetId());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1550
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1551 job.MarkSuccess();
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1552 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1553 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1554
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1555 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1557 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1558 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1559 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1560
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1561
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1562 TEST(JobsRegistry, Retry)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1563 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1564 JobsRegistry registry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1565
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1566 std::string id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1567 registry.Submit(id, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1568
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1569 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1570
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1571 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1572 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1573 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1574 job.MarkRetry(0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1575
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1576 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1577 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1578
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1579 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Retry));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1580
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1581 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1582 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Retry));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1583
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1584 registry.ScheduleRetries();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1585 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1586
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1587 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1588 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1589 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1590 job.MarkSuccess();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1591
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1592 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1593 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1594
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1595 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1596 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1597
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1598
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1599 TEST(JobsRegistry, PausePending)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1600 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1601 JobsRegistry registry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1602
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1603 std::string id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1604 registry.Submit(id, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1605
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1606 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1607
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1608 registry.Pause(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1609 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1610
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1611 registry.Pause(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1612 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1613
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1614 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1615 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1616
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1617 registry.Resume(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1618 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1619 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1620
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1621
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1622 TEST(JobsRegistry, PauseRunning)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1623 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1624 JobsRegistry registry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1625
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1626 std::string id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1627 registry.Submit(id, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1628
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1629 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1630
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1631 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1632 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1633 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1634
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1635 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1636 job.SchedulePause();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1637 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1638 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1639
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1640 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1641
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1642 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1643 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1644
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1645 registry.Resume(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1646 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1647
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1648 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1649 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1650 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1651
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1652 job.MarkSuccess();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1653 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1654 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1655
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1656 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1657 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1658
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1659
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1660 TEST(JobsRegistry, PauseRetry)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1661 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1662 JobsRegistry registry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1663
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1664 std::string id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1665 registry.Submit(id, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1666
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1667 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1668
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1669 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1670 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1671 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1672
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1673 job.MarkRetry(0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1674 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1675 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1676
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1677 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Retry));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1678
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1679 registry.Pause(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1680 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1681
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1682 registry.Resume(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1683 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1684
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1685 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1686 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1687 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1688
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1689 job.MarkSuccess();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1690 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1691 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1692
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1693 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1694 }