| 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. |