annotate UnitTestsSources/MultiThreadingTests.cpp @ 2562:1e66fe3ddf9f jobs

refactoring
author Sebastien Jodogne <s.jodogne@gmail.com>
date Fri, 04 May 2018 17:28:47 +0200
parents 9b7680dee75d
children 98dfc1948d00
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
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
293 enum JobStepCode
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
294 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
295 JobStepCode_Success,
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
296 JobStepCode_Failure,
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
297 JobStepCode_Continue,
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
298 JobStepCode_Retry
2556
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:
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
305 JobStepCode status_;
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
306
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
307 public:
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
308 explicit JobStepResult(JobStepCode status) :
2557
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
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
317 JobStepCode GetCode() const
2556
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) :
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
331 JobStepResult(JobStepCode_Retry),
2557
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
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
360 struct JobStatus
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
361 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
362 ErrorCode errorCode_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
363 float progress_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
364 Json::Value description_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
365
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
366 JobStatus() :
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
367 errorCode_(ErrorCode_Success),
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
368 progress_(0),
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
369 description_(Json::objectValue)
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
370 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
371 }
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
372
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
373 JobStatus(ErrorCode code,
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
374 float progress) :
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
375 errorCode_(code),
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
376 progress_(progress),
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
377 description_(Json::objectValue)
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
378 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
379 if (progress < 0 ||
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
380 progress > 1)
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
381 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
382 throw OrthancException(ErrorCode_ParameterOutOfRange);
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
383 }
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
384 }
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
385 };
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
386
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
387
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
388 class JobInfo
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
389 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
390 private:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
391 std::string id_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
392 int priority_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
393 ErrorCode errorCode_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
394 JobState state_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
395 boost::posix_time::ptime infoTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
396 boost::posix_time::ptime creationTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
397 boost::posix_time::time_duration runtime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
398 boost::posix_time::ptime eta_;
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
399 JobStatus status_;
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
400
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
401 public:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
402 JobInfo(const std::string& id,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
403 int priority,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
404 JobState state,
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
405 const JobStatus& status,
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
406 const boost::posix_time::ptime& creationTime,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
407 const boost::posix_time::time_duration& runtime) :
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
408 id_(id),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
409 priority_(priority),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
410 state_(state),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
411 infoTime_(boost::posix_time::microsec_clock::universal_time()),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
412 creationTime_(creationTime),
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
413 runtime_(runtime),
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
414 status_(status)
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
415 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
416 float ms = static_cast<float>(runtime_.total_milliseconds());
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
417 float remaining = boost::math::llround(1.0f - status_.progress_) * ms;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
418 eta_ = infoTime_ + boost::posix_time::milliseconds(remaining);
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
419 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
420
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
421 const std::string& GetIdentifier() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
422 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
423 return id_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
424 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
425
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
426 int GetPriority() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
427 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
428 return priority_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
429 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
430
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
431 ErrorCode GetErrorCode() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
432 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
433 return errorCode_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
434 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
435
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
436 JobState GetState() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
437 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
438 return state_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
439 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
440
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
441 const boost::posix_time::ptime& GetInfoTime() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
442 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
443 return infoTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
444 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
445
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
446 const boost::posix_time::ptime& GetCreationTime() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
447 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
448 return creationTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
449 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
450
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
451 const boost::posix_time::time_duration& GetRuntime() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
452 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
453 return runtime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
454 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
455
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
456 const boost::posix_time::ptime& GetEstimatedTimeOfArrival() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
457 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
458 return eta_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
459 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
460
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
461 const JobStatus& GetStatus() const
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
462 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
463 return status_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
464 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
465
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
466 JobStatus& GetStatus()
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
467 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
468 return status_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
469 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
470 };
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
471
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
472
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
473 class JobHandler : public boost::noncopyable
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
474 {
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
475 private:
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
476 std::string id_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
477 JobState state_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
478 std::auto_ptr<IJob> job_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
479 int priority_; // "+inf()" means highest priority
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
480 boost::posix_time::ptime creationTime_;
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
481 boost::posix_time::ptime lastStateChangeTime_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
482 boost::posix_time::time_duration runtime_;
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
483 boost::posix_time::ptime retryTime_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
484 bool pauseScheduled_;
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
485 JobStatus lastStatus_;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
486
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
487 void SetStateInternal(JobState state)
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 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
490
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
491 if (state_ == JobState_Running)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
492 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
493 runtime_ += (now - lastStateChangeTime_);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
494 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
495
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
496 state_ = state;
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
497 lastStateChangeTime_ = now;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
498 pauseScheduled_ = false;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
499 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
500
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
501 public:
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
502 JobHandler(IJob* job,
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
503 int priority) :
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
504 id_(Toolbox::GenerateUuid()),
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
505 state_(JobState_Pending),
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
506 job_(job),
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
507 priority_(priority),
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
508 creationTime_(boost::posix_time::microsec_clock::universal_time()),
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
509 lastStateChangeTime_(creationTime_),
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
510 runtime_(boost::posix_time::milliseconds(0)),
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
511 retryTime_(creationTime_),
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
512 pauseScheduled_(false)
2557
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 if (job == NULL)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
515 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
516 throw OrthancException(ErrorCode_NullPointer);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
517 }
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 const std::string& GetId() 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 id_;
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
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
525 IJob& GetJob() const
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
526 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
527 assert(job_.get() != NULL);
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
528 return *job_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
529 }
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
530
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
531 void SetPriority(int priority)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
532 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
533 priority_ = priority;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
534 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
535
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
536 int GetPriority() const
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
537 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
538 return priority_;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
539 }
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 JobState GetState() const
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 return state_;
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
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
546 void SetState(JobState state)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
547 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
548 if (state == JobState_Retry)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
549 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
550 // Use "SetRetryState()"
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
551 throw OrthancException(ErrorCode_BadSequenceOfCalls);
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 else
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
554 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
555 SetStateInternal(state);
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
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
559 void SetRetryState(unsigned int timeout)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
560 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
561 if (state_ == JobState_Running)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
562 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
563 SetStateInternal(JobState_Retry);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
564 retryTime_ = (boost::posix_time::microsec_clock::universal_time() +
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
565 boost::posix_time::milliseconds(timeout));
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
566 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
567 else
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 // Only valid for running jobs
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
570 throw OrthancException(ErrorCode_BadSequenceOfCalls);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
571 }
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
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
574 void SchedulePause()
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 if (state_ == JobState_Running)
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 pauseScheduled_ = true;
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 else
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 // Only valid for running jobs
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
583 throw OrthancException(ErrorCode_BadSequenceOfCalls);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
584 }
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
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
587 bool IsPauseScheduled()
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
588 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
589 return pauseScheduled_;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
590 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
591
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
592 bool IsRetryReady(const boost::posix_time::ptime& now) const
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
593 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
594 if (state_ != JobState_Retry)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
595 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
596 throw OrthancException(ErrorCode_BadSequenceOfCalls);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
597 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
598 else
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
599 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
600 return retryTime_ <= now;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
601 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
602 }
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
603
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
604 JobStatus& GetLastStatus()
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
605 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
606 return lastStatus_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
607 }
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
608
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
609 const JobStatus& GetLastStatus() const
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
610 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
611 return lastStatus_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
612 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
613 };
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
614
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
615
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
616 class JobsRegistry : public boost::noncopyable
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
617 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
618 private:
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
619 struct PriorityComparator
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
620 {
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
621 bool operator() (JobHandler*& a,
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
622 JobHandler*& b) const
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
623 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
624 return a->GetPriority() < b->GetPriority();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
625 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
626 };
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 typedef std::map<std::string, JobHandler*> JobsIndex;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
629 typedef std::list<const JobHandler*> CompletedJobs;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
630 typedef std::set<JobHandler*> RetryJobs;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
631 typedef std::priority_queue<JobHandler*,
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
632 std::vector<JobHandler*>, // Could be a "std::deque"
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
633 PriorityComparator> PendingJobs;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
634
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
635 boost::mutex mutex_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
636 JobsIndex jobsIndex_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
637 PendingJobs pendingJobs_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
638 CompletedJobs completedJobs_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
639 RetryJobs retryJobs_;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
640
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
641 boost::condition_variable pendingJobAvailable_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
642 size_t maxCompletedJobs_;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
643
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
644
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
645 #ifndef NDEBUG
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
646 bool IsPendingJob(const JobHandler& job) const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
647 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
648 PendingJobs copy = pendingJobs_;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
649 while (!copy.empty())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
650 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
651 if (copy.top() == &job)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
652 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
653 return true;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
654 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
655
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
656 copy.pop();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
657 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
658
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
659 return false;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
660 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
661
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
662 bool IsCompletedJob(const JobHandler& job) const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
663 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
664 for (CompletedJobs::const_iterator it = completedJobs_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
665 it != completedJobs_.end(); ++it)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
666 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
667 if (*it == &job)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
668 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
669 return true;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
670 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
671 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
672
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
673 return false;
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 bool IsRetryJob(JobHandler& job) const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
677 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
678 return retryJobs_.find(&job) != retryJobs_.end();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
679 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
680 #endif
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
681
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
682
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
683 void CheckInvariants()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
684 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
685 #ifndef NDEBUG
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
686 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
687 PendingJobs copy = pendingJobs_;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
688 while (!copy.empty())
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
689 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
690 assert(copy.top()->GetState() == JobState_Pending);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
691 copy.pop();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
692 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
693 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
694
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
695 assert(completedJobs_.size() <= maxCompletedJobs_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
696
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
697 for (CompletedJobs::const_iterator it = completedJobs_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
698 it != completedJobs_.end(); ++it)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
699 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
700 assert((*it)->GetState() == JobState_Success ||
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
701 (*it)->GetState() == JobState_Failure);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
702 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
703
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
704 for (RetryJobs::const_iterator it = retryJobs_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
705 it != retryJobs_.end(); ++it)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
706 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
707 assert((*it)->GetState() == JobState_Retry);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
708 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
709
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
710 for (JobsIndex::iterator it = jobsIndex_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
711 it != jobsIndex_.end(); ++it)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
712 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
713 JobHandler& job = *it->second;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
714
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
715 assert(job.GetId() == it->first);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
716
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
717 switch (job.GetState())
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
718 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
719 case JobState_Pending:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
720 assert(!IsRetryJob(job) && IsPendingJob(job) && !IsCompletedJob(job));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
721 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
722
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
723 case JobState_Success:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
724 case JobState_Failure:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
725 assert(!IsRetryJob(job) && !IsPendingJob(job) && IsCompletedJob(job));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
726 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
727
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
728 case JobState_Retry:
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_Running:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
733 case JobState_Paused:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
734 assert(!IsRetryJob(job) && !IsPendingJob(job) && !IsCompletedJob(job));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
735 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
736
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
737 default:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
738 throw OrthancException(ErrorCode_InternalError);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
739 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
740 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
741 #endif
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
742 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
743
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
744
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
745 void ForgetOldCompletedJobs()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
746 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
747 if (maxCompletedJobs_ != 0)
2557
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 while (completedJobs_.size() > maxCompletedJobs_)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
750 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
751 assert(completedJobs_.front() != NULL);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
752
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
753 std::string id = completedJobs_.front()->GetId();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
754 assert(jobsIndex_.find(id) != jobsIndex_.end());
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
755
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
756 jobsIndex_.erase(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
757 delete(completedJobs_.front());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
758 completedJobs_.pop_front();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
759 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
760 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
761 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
762
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
763
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
764 void MarkRunningAsCompleted(JobHandler& job,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
765 bool success)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
766 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
767 LOG(INFO) << "Job has completed with " << (success ? "success" : "failure")
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
768 << ": " << job.GetId();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
769
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
770 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
771 assert(job.GetState() == JobState_Running);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
772
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
773 job.SetState(success ? JobState_Success : JobState_Failure);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
774
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
775 completedJobs_.push_back(&job);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
776 ForgetOldCompletedJobs();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
777
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
778 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
779 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
780
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
781
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
782 void MarkRunningAsRetry(JobHandler& job,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
783 unsigned int timeout)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
784 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
785 LOG(INFO) << "Job scheduled for retry in " << timeout << "ms: " << job.GetId();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
786
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
787 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
788
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
789 assert(job.GetState() == JobState_Running &&
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
790 retryJobs_.find(&job) == retryJobs_.end());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
791
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
792 retryJobs_.insert(&job);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
793 job.SetRetryState(timeout);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
794
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
795 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
796 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
797
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
798
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
799 void MarkRunningAsPaused(JobHandler& job)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
800 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
801 LOG(INFO) << "Job paused: " << job.GetId();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
802
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
803 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
804 assert(job.GetState() == JobState_Running);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
805
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
806 job.SetState(JobState_Paused);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
807
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
808 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
809 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
810
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
811
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
812 public:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
813 JobsRegistry() :
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
814 maxCompletedJobs_(10)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
815 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
816 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
817
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
818
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
819 ~JobsRegistry()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
820 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
821 for (JobsIndex::iterator it = jobsIndex_.begin(); it != jobsIndex_.end(); ++it)
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 assert(it->second != NULL);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
824 delete it->second;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
825 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
826 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
827
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
828
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
829 void SetMaxCompletedJobs(size_t i)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
830 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
831 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
832 CheckInvariants();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
833
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
834 maxCompletedJobs_ = i;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
835 ForgetOldCompletedJobs();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
836
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
837 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
838 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
839
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
840
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
841 void ListJobs(std::set<std::string>& target)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
842 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
843 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
844 CheckInvariants();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
845
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
846 for (JobsIndex::const_iterator it = jobsIndex_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
847 it != jobsIndex_.end(); ++it)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
848 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
849 target.insert(it->first);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
850 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
851 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
852
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
853
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
854 void Submit(std::string& id,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
855 IJob* job, // Takes ownership
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
856 int priority)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
857 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
858 std::auto_ptr<JobHandler> handler(new JobHandler(job, priority));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
859
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
860 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
861 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
862
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
863 id = handler->GetId();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
864
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
865 pendingJobs_.push(handler.get());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
866 pendingJobAvailable_.notify_one();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
867
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
868 jobsIndex_.insert(std::make_pair(id, handler.release()));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
869
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
870 LOG(INFO) << "New job submitted: " << id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
871
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
872 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
873 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
874
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
875
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
876 void Submit(IJob* job, // Takes ownership
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
877 int priority)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
878 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
879 std::string id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
880 Submit(id, job, priority);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
881 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
882
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
883
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
884 void SetPriority(const std::string& id,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
885 int priority)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
886 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
887 LOG(INFO) << "Changing priority to " << priority << " for job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
888
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
889 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
890 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
891
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
892 JobsIndex::iterator found = jobsIndex_.find(id);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
893
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
894 if (found == jobsIndex_.end())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
895 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
896 LOG(WARNING) << "Unknown job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
897 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
898 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
899 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
900 found->second->SetPriority(priority);
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
901
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
902 if (found->second->GetState() == JobState_Pending)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
903 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
904 // If the job is pending, we need to reconstruct the
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
905 // priority queue, as the heap condition has changed
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
906
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
907 PendingJobs copy;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
908 std::swap(copy, pendingJobs_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
909
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
910 assert(pendingJobs_.empty());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
911 while (!copy.empty())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
912 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
913 pendingJobs_.push(copy.top());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
914 copy.pop();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
915 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
916 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
917 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
918
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
919 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
920 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
921
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
922
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
923 void Pause(const std::string& id)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
924 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
925 LOG(INFO) << "Pausing job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
926
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
927 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
928 CheckInvariants();
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 JobsIndex::iterator found = jobsIndex_.find(id);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
931
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
932 if (found == jobsIndex_.end())
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 LOG(WARNING) << "Unknown job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
935 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
936 else
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
937 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
938 switch (found->second->GetState())
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
939 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
940 case JobState_Pending:
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
941 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
942 // If the job is pending, we need to reconstruct the
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
943 // priority queue to remove it
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
944 PendingJobs copy;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
945 std::swap(copy, pendingJobs_);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
946
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
947 assert(pendingJobs_.empty());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
948 while (!copy.empty())
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
949 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
950 if (copy.top()->GetId() != id)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
951 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
952 pendingJobs_.push(copy.top());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
953 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
954
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
955 copy.pop();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
956 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
957
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
958 found->second->SetState(JobState_Paused);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
959
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
960 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
961 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
962
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
963 case JobState_Retry:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
964 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
965 RetryJobs::iterator item = retryJobs_.find(found->second);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
966 assert(item != retryJobs_.end());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
967 retryJobs_.erase(item);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
968
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
969 found->second->SetState(JobState_Paused);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
970
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
971 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
972 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
973
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
974 case JobState_Paused:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
975 case JobState_Success:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
976 case JobState_Failure:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
977 // Nothing to be done
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
978 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
979
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
980 case JobState_Running:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
981 found->second->SchedulePause();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
982 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
983
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
984 default:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
985 throw OrthancException(ErrorCode_InternalError);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
986 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
987 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
988
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
989 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
990 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
991
2557
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 void Resume(const std::string& id)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
994 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
995 LOG(INFO) << "Resuming job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
996
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
997 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
998 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
999
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1000 JobsIndex::iterator found = jobsIndex_.find(id);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1001
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1002 if (found == jobsIndex_.end())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1003 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1004 LOG(WARNING) << "Unknown job: " << id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1005 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1006 else if (found->second->GetState() != JobState_Paused)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1007 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1008 LOG(WARNING) << "Cannot resume a job that is not paused: " << id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1009 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1010 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1011 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1012 found->second->SetState(JobState_Pending);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1013 pendingJobs_.push(found->second);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1014 pendingJobAvailable_.notify_one();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1015 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1016
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1017 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1018 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1019
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1020
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1021 void Resubmit(const std::string& id)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1022 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1023 LOG(INFO) << "Resubmitting failed job: " << id;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1024
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1025 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1026 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1027
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1028 JobsIndex::iterator found = jobsIndex_.find(id);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1029
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1030 if (found == jobsIndex_.end())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1031 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1032 LOG(WARNING) << "Unknown job: " << id;
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1033 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1034 else if (found->second->GetState() != JobState_Failure)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1035 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1036 LOG(WARNING) << "Cannot resubmit a job that has not failed: " << id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1037 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1038 else
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1039 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1040 bool ok = false;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1041 for (CompletedJobs::iterator it = completedJobs_.begin();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1042 it != completedJobs_.end(); ++it)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1043 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1044 if (*it == found->second)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1045 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1046 ok = true;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1047 completedJobs_.erase(it);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1048 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1049 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1050 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1051
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1052 assert(ok);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1053
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1054 found->second->SetState(JobState_Pending);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1055 pendingJobs_.push(found->second);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1056 pendingJobAvailable_.notify_one();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1057 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1058
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1059 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1060 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1061
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1062
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1063 void ScheduleRetries()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1064 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1065 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1066 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1067
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1068 RetryJobs copy;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1069 std::swap(copy, retryJobs_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1070
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1071 const boost::posix_time::ptime now = boost::posix_time::microsec_clock::universal_time();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1072
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1073 assert(retryJobs_.empty());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1074 for (RetryJobs::iterator it = copy.begin(); it != copy.end(); ++it)
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 if ((*it)->IsRetryReady(now))
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1077 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1078 LOG(INFO) << "Retrying job: " << (*it)->GetId();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1079 (*it)->SetState(JobState_Pending);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1080 pendingJobs_.push(*it);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1081 pendingJobAvailable_.notify_one();
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1082 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1083 else
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1084 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1085 retryJobs_.insert(*it);
2557
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 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1088
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1089 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1090 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1091
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1092
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1093 bool GetState(JobState& state,
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1094 const std::string& id)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1095 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1096 boost::mutex::scoped_lock lock(mutex_);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1097 CheckInvariants();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1098
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1099 JobsIndex::const_iterator it = jobsIndex_.find(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1100 if (it == jobsIndex_.end())
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1101 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1102 return false;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1103 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1104 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1105 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1106 state = it->second->GetState();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1107 return true;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1108 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1109 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1110
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1111
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1112 class RunningJob : public boost::noncopyable
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1113 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1114 private:
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1115 JobsRegistry& registry_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1116 JobHandler* handler_; // Can only be accessed if the registry
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1117 // mutex is locked!
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1118 IJob* job_; // Will by design be in mutual exclusion,
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1119 // because only one RunningJob can be
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1120 // executed at a time on a JobHandler
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1121
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1122 std::string id_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1123 int priority_;
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1124 JobState targetState_;
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1125 unsigned int targetRetryTimeout_;
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1126
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1127 public:
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1128 RunningJob(JobsRegistry& registry,
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1129 unsigned int timeout) :
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1130 registry_(registry),
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1131 handler_(NULL),
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1132 targetState_(JobState_Failure),
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1133 targetRetryTimeout_(0)
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1134 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1135 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1136 boost::mutex::scoped_lock lock(registry_.mutex_);
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1137
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1138 while (registry_.pendingJobs_.empty())
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1139 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1140 if (timeout == 0)
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1141 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1142 registry_.pendingJobAvailable_.wait(lock);
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1143 }
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1144 else
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1145 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1146 bool success = registry_.pendingJobAvailable_.timed_wait
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1147 (lock, boost::posix_time::milliseconds(timeout));
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1148 if (!success)
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1149 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1150 // No pending job
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1151 return;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1152 }
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1153 }
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1154 }
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1155
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1156 handler_ = registry_.pendingJobs_.top();
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1157 registry_.pendingJobs_.pop();
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1158
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1159 assert(handler_->GetState() == JobState_Pending);
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1160 handler_->SetState(JobState_Running);
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1161
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1162 job_ = &handler_->GetJob();
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1163 id_ = handler_->GetId();
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1164 priority_ = handler_->GetPriority();
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1165 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1166 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1167
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1168 ~RunningJob()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1169 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1170 if (IsValid())
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1171 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1172 boost::mutex::scoped_lock lock(registry_.mutex_);
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1173
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1174 switch (targetState_)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1175 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1176 case JobState_Failure:
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1177 registry_.MarkRunningAsCompleted(*handler_, false);
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1178 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1179
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1180 case JobState_Success:
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1181 registry_.MarkRunningAsCompleted(*handler_, true);
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1182 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1183
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1184 case JobState_Paused:
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1185 registry_.MarkRunningAsPaused(*handler_);
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1186 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1187
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1188 case JobState_Retry:
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1189 registry_.MarkRunningAsRetry(*handler_, targetRetryTimeout_);
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1190 break;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1191
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1192 default:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1193 assert(0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1194 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1195 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1196 }
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1197
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1198 bool IsValid() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1199 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1200 return (handler_ != NULL &&
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1201 job_ != NULL);
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1202 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1203
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1204 const std::string& GetId() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1205 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1206 if (!IsValid())
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1207 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1208 throw OrthancException(ErrorCode_BadSequenceOfCalls);
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1209 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1210 else
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1211 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1212 return id_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1213 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1214 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1215
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1216 int GetPriority() const
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1217 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1218 if (!IsValid())
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1219 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1220 throw OrthancException(ErrorCode_BadSequenceOfCalls);
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1221 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1222 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1223 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1224 return priority_;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1225 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1226 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1227
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1228 bool IsPauseScheduled()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1229 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1230 if (!IsValid())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1231 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1232 throw OrthancException(ErrorCode_BadSequenceOfCalls);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1233 }
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1234 else
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1235 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1236 boost::mutex::scoped_lock lock(registry_.mutex_);
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1237 registry_.CheckInvariants();
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1238 assert(handler_->GetState() == JobState_Running);
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1239
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1240 return handler_->IsPauseScheduled();
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1241 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1242 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1243
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1244 void MarkSuccess()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1245 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1246 if (!IsValid())
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 throw OrthancException(ErrorCode_BadSequenceOfCalls);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1249 }
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1250 else
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1251 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1252 targetState_ = JobState_Success;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1253 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1254 }
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1255
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1256 void MarkFailure()
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1257 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1258 if (!IsValid())
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1259 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1260 throw OrthancException(ErrorCode_BadSequenceOfCalls);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1261 }
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1262 else
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1263 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1264 targetState_ = JobState_Failure;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1265 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1266 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1267
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1268 void MarkPause()
2558
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);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1273 }
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1274 else
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1275 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1276 targetState_ = JobState_Paused;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1277 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1278 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1279
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1280 void MarkRetry(unsigned int timeout)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1281 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1282 if (!IsValid())
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1283 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1284 throw OrthancException(ErrorCode_BadSequenceOfCalls);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1285 }
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1286 else
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1287 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1288 targetState_ = JobState_Retry;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1289 targetRetryTimeout_ = timeout;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1290 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1291 }
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1292
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1293 /*void ExecuteStep()
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1294 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1295 if (!IsValid())
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1296 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1297 throw OrthancException(ErrorCode_BadSequenceOfCalls);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1298 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1299
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1300 if (IsPauseScheduled())
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1301 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1302 targetState_ = JobState_Paused;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1303 return;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1304 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1305
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1306 std::auto_ptr<JobStepResult> result;
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1307 ErrorCode code;
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1308
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1309 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1310 bool ok = false;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1311
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1312 try
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1313 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1314 result.reset(job_->ExecuteStep());
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1315 ok = true;
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1316
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1317 if (result->GetCode() == JobStepCode_Failure)
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1318 {
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1319 code = ErrorCode_InternalError;
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1320 }
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1321 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1322 catch (OrthancException& e)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1323 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1324 code = e.GetErrorCode();
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1325 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1326 catch (boost::bad_lexical_cast&)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1327 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1328 code = ErrorCode_BadFileFormat;
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1329 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1330 catch (...)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1331 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1332 code = ErrorCode_InternalError;
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1333 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1334
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1335 if (ok)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1336 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1337 code = ErrorCode_Success;
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1338 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1339 else
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1340 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1341 result.reset(new JobStepResult(JobStepCode_Failure));
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1342 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1343 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1344
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1345 switch (result->GetCode())
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1346 {
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1347 case JobStepCode_Success:
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1348 targetState_ = JobState_Success;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1349 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1350
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1351 case JobStepCode_Failure:
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1352 targetState_ = JobState_Failure;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1353 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1354
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1355 case JobStepCode_Continue:
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1356 targetState_ = JobState_Running;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1357 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1358
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1359 case JobStepCode_Retry:
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1360 targetState_ = JobState_Retry;
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1361 targetRetryTimeout_ = dynamic_cast<RetryResult&>(*result).GetTimeout();
2559
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1362 break;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1363
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1364 default:
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1365 throw OrthancException(ErrorCode_InternalError);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2558
diff changeset
1366 }
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1367 }*/
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1368 };
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1369 };
2556
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2447
diff changeset
1370 }
2557
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
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1374 class DummyJob : public Orthanc::IJob
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1375 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1376 private:
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1377 JobStepResult result_;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1378
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1379 public:
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1380 DummyJob() :
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1381 result_(Orthanc::JobStepCode_Success)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1382 {
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
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1385 explicit DummyJob(JobStepResult result) :
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1386 result_(result)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1387 {
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 virtual JobStepResult* ExecuteStep()
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 return new JobStepResult(result_);
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1393 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1394
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1395 virtual void ReleaseResources()
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 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1398
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1399 virtual float GetProgress()
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1400 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1401 return 0;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1402 }
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 virtual void FormatStatus(Json::Value& value)
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
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1409
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1410 static bool CheckState(Orthanc::JobsRegistry& registry,
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1411 const std::string& id,
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1412 Orthanc::JobState state)
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1413 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1414 Orthanc::JobState s;
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1415 if (registry.GetState(s, id))
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1416 {
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1417 return state == s;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1418 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1419 else
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 return false;
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1422 }
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
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1425
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1426 TEST(JobsRegistry, Priority)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1427 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1428 JobsRegistry registry;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1429
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1430 std::string i1, i2, i3, i4;
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1431 registry.Submit(i1, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1432 registry.Submit(i2, new DummyJob(), 30);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1433 registry.Submit(i3, new DummyJob(), 20);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1434 registry.Submit(i4, new DummyJob(), 5);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1435
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1436 registry.SetMaxCompletedJobs(2);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1437
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1438 std::set<std::string> id;
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1439 registry.ListJobs(id);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1440
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1441 ASSERT_EQ(4u, id.size());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1442 ASSERT_TRUE(id.find(i1) != id.end());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1443 ASSERT_TRUE(id.find(i2) != id.end());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1444 ASSERT_TRUE(id.find(i3) != id.end());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1445 ASSERT_TRUE(id.find(i4) != id.end());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1446
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1447 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1448
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1449 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1450 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1451 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1452 ASSERT_EQ(30, job.GetPriority());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1453 ASSERT_EQ(i2, job.GetId());
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, i2, Orthanc::JobState_Running));
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 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Failure));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1459 ASSERT_TRUE(CheckState(registry, i3, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1460
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1461 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1462 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1463 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1464 ASSERT_EQ(20, job.GetPriority());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1465 ASSERT_EQ(i3, job.GetId());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1466
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1467 job.MarkSuccess();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1468
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1469 ASSERT_TRUE(CheckState(registry, i3, Orthanc::JobState_Running));
2557
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 ASSERT_TRUE(CheckState(registry, i3, Orthanc::JobState_Success));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1473
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1474 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1475 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1476 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1477 ASSERT_EQ(10, job.GetPriority());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1478 ASSERT_EQ(i1, job.GetId());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1479 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1480
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 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1483 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1484 ASSERT_EQ(5, job.GetPriority());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1485 ASSERT_EQ(i4, job.GetId());
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
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1488 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1489 JobsRegistry::RunningJob job(registry, 1);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1490 ASSERT_FALSE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1491 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1492
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1493 Orthanc::JobState s;
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1494 ASSERT_TRUE(registry.GetState(s, i1));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1495 ASSERT_FALSE(registry.GetState(s, i2)); // Removed because oldest
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1496 ASSERT_FALSE(registry.GetState(s, i3)); // Removed because second oldest
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1497 ASSERT_TRUE(registry.GetState(s, i4));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1498
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1499 registry.SetMaxCompletedJobs(1); // (*)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1500 ASSERT_FALSE(registry.GetState(s, i1)); // Just discarded by (*)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1501 ASSERT_TRUE(registry.GetState(s, i4));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1502 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1503
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1504
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1505 TEST(JobsRegistry, Simultaneous)
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1506 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1507 JobsRegistry registry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1508
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1509 std::string i1, i2;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1510 registry.Submit(i1, new DummyJob(), 20);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1511 registry.Submit(i2, new DummyJob(), 10);
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_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1514 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Pending));
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 JobsRegistry::RunningJob job1(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1518 JobsRegistry::RunningJob job2(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1519
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1520 ASSERT_TRUE(job1.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1521 ASSERT_TRUE(job2.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1522
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1523 job1.MarkFailure();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1524 job2.MarkSuccess();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1525
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1526 ASSERT_TRUE(CheckState(registry, i1, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1527 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1528 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1529
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1530 ASSERT_TRUE(CheckState(registry, i1, Orthanc::JobState_Failure));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1531 ASSERT_TRUE(CheckState(registry, i2, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1532 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1533
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1534
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1535 TEST(JobsRegistry, Resubmit)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1536 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1537 JobsRegistry registry;
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1538
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1539 std::string id;
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1540 registry.Submit(id, new DummyJob(), 10);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1541
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1542 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1543
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1544 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1545 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1546
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1547 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1548 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1549 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1550 job.MarkFailure();
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1551
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1552 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1553
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1554 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1555 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1556 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1557
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1558 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Failure));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1559
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1560 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1561 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1562
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1563 {
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1564 JobsRegistry::RunningJob job(registry, 0);
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1565 ASSERT_TRUE(job.IsValid());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1566 ASSERT_EQ(id, job.GetId());
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1567
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1568 job.MarkSuccess();
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1569 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
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 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1573
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1574 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1575 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1576 }
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 TEST(JobsRegistry, Retry)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1580 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1581 JobsRegistry registry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1582
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1583 std::string id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1584 registry.Submit(id, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1585
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1586 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1587
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1588 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1589 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1590 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1591 job.MarkRetry(0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1592
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1593 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1594 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1595
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1596 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Retry));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1597
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1598 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1599 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Retry));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1600
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1601 registry.ScheduleRetries();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1602 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1603
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1604 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1605 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1606 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1607 job.MarkSuccess();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1608
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1609 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1610 }
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1611
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1612 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1613 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1614
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1615
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1616 TEST(JobsRegistry, PausePending)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1617 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1618 JobsRegistry registry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1619
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1620 std::string id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1621 registry.Submit(id, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1622
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1623 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1624
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1625 registry.Pause(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1626 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1627
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1628 registry.Pause(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1629 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1630
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1631 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1632 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1633
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1634 registry.Resume(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1635 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
2557
b4516a6f214b state machine
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2556
diff changeset
1636 }
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1637
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1638
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1639 TEST(JobsRegistry, PauseRunning)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1640 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1641 JobsRegistry registry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1642
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1643 std::string id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1644 registry.Submit(id, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1645
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 registry.Resubmit(id);
2562
1e66fe3ddf9f refactoring
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2559
diff changeset
1653 job.MarkPause();
2558
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1654 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1655 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1656
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1657 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1658
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1659 registry.Resubmit(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1660 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1661
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1662 registry.Resume(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1663 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1664
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1665 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1666 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1667 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1668
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1669 job.MarkSuccess();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1670 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1671 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1672
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1673 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1674 }
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 TEST(JobsRegistry, PauseRetry)
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1678 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1679 JobsRegistry registry;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1680
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1681 std::string id;
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1682 registry.Submit(id, new DummyJob(), 10);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1683
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1684 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1685
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1686 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1687 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1688 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1689
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1690 job.MarkRetry(0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1691 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1692 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1693
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1694 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Retry));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1695
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1696 registry.Pause(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1697 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Paused));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1698
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1699 registry.Resume(id);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1700 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Pending));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1701
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1702 {
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1703 JobsRegistry::RunningJob job(registry, 0);
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1704 ASSERT_TRUE(job.IsValid());
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1705
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1706 job.MarkSuccess();
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1707 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Running));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1708 }
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1709
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1710 ASSERT_TRUE(CheckState(registry, id, Orthanc::JobState_Success));
Sebastien Jodogne <s.jodogne@gmail.com>
parents: 2557
diff changeset
1711 }