1 |
tdb |
1.1 |
Minutes of meeting 29/11/00 @ 5pm |
2 |
|
|
Location: UKC Computer Science Multimedia Lab then |
3 |
|
|
Eliot College computer room. |
4 |
|
|
|
5 |
|
|
Present: ajm4, tdb1 |
6 |
|
|
Absent: None |
7 |
|
|
|
8 |
|
|
AJ and Tim met to discuss and work on the code tidying |
9 |
|
|
issues. It was decided, afterwards, that the work should be |
10 |
|
|
documented in the form of minutes, so that a record can be |
11 |
|
|
made of what's happened. |
12 |
|
|
|
13 |
|
|
Meeting start (MM Lab): 17:00 |
14 |
|
|
Break for dinner: 22:00 |
15 |
|
|
Resume (Eliot) : 22:30 |
16 |
|
|
Finish : 04:30 |
17 |
|
|
|
18 |
|
|
|
19 |
|
|
Firstly a new class was introduced to the system. This |
20 |
|
|
class, called the ReferenceManager, was originally designed |
21 |
|
|
to hold references to the various items that a system |
22 |
|
|
component (ie. the filter) would require. This would cut |
23 |
|
|
down on passing of references between classes. |
24 |
|
|
|
25 |
|
|
However, as the was implemented it became clear that it |
26 |
|
|
could infact handle the CORBA references as well, and then, |
27 |
|
|
further to that, all the CORBA initialisation. In the end it |
28 |
|
|
ended up handling all the CORBA code, and held references to |
29 |
|
|
all the things required by a system component. |
30 |
|
|
|
31 |
|
|
The class was designed as a singleton, again to cut down on |
32 |
|
|
reference passing. Generally the class needs to be |
33 |
|
|
initialised before use, and this will be done in the main |
34 |
|
|
method of the component it's involved in. From then on a |
35 |
|
|
reference can be obtained by any class within the component |
36 |
|
|
using one simple method call. |
37 |
|
|
|
38 |
|
|
This has significantly reduced the complexity of the main |
39 |
|
|
methods of the system components, and allowed a clear line |
40 |
|
|
to be drawn between the CORBA code and the rest of the code. |
41 |
|
|
This has also meant that all the CORBA error handling is |
42 |
|
|
focused in one place, and can therefore be more thoroughly |
43 |
|
|
dealt with. |
44 |
|
|
|
45 |
|
|
|
46 |
|
|
The next item to be dealt with was the packaging. This was a |
47 |
|
|
major, but required, task and took some organising. The |
48 |
|
|
whole system now resides below the following package, |
49 |
|
|
|
50 |
|
|
uk.ac.ukc.iscream; |
51 |
|
|
|
52 |
|
|
The currently list of subpackages are, |
53 |
|
|
|
54 |
|
|
uk.ac.ukc.iscream.core; |
55 |
|
|
uk.ac.ukc.iscream.core.loggers; |
56 |
|
|
uk.ac.ukc.iscream.filter; |
57 |
|
|
uk.ac.ukc.iscream.filtermanager; |
58 |
|
|
uk.ac.ukc.iscream.rootfilter; |
59 |
|
|
uk.ac.ukc.iscream.util; |
60 |
|
|
|
61 |
|
|
The first five containing the existing components, but the |
62 |
|
|
last has been created to house more general purpose classes. |
63 |
|
|
Currently it has the xml parsing classes, and the |
64 |
|
|
ReferenceManager. It will, in the future, hold a class to |
65 |
|
|
deal with naming and generation of toString()'s. |
66 |
|
|
|
67 |
|
|
The inclusion of these packages has made the directory |
68 |
|
|
structure look a bit unwieldy, but as you'll see further |
69 |
|
|
down makefile's have been written to make compilation |
70 |
|
|
easier. |
71 |
|
|
|
72 |
|
|
As a side note, the generated IDL classes reside in similar |
73 |
|
|
packages, although they are in a separate location to avoid |
74 |
|
|
confusion. Ultimately they will be merged. |
75 |
|
|
|
76 |
|
|
|
77 |
|
|
Makefile's have been generated for the whole system. They |
78 |
|
|
allow easy compilation of any component, or the whole thing. |
79 |
|
|
They also allow for easy cleaning of compiled class files. |
80 |
|
|
It is expected that they will in future support packaging of |
81 |
|
|
the system into JAR files. |
82 |
|
|
|
83 |
|
|
They work on a recursive structure, with a single root |
84 |
|
|
Makefile. Each directory contains it's own Makefile for the |
85 |
|
|
files it contains, with links to any subdirectories that |
86 |
|
|
also contain Makefiles. This makes the system much more |
87 |
|
|
manageable when adding, changing, and removing class files. |
88 |
|
|
Ultimately only the root Makefile need be used, and it will |
89 |
|
|
recursively execute all the others. |
90 |
|
|
|
91 |
|
|
|
92 |
|
|
As a result of the above packaging, and the ReferenceManager |
93 |
|
|
it became necessary to tidy large chunks of code. The main |
94 |
|
|
two done were the Filter and RootFilter. Both now make use |
95 |
|
|
of the ReferenceManager throughout, and have a nice common |
96 |
|
|
theme through them - similar variable names, and class |
97 |
|
|
structure. These classes are now much nearer the required |
98 |
|
|
standards. |
99 |
|
|
|
100 |
|
|
Also modified were the loggers in the core. They now reside |
101 |
|
|
in a seperate subdirectory of the core, although this could |
102 |
|
|
be moved. This layer of seperation makes life much easier |
103 |
|
|
for managing and adding loggers. |
104 |
|
|
|
105 |
|
|
|
106 |
|
|
That was about it for changes. A few bugs were fixed as |
107 |
|
|
things progressed, but the functionality of the system (from |
108 |
|
|
an external point of view) remained pretty much the same |
109 |
|
|
throughout. |
110 |
|
|
|
111 |
|
|
A short list of "nice features" to be added was prepared. |
112 |
|
|
This list is in no certain order, and there is no gaurantee |
113 |
|
|
they'll even be done in the near future. |
114 |
|
|
|
115 |
|
|
- redesign the toString() methods. Modifying them, at |
116 |
|
|
present requires changing every class. This is bad. A |
117 |
|
|
new class in the util package could be created to |
118 |
|
|
centralise this behaviour. |
119 |
|
|
The toString() method is mainly used for logging |
120 |
|
|
purposes. |
121 |
|
|
|
122 |
|
|
- implement "remote logging" sessions. This would require |
123 |
|
|
modifying the exist setup, but the majority of code is |
124 |
|
|
in place to support it already. The idea of remote |
125 |
|
|
logging is that a logging console could be fired up |
126 |
|
|
independently on the server, and allow the user to |
127 |
|
|
monitor the status of the system. |
128 |
|
|
|
129 |
|
|
- improve error handling, and exception catching. Fatal |
130 |
|
|
errors should be informative to the end user. |
131 |
|
|
|
132 |
|
|
- the SimpleSwingLogger is broken. It will, after quite a |
133 |
|
|
few thousand lines, run out of memory and crash. This |
134 |
|
|
bug was identified previously, but the cause only found |
135 |
|
|
during the course of this meet. |
136 |
|
|
|
137 |
|
|
- allow hosts (and clients?) to log to the central logger |
138 |
|
|
as well. This would require a "logger proxy" of sorts, |
139 |
|
|
but it is suggested that this could be encapsulated in |
140 |
|
|
the existing UDP packages - ie. generate a packet of |
141 |
|
|
type "logmsg" that the UDP reading class will direct |
142 |
|
|
straight into the system logger. |
143 |
|
|
|
144 |
|
|
- sort of garbage collection issues, specifically with the |
145 |
|
|
Configuration objects, although there may be other areas |
146 |
|
|
that have yet to be found. |
147 |
|
|
|
148 |
|
|
- the configuration system support dynamic updates, and |
149 |
|
|
the host makes good use of this feature. However, it |
150 |
|
|
would be nice if the server could implement this |
151 |
|
|
internally where possible. |
152 |
|
|
|
153 |
|
|
|
154 |
|
|
The semi-meeting was concluded at a rather late (or early) |
155 |
|
|
hour after some very productive work. |