Bug 1010237

Summary: session-migration: attempt to get a running process instance fails on strategy not found
Product: [Retired] JBoss BPMS Platform 6 Reporter: Marek Baluch <mbaluch>
Component: jBPM CoreAssignee: Maciej Swiderski <mswiders>
Status: CLOSED CURRENTRELEASE QA Contact: Marek Baluch <mbaluch>
Severity: urgent Docs Contact:
Priority: unspecified    
Version: 6.0.0Keywords: TestBlocker
Target Milestone: ER4   
Target Release: 6.0.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-08-06 20:08:28 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
process definition
none
log none

Description Marek Baluch 2013-09-20 10:46:21 UTC 
Description of problem:

The following code fails with 'No strategy of type org.drools.marshalling.impl.SerializablePlaceholderResolverStrategy available.'

KieSession ksession = restoreKnowledgeSession(sinfo.getSessionId(), "BPMN2-RepeatingTimer.bpmn2");
		ProcessInstance pi = ksession.getProcessInstance(sinfo.getProcessInstanceIds().get(0));
		Assert.assertNotNull(pi);
		Assert.assertTrue(pi.getState() == ProcessInstance.STATE_ACTIVE);

The change to org.drools.core.marshalling.impl.SerializablePlaceholderResolverStrategy might be required on more places. See attached log.
Comment 1 Marek Baluch 2013-09-20 10:47:47 UTC 
Created attachment 800427 [details]
process definition
Comment 2 Maciej Swiderski 2013-09-24 16:09:50 UTC 
Marek, could you please attach the log that you mentioned in the description? I can only see the process definition. Would be nice to see the actual code you use for that. Is there any repo where I can take a look at how you test the migration between 5 and 6?
Comment 3 Marek Baluch 2013-09-24 17:17:45 UTC 
Hi Maciej,

argh I forgot to attach the log :(. Will correct that ASAP. In mean time here's the repo:

http://git.app.eng.bos.redhat.com/?p=jbossqe/brms.git;a=tree;f=test-jbpm-session-migration;h=c135fc10bce3a6bc7e08b00538407da39f7debcc;hb=refs/heads/master

Store part is made in from_5.3.x/.../StoreSessionTest.timerTest() and the load part is made in to_6.x/.../LoadSessionTest.timerTest()
Comment 4 Marek Baluch 2013-09-24 17:25:04 UTC 
Created attachment 802373 [details]
log
Comment 5 Maciej Swiderski 2013-09-25 17:31:39 UTC 
ensured backward compatibility for strategies

master
https://github.com/droolsjbpm/drools/commit/8e70c9776caec914df94ba3989c4802bbc665c56
https://github.com/droolsjbpm/drools/commit/72c5a8b12f39a7776c0593500f8da7066cee2906

6.0.x
https://github.com/droolsjbpm/drools/commit/b283d728afb51f6b3abdd8cc7c34fd5db01c282d
https://github.com/droolsjbpm/drools/commit/b584b63db13657aaebe447db08f8779629ec965b
Comment 6 Marek Baluch 2013-10-18 10:32:28 UTC 
Verified on ER4.