Versioning multiple instances not implemented

40 views
Skip to first unread message

Werner Mahalek

unread,
Dec 30, 2012, 5:43:31 AM12/30/12
to twig-p...@googlegroups.com
Hi John,

I tried to do a "datastore.storeAll(toSave)" on a list of versioned entities and got the UnsupportedOperationException "Versioning multiple instances not implemented". Would it be very hard to implement this functionality in twig-persist?

Kind Regards,
Werner

John Patterson

unread,
Jan 9, 2013, 8:14:55 PM1/9/13
to twig-p...@googlegroups.com
Sorry, I meant to look into this further before answering.  Have had a quick look and can see that I was just being lazy not implementing it to handle multiple instances.  Should have a fix ready today.

John Patterson

unread,
Jan 9, 2013, 10:16:04 PM1/9/13
to twig-p...@googlegroups.com
I've checked in the update now with a new unit test.

Now bulk update of versioned instances should work if you build from the latest source.


This push also exposes the new ObjectDatastore.beginOrJoinTransaction()

Werner Mahalek

unread,
Jan 20, 2013, 5:19:39 AM1/20/13
to twig-p...@googlegroups.com
Hello John,

at last I had time to get the newest version of twig-persist from the repository. But I ran into a problem while compiling the code:
 "CompositeTypeConverter cannot be resolved to a type" e.g. in ConverterTranslator.java
It seems that the com.vercer.convert.CompositeTypeConverter code is missing from the repository.

Kind Regards,
Werner

Christian Sindberg

unread,
Jan 20, 2013, 5:05:36 PM1/20/13
to twig-p...@googlegroups.com
Can confirm this one.

Bug is introduced in revision 181 as 180 is building fine.

John Patterson

unread,
Jan 20, 2013, 5:40:30 PM1/20/13
to twig-p...@googlegroups.com
Doh I didn't update the shared converter code.

Done now and checked in.

Werner Mahalek

unread,
Jan 22, 2013, 3:06:40 AM1/22/13
to twig-p...@googlegroups.com
It's working now. Thank you very much!
Reply all
Reply to author
Forward
0 new messages