Re: [hibernate-dev] Metamodel tests

2012-07-31 Thread Gail Badner
SQL is not exported for > PersistentIdentifierGenerator objects). > > - Original Message - > > > From: "Steve Ebersole" < st...@hibernate.org > > To: "Strong Liu" < st...@hibernate.org > > Cc: "Hibernate hibernate-dev" < hiberna

Re: [hibernate-dev] Metamodel tests

2012-07-30 Thread Strong Liu
le" >> To: "Strong Liu" >> Cc: "Hibernate hibernate-dev" >> Sent: Sunday, July 29, 2012 3:11:00 PM >> Subject: Re: [hibernate-dev] Metamodel tests >> >> For what its worth... I tried adding this for y'all before I left, >>

Re: [hibernate-dev] Metamodel tests

2012-07-30 Thread Gail Badner
This is due to HHH-6566 (Create and drop SQL is not exported for PersistentIdentifierGenerator objects). - Original Message - > From: "Steve Ebersole" > To: "Strong Liu" > Cc: "Hibernate hibernate-dev" > Sent: Sunday, July 29, 2012 3:11:00 P

Re: [hibernate-dev] Metamodel tests

2012-07-30 Thread Hardy Ferentschik
On Jul 29, 2012, at 10:47 PM, Steve Ebersole wrote: > Nonetheless, returning null instead of empty string is the convention we > decided on in Austin. It just does not make sense to keep doing that > isEmpty() style checks higher and higher. +1

Re: [hibernate-dev] Metamodel tests

2012-07-30 Thread Hardy Ferentschik
On Jul 29, 2012, at 10:38 PM, Steve Ebersole wrote: > Just a reminder that I am on vacation this next week. > > If you want this in the meantime, you just need to alter the > org.hibernate.metamodel.spi.relational.Database#generateSchemaCreationScript > and > org.hibernate.metamodel.spi.relat

Re: [hibernate-dev] Metamodel tests

2012-07-29 Thread Steve Ebersole
For what its worth... I tried adding this for y'all before I left, but it actually does not at all affect the tests. Not sure what tests you saw failing because the id sequence/table is not there, but I tried a few tests that use sequences and they all fail trying to build the MetaData object.

Re: [hibernate-dev] Metamodel tests

2012-07-29 Thread Steve Ebersole
My guess is that most likely nothing here really changed. We had slews of test failures that tbh hid lots of problems. Nonetheless, returning null instead of empty string is the convention we decided on in Austin. It just does not make sense to keep doing that isEmpty() style checks higher an

Re: [hibernate-dev] Metamodel tests

2012-07-29 Thread Steve Ebersole
Just a reminder that I am on vacation this next week. If you want this in the meantime, you just need to alter the org.hibernate.metamodel.spi.relational.Database#generateSchemaCreationScript and org.hibernate.metamodel.spi.relational.Database#generateDropSchemaScript methods. There is a or

Re: [hibernate-dev] Metamodel tests

2012-07-29 Thread Steve Ebersole
Yep I have not hooked my new schema management tool stuff into metamodel creation yet. On Jul 29, 2012 1:24 PM, "Strong Liu" wrote: > btw, some tests are failing due to the identifier generator required > sequence are not created. > > org.hibernate.id.PersistentIdentifierGenerator#sqlCreateString

Re: [hibernate-dev] Metamodel tests

2012-07-29 Thread Strong Liu
btw, some tests are failing due to the identifier generator required sequence are not created. org.hibernate.id.PersistentIdentifierGenerator#sqlCreateStrings is never get called (yes, i see there is a org.hibernate.id.SequenceGenerator#registerExportables) and others are failing with stacktr

Re: [hibernate-dev] Metamodel tests

2012-07-29 Thread Strong Liu
I was not aware of that, the root issue is "org.hibernate.metamodel.internal.source.annotations.attribute.Column#name" has a default empty string value. On Jul 30, 2012, at 1:30 AM, Steve Ebersole wrote: > Hadn't we all agreed though that the convention was for sources to return > null and n

Re: [hibernate-dev] Metamodel tests

2012-07-29 Thread Steve Ebersole
Hadn't we all agreed though that the convention was for sources to return null and not empty strings? On Jul 29, 2012 11:11 AM, "Strong Liu" wrote: > https://hibernate.onjira.com/browse/HHH-7480 > > don't know which change caused this, but it is fixed now > > On Jul 28, 2012, at 10:27 AM, Steve E

Re: [hibernate-dev] Metamodel tests

2012-07-29 Thread Strong Liu
https://hibernate.onjira.com/browse/HHH-7480 don't know which change caused this, but it is fixed now On Jul 28, 2012, at 10:27 AM, Steve Ebersole wrote: > Not sure if the expectation at this point is that all tests on the > metamodel branch pass when hibernate.test.new_metadata_mappings is se

[hibernate-dev] Metamodel tests

2012-07-27 Thread Steve Ebersole
Not sure if the expectation at this point is that all tests on the metamodel branch pass when hibernate.test.new_metadata_mappings is set to true or not. That was supposed to be the idea with the new @FailureExpectedWithNewMetamodel annotation which I see is in place now. But I am still seei

Re: [hibernate-dev] metamodel tests

2012-02-13 Thread Strong Liu
new hudson job internal : https://hudson.qa.jboss.com/hudson/job/hibernate-core-master-metamodel/ external: seems ci.jboss.org currently down, please wait for while and go there and search "hibernate-core-master-metamodel" looking the failures soon - Best Regards, Str

[hibernate-dev] metamodel tests

2012-02-12 Thread Steve Ebersole
There are a bunch of test failures in the new metamodel tests. I know this stuff is changing all the time, but we really need to work on making sure that work we are doing is not causing regressions. Otherwise, this work will quickly become a situation of "1 step forward, 2 steps back" instead