From: SourceForge.net <no...@so...> - 2003-05-22 15:35:00
|
Patches item #741810, was opened at 2003-05-22 10:35 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=376687&aid=741810&group_id=22866 Category: JBossCMP Group: v3.2 Status: Open Resolution: None Priority: 5 Submitted By: Peter Spiess (pspiess) Assigned to: Nobody/Anonymous (nobody) Summary: Do not fix table names if not being created Initial Comment: We have tables across many schemas in an single Oracle instance. There are constraints that cross schema boundaries. I would really like to configure JBoss to just reference the table name within the schema that it exists so I have been putting the schema name in the table name element of jbosscmp-jdbc.xml (using JBoss 3.2.1). However it looks like SQLUtil.fixTableName shortens the table name to a max length based on the database. Wouldn't this name be exclusive of the schema name? For example, a table name I would like to use is ICENTRAL_EXCHANGE.COMPANY_HEADER which is 32 characters in total (getMaxTableNameLength() returns 30 as the max length allowed). However, the table name is only 14 characters therefore it does not violate the maxlength set by the database. I can't speak for other database vendors, but if an Oracle table name has a period in the name, couldn't SQLUtil ignore the stuff on the left side of the period when fixing the table name length? The attached patch would only mangle the table name if JBoss is going to create the table. As I understand it, this fix was put in for when JBoss creates relationship tables to keep them to a maximum length. This is a simple and imperfect change. At worst I think that if users specify a table name that is invalid it will still throw a table not found exception which is what is currently happening in my case. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=376687&aid=741810&group_id=22866 |