From: | Lew <noone(at)lewscanon(dot)com> |
---|---|
To: | pgsql-jdbc(at)postgresql(dot)org |
Subject: | Re: Problems with Hibernate Discriminators and 9.0-801.jdbc4 |
Date: | 2011-03-28 13:27:22 |
Message-ID: | imq2bn$akn$1@news.albasani.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-jdbc |
Chris Wareham wrote:
> Dave Cramer wrote:
>> Ok, looking at the hibernate source this is what is in the PostgreSQLDialect
>>
>> public String getIdentitySelectString(String table, String column, int type) {
>> return new StringBuffer().append("select currval('")
>> .append(table)
>> .append('_')
>> .append(column)
>> .append("_seq')")
>> .toString();
>> }
>> }
> Just as an aside, should that not be using a StringBuilder rather than
> a StringBuffer? Or does Hibernate still explicitly support Java 1.4?
My Red-Flag-O-Meter triggered on that, too. What Hibernate ought to do is:
return "select currval('" + table + '_' + column + "_seq')";
for Pete's sake.
I'm off to check if EclipseLink or OpenJPA is better written.
--
Lew
Honi soit qui mal y pense.
http://upload.wikimedia.org/wikipedia/commons/c/cf/Friz.jpg
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Cramer | 2011-03-28 13:40:53 | Re: Problems with Hibernate Discriminators and 9.0-801.jdbc4 |
Previous Message | sdavidr | 2011-03-28 13:05:40 | Re: Problems with Hibernate Discriminators and 9.0-801.jdbc4 |