I've had a look at the relationships and it's making a little more sense. I've fiddled with the form and subform and set up the structure of each properly. I've also changed the control source for the combo box. The combo box should be looking to the ReviewedBy table and the ID number should be saved to the ReviewedID field in the ReviewVesselView join table. Your structure is actually not far off and you've done well to get the join tables, that's something that a lot of people miss first off.
Some tips. I would add an autonumber field to all your tables and make that your primary key field - it makes things a lot easier. If you want to create your own number reference you can do that too but an autonumber field makes sure that you can't create duplicate records - I find it's more stable than a self generated number as a primary key.
Don't have spaces in your table, field, query, form names. If you have to put a space, use an Underscore, but Access does prefer it if you don't use spaces and it makes referencing fields a lot easier. It's also handy to prefix your different object types - tblReviewedBy; qryReviewedBy; frmReviewedBy. Again it makes referencing your objects easier.
A personal favourite of mine is not using the same field name in different tables. If you create a query that needs both the AuthorVesselView and AuthorName you have AuthorID in both tables. It can be difficult to decipher which AuthorID field from which table you have, especially if you need to reference that field in VBA. I like to prefix table fields with something from the table name to make the field name unique - AVVAuthorID ANAuthorID
Shame on your boss for putting you under this pressure!