jpa - Issue with bidirectional one-to-one relationship trying to INSERT instead of UPDATE in Java EclipseLink -


i'm having trouble understanding why bidirectional relationship not working here quite expect to. when persist owning side after setting relationships both ways, cascading takes place , 'insert inverseside' sql statement executed. causes exception thrown trying add duplicate pk inverse side entity's table.

i'd assume there should 'update' statement instead, inverse side entity in db , - far rookie guts can tell - managed jpa.

database mockup:

table inverseside   id  table owningside   id   inverseside_id

java mockup:

dao interfaces

@repository public interface isdao extends jparepository<inverseside, long>  @repository public interface osdao extends jparepository<owningside, long>

entity classes

public class inverseside {   private long id;    @onetoone(optional = true, mappedby = "inverse")   private owningside owner;    public void setowner(owningside os) {     owner = os;   } }  public class owningside {   private long id;    @onetoone(optional = false, cascade = cascadetype.all)   private inverseside inverse;    public void setinverse(inverseside is) {     inverse = is;     inverse.setowner(this);   } }

test code

inverseside inverse = isdao.findall().get(0); // returns valid object id 1000 owningside owner = new owningside();  owner.setinverse(inverse); osdao.save(owner); // exception  /*   org.springframework.transaction.transactionsystemexception:    not commit jpa transaction; nested exception    javax.persistence.rollbackexception: exception [eclipselink-4002] (eclipse    persistence services - 2.7.0.v20170811-d680af5):    org.eclipse.persistence.exceptions.databaseexception   internal exception: org.h2.jdbc.jdbcsqlexception: unique index or primary    key violation: "primary key on public.inverseside(id)"; sql statement:   insert inverseside (id, ...   error code: 23505   call: insert inverseside (id, ...           bind => [1000, ... */

update

inverse side insert due owning side being new - , consequently unmanaged - jpa. in other words, though known jpa's point of view, persisting unmanaged os led connected being treated unmanaged.

thus, answer persist new inverseless owner first, complete relationship both ways , persist owner again:

inverseside inverse = isdao.findall().get(0); owningside owner = new owningside();  osdao.save(owner);  owner.setinverse(inverse); osdao.save(owner);

perhaps should have anticipated behavior. however, doesn't feel intuitive, , none of sources studied mentioned this.

most instances of type of problem comes due fact object inverse detached. barring specific transactional boundaries, each call repository methods own transaction, , return value detached. , thinks it's new object, , insert instead of update.

however, if have transactional boundaries set properly, don't think jpa implementation want do. it's not simple matter of updating is. you're expectations, in pseudo code:

select os os.is.id = :isid  update os set = null os.is.id = :isid  insert os (:newos)

the first 2 statements isn't in realm of responsibility entity manager. implementation seems more appropriate manytoone relationship. otherwise, should load other os, remove it's attachment is, save that, create new os, attach is, , save that. in same transaction.


Comments

Post a Comment

Popular posts from this blog

android - InAppBilling registering BroadcastReceiver in AndroidManifest -

i'm implementing inapp billing in android app. works well, however, i'm trying decouple broadcast receiver activity manifest. suggestion in android's trivialdrive sample: // important: dynamically register broadcast messages updated purchases. // register receiver here instead of <receiver> in manifest // because call getpurchases() @ startup, therefore can ignore // broadcasts sent while app isn't running. // note: registering listener in activity bad idea, done here // because sample. regardless, receiver must registered after // iabhelper setup, before first call getpurchases(). currently there's class extends broadcastreceiver : public class iabbroadcastreceiver extends broadcastreceiver { /** * intent action receiver should filter for. */ public static final string action = "com.android.vending.billing.purchases_updated"; private final iabbroadcastlistener mlistener; public iabbroadcastreceiver(iabbroadcastlistener listener) { m...
Read more

python Tkinter Capturing keyboard events save as one single string -

is possible store keyboard events 1 string? code below stores 1 char , prints it. card reader or bar code reader, contains collection of character/ string not 1 character @ time. goal save char pressed text variable. from tkinter import * root = tk() def key(event): text= event.char text+= event.char print ("pressed", text) def callback(event): frame.focus_set() print ("clicked at", event.x, event.y) frame = frame(root, width=100, height=100) frame.bind("<key>", key) frame.bind("<button-1>", callback) frame.pack() root.mainloop() currently, creating text variable , everytime key() function gets called, , text stores last character typed. you can define text module level variable , , use module level text inside key function - text = '' def key(event): global text text+= event.char print("pressed", text)
Read more

sql server - Why does Linq-to-SQL add unnecessary COUNT()? -

here tables: create table customer ( cid int primary key, name varchar(32) ) create table ord ( oid int primary key, cid int foreign key references customer, address varchar(20) ) here's linq-to-sql statement: var aaa = c in db.customer select new { c, o = c.ord.tolist() }; and here's inexplicable query generated linq-to-sql: select [t0].[cid] [cid], [t0].[name] [name], [t1].[oid] [oid], [t1].[cid] [cid2], [t1].[address] [address], (select count(*) [dbo].[ord] [t2] [t2].[cid] = [t0].[cid]) [value] [dbo].[customer] [t0] left outer join [dbo].[ord] [t1] on [t1].[cid] = [t0].[cid] order [t0].[cid], [t1].[oid] i want understand how rid of count(*) part. it's entirely uncalled for! linq needs count(*) in order determine number of entries returned o = c.ord.tolist() .
Read more