SpringBootでOneToOneテーブル結合し自動生成したIDを結合先に登録したいがnullエラーとなる。

Question

前提・実現したいこと

SpringBootの勉強中です。
1つのフォームで、2つのテーブルの内容を登録したくて、OneToOne結合しています。
主のテーブルの自動採番（mst_sensorのsensor_id、mst_mailのmail_id）はinsert文に表示されていますが、その採番されたID（sensor_id）を結合先のテーブル（mst_mail）に登録したいのですが、nullとなり登録できずに困っています。
よろしくお願いします。

発生している問題・エラーメッセージ

10. insert into mst_sensor (sensor_name, updatedata, sensor_id) values ('AA11', '05/25/2018 09:54:22.253', 36)  {executed in 66 msec}

10. insert into mst_mail (sensor_id, sendmail_address, sendmail_threshold, updatedata, mail_id) values (NULL, NULL, 1, '05/25/2018 09:54:22.047', 11) {FAILED after 47 msec}

org.postgresql.util.PSQLException: ERROR: null value in column "sensor_id" violates not-null constraint
  詳細: Failing row contains (null, 0, 1, null, 2018-05-25 09:54:22.047, 11).

該当のソースコード

postgresql

\d mst_sensor
Table "public.mst_sensor"
Column | Type | Collation | Nullable | Default 
-------+------+-----------+----------+--------------------
sensor_id | integer | | not null | nextval('mst_sensor_sensor_id_seq'::regclass)
sensor_name | character varying(30) | | not null | 
updatedata | timestamp without time zone | | not null | 
Indexes:
"mst_sensor_pkey" PRIMARY KEY, btree (sensor_id)


\d mst_sensor_sensor_id_seq
Sequence "public.mst_sensor_sensor_id_seq"
Type | Start | Minimum | Maximum | Increment | Cycles? | Cache 
-----+-------+---------+---------+-----------+---------+-------
integer | 1 | 1 | 2147483647 | 1 | no | 1
Owned by: public.mst_sensor.sensor_id


\d mst_mail
Table "public.mst_mail"
Column | Type | Collation | Nullable | Default 
-------+------+-----------+----------+-----------------------
sensor_id | integer | | not null | 
sendmail_threshold | integer | | | 
sendmail_address | character varying(128) | | | 
updatedata | timestamp without time zone | | not null | 
mail_id | integer | | not null | nextval('mst_mail_mail_id_seq'::regclass)
Indexes:
"mst_mail_pkey" PRIMARY KEY, btree (mail_id)


\d mst_mail_mail_id_seq
Sequence "public.mst_mail_mail_id_seq"
Type | Start | Minimum | Maximum | Increment | Cycles? | Cache 
-----+-------+---------+---------+-----------+---------+-------
bigint | 1 | 1 | 9223372036854775807 | 1 | no | 1

Machine.java

@Data
@NoArgsConstructor
@AllArgsConstructor
@Entity
@Table(name = "mst_sensor")
public class Machine {
	
	private Integer sensorId;
	private String sensorName;
	private Timestamp updatedata;
	private MachineMailAdd machineMailAdd;
	
	@Id
	@Column(name="sensor_id")
	@GeneratedValue(strategy=GenerationType.SEQUENCE, generator="sensorIdSeq")
	@SequenceGenerator(name="sensorIdSeq", sequenceName="mst_sensor_sensor_id_seq", allocationSize=1)
	public Integer getSensorId() {
		return sensorId;
	}
	
	public void setSensorId(Integer sensorId) {
		this.sensorId = sensorId;
	}
	
	@Column(name="sensor_name")
	@NotNull
	@Size(min = 1, max = 15)
	public String getSensorName() {
		return sensorName;
	}
	
	public void setSensorName(String sensorName) {
		this.sensorName = sensorName;
	}

	@Column(name="updatedata")
	public Timestamp getUpdatedata() {
		return updatedata;
	}
	
	public void setUpdatedata(Timestamp updatedata) {
		this.updatedata = updatedata;
	}
	
	@OneToOne(mappedBy = "machine", cascade = CascadeType.ALL)
	public MachineMailAdd getMachineMailAdd() {
		return machineMailAdd;
	}
	
	public void setMachineMailAdd(MachineMailAdd machineMailAdd) {
		this.machineMailAdd = machineMailAdd;
	}
	
}

MachineMailAdd.java

@Data
@NoArgsConstructor
@AllArgsConstructor
@Entity
@Table(name = "mst_mail")
public class MachineMailAdd {
	
	private Integer mailId;
	private Integer sendmailThreshold;
	private String sendmailAddress;
	private Timestamp updatedata = new Timestamp(System.currentTimeMillis());
	private Machine machine;
	
	@Id
	@Column(name="mail_id")
	@GeneratedValue(strategy=GenerationType.SEQUENCE, generator="mailIdSeq")
	@SequenceGenerator(name="mailIdSeq", sequenceName="mst_mail_mail_id_seq", allocationSize=1)
	public Integer getMailId() {
		return mailId;
	}
	
	public void setMailId(Integer mailId) {
		this.mailId = mailId;
	}
	
	@Column(name="sendmail_threshold")
	@NotNull
	public Integer getSendmailThreshold() {
		return sendmailThreshold;
	}
	
	public void setSendmailThreshold(Integer sendmailThreshold) {
		this.sendmailThreshold = sendmailThreshold;
	}

	@Column(name="sendmail_address")
	@Size(min = 1, max = 128)
	public String getSendmailAddress() {
		return sendmailAddress;
	}
	
	public void setSendmailAddress(String sendmailAddress) {
		this.sendmailAddress = sendmailAddress;
	}

	@OneToOne(cascade = CascadeType.ALL)
	@JoinColumn(name = "sensor_id")
	public Machine getMachine() {
		return machine;
	}
	
	public void setMachine(Machine machine) {
		this.machine = machine;
	}
	
}

MachineController.java

	@RequestMapping("add")
	String create(Model model) {
		return "admin/machine/add";
	}
	
	@PostMapping(path = "add")
	String createAdd(@Validated Machine form, BindingResult result, Model model) {
		if (result.hasErrors()) {
			return create(model);
		}
		Machine machine = new Machine();
		BeanUtils.copyProperties(form, machine);
		machine.setUpdatedata(new Timestamp(System.currentTimeMillis()));
		machineService.create(machine);
		
		return "redirect:/admin/machine/";
	}

補足情報（FW/ツールのバージョンなど）

Java
SpringBoot
PostgreSQL

Comments

[退会済みユーザー]

シーケンスがないのでは？

[hide--]

回答の返信として詳細書いておりますので、ご確認ください。

[退会済みユーザー]

そもそも 上記サンプルソースでは new MachineMailAdd + SET がないので MachineMailAdd に対して INSERT は動かないはずですよ

[hide--]

ログを追記しましたが、両テーブルinsertが発行され、machineMailAddのinsertでエラーが出ている状況です。エンティティで結合しているのでこれで出来るかと思っていましたが、間違えているでしょうか？

Answer 1

自己解決しました。
連絡するカラム（sensor_id）を採番するテーブル（mst_sensor）ではなく、
参照して登録するテーブル（mst_mail）をコントローラーで保存する形に変更することで、
実現することができました。

ありがとうございました。

MachineController.java

@PostMapping(path = "add")
String createAdd(@Validated MachineMail form, BindingResult result, Model model) {
	if (result.hasErrors()) {
		return create(model);
	}
	MachineMail machineMail = new MachineMail();
	BeanUtils.copyProperties(form, machineMail);
	machineMail.setUpdatedata(new Timestamp(System.currentTimeMillis()));
	machineMailService.create(machineMail);
	
	return "redirect:/admin/machine/";
}

Answer 2

質問のマナーとして、CREATE TABLEとCREATE SEQUENCE を質問に載せてください。

PostgreSQL で連番の数字のフィールドを作る方法 (sequence について)

create sequence LiquorID_seq;

と

insert into LiquorList (LiquorID, LiquorName, LiquorDate)
values (nextval('LiquorID_seq'), 'ビール', '2007-12-01');

を参考にされては？

追記
どこの掲示板でも同じことですが、自分で質問する前に他の人の質問を何件か読んで質問の書き方を学びます。
最初の質問にCREATE TABLE を載せましょう。テーブルの定義がないとINSERTは実行できません。
PostgreSQL CREATE TABLE文取得方法

ALTER SEQUENCE でSEQUENCEの初期値を変更できます。

ここまで書けば後は自分で直せますか？

Comments

[hide--]

はじめて質問するもので、見やすくするにはどこにどう入れたらいいか分からずすみません。
それぞれ、mst_sensorとmst_mailのシーケンスを設定していてSQLのログ上は出ています。

10. insert into mst_sensor (sensor_name, updatedata, sensor_id) values ('AA11', '05/25/2018 09:54:22.253', 36)
10. insert into mst_mail (sensor_id, sendmail_address, sendmail_threshold, updatedata, mail_id) values (NULL, NULL, 1, '05/25/2018 09:54:22.047', 11)

この二つ目のmst_mailのinsert文にあるsensor_idに、一つ目のsensor_id「36」が入ってほしいのです。

\d mst_sensor
Table "public.mst_sensor"
Column | Type | Collation | Nullable | Default
---------------------+-----------------------------+-----------+----------+-----------------------------------------------
sensor_id | integer | | not null | nextval('mst_sensor_sensor_id_seq'::regclass)
sensor_name | character varying(30) | | not null |
updatedata | timestamp without time zone | | not null |
Indexes:
"mst_sensor_pkey" PRIMARY KEY, btree (sensor_id)

\d mst_sensor_sensor_id_seq
Sequence "public.mst_sensor_sensor_id_seq"
Type | Start | Minimum | Maximum | Increment | Cycles? | Cache
---------+-------+---------+------------+-----------+---------+-------
integer | 1 | 1 | 2147483647 | 1 | no | 1
Owned by: public.mst_sensor.sensor_id

\d mst_mail
Table "public.mst_mail"
Column | Type | Collation | Nullable | Default
--------------------+-----------------------------+-----------+----------+-------------------------------------------
sensor_id | integer | | not null |
sendmail_threshold | integer | | |
sendmail_address | character varying(128) | | |
updatedata | timestamp without time zone | | not null |
mail_id | integer | | not null | nextval('mst_mail_mail_id_seq'::regclass)
Indexes:
"mst_mail_pkey" PRIMARY KEY, btree (mail_id)

\d mst_mail_mail_id_seq
Sequence "public.mst_mail_mail_id_seq"
Type | Start | Minimum | Maximum | Increment | Cycles? | Cache
--------+-------+---------+---------------------+-----------+---------+-------
bigint | 1 | 1 | 9223372036854775807 | 1 | no | 1

[退会済みユーザー]

この手のは出力は質問を編集して追記してください。

[hide--]

ありがとうございます。追記してみました。