[jboss-jira] [JBoss JIRA] Commented: (JBRULES-2291) char primary keys used in drools-bam persistence entities.

[Juraj Tomasov (JIRA)]

    [ https://jira.jboss.org/jira/browse/JBRULES-2291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12489125#action_12489125 ] 

Juraj Tomasov commented on JBRULES-2291:
----------------------------------------

Index: src/main/java/org/drools/process/audit/ProcessInstanceLog.java
===================================================================
--- src/main/java/org/drools/process/audit/ProcessInstanceLog.java	(revision 29567)
+++ src/main/java/org/drools/process/audit/ProcessInstanceLog.java	(working copy)
@@ -4,7 +4,7 @@
 
 public class ProcessInstanceLog {
     
-	private String id;
+	private long id;
     private long processInstanceId;
     private String processId;
     private Date start;
@@ -19,11 +19,11 @@
         setStart(new Date());
     }
     
-    public String getId() {
+    public long getId() {
     	return id;
     }
     
-    void setId(String id) {
+    void setId(long id) {
 		this.id = id;
 	}
 


> char primary keys used in drools-bam persistence entities.
> ----------------------------------------------------------
>
>                 Key: JBRULES-2291
>                 URL: https://jira.jboss.org/jira/browse/JBRULES-2291
>             Project: Drools
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: drools-process
>    Affects Versions: 5.1.0.M1
>            Reporter: Juraj Tomasov
>            Assignee: Mark Proctor
>             Fix For: 5.1.0.M2
>
>
> Beans NodeInstanceLog and ProcessInstanceLog in package org.drools.process.audit should not have String primary keys. Appropriate hbm.xml mapping files and these beans should be changed. Integration tests for example with MySQL db would cover such bugs.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira