Bug 841896

Summary: JDBC cache store should quote generated table name
Product: [JBoss] JBoss Data Grid 6 Reporter: Tristan Tarrant <ttarrant>
Component: InfinispanAssignee: Tristan Tarrant <ttarrant>
Status: VERIFIED --- QA Contact: Martin Gencur <mgencur>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.0.0CC: jdg-bugs
Target Milestone: ER1   
Target Release: 6.0.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
In JBoss Data Grid 6.0.0 the JDBC cache store would not enclose generated table names within quotes, which caused problems when attempting to use special characters. This has now been resolved, and table names are quoted.
Story Points: ---
Clone Of: Environment:
Last Closed: 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
5.1.5 stacktrace
none
5.1.6 Stacktrace none

Description Tristan Tarrant 2012-07-20 13:51:37 UTC

Comment 1 JBoss JIRA Server 2012-07-24 13:22:56 UTC
Galder ZamarreƱo <galder.zamarreno> made a comment on jira ISPN-2155

@Paul, it will be fixed for 5.1.x, we're marking them as 'jdg' and 'jdg6' tags, so Tristan will come and triage those issues marked with the tag and cherry pick them.

Comment 2 Martin Gencur 2012-07-26 12:58:25 UTC
This does not work fully correctly yet. I changed one of our tests to have TABLE_NAME_PREFIX = "edg/bin". With 5.1.5.FINAL-redhat-1  the test failed on creating the table. With 5.1.6.FINAL-redhat.-1 it fails on storing the data into the table. Attaching stacktraces for both 5.1.5 and 5.1.6.

Comment 3 Martin Gencur 2012-07-26 12:58:57 UTC
Created attachment 600503 [details]
5.1.5 stacktrace

Comment 4 Martin Gencur 2012-07-26 12:59:20 UTC
Created attachment 600504 [details]
5.1.6 Stacktrace

Comment 5 Martin Gencur 2012-07-26 13:08:30 UTC
I'm sorry, my bad. This was a problem in our tests. Verified.