Не думаю, что это необходимо, поскольку отображение @ OneToOne должно косвенно создавать внешний ключ для столбца.
С другой стороны, если вы считаете, что это поможет вам и другим разработчикам легче понять код и не вызывает никаких проблем, вы можете оставить его там.
Однако, похоже, что аннотация ForeingKey специфична для режима гибернации, когда аннотация OneToOne является частью Java Persistence API. Это могло бы поддержать идею убрать это.