It is a problem because Investigations are currently identified by
name, visitId, facilityCycle, instrument where both facilityCycle and instrument may be null. I need to get rid of constraints with nullable fields as database support is very poor.
I would like to change it to being identified just by facility and name. This also makes it clean for the icat data server. However as both diamond and isis use the combination of name and visitid to make it unique this is not practical. So instead I propose that Investigations have uniqueness constraints on facility, name, visitid
this implies that visitids must non-nullable
Steve