we should throw the NotImplementedException if workspace is null, not silently ignore