Thank you. Great job.
And now I can have some fun with git.
Davide
On Thu, Jun 2, 2011 at 7:26 PM, Sanne Grinovero wrote:
> Hi Davide,
> so, this took me a while as there where a number of problems with the
> annotation processor; thanks to John and Steve for pointing me in the
> right direction
Seems to make sense from my perspective, but I'm the least familiar with this
code.
On Jun 2, 2011, at 3:24 PM, Steve Ebersole wrote:
> I guess one thing I am interested in is whether you see an issue in this
> style of process with annotations. On the HBM side of things, this will
> work out
I guess one thing I am interested in is whether you see an issue in this
style of process with annotations. On the HBM side of things, this will
work out beautifully. Do you foresee any issues with this on the
annotation side?
On 06/02/2011 02:51 PM, Steve Ebersole wrote:
> Hardy, et al.
>
>
Hardy, et al.
I have been working mostly on low-hanging fruit in this metamodel code
getting a handle on the code y'all wrote.
I wanted to discuss one proposal for change in particular. Currently
the ctor for MetadataImpl decides whether to process HBM or annotations
first and then processes
Hi Davide,
so, this took me a while as there where a number of problems with the
annotation processor; thanks to John and Steve for pointing me in the
right direction.
The main problem was: http://bugs.sun.com/view_bug.do?bug_id=6512707
basically we can't have the annotation processor run during
Not sure there is much we can do right now. Yes that form of doing
SchemaExport, et al will go away. So by the time we would get this
fixed it would be for Beta1 anyway.
You are obviously using SchemaExport directly (as Hibernate itself uses
the new ServiceRegistry ctor form). So I would sug
Hi, Steve,
I just looked in ManagedProviderConnectionHelper which is used in
SchemaExport, SchemaUpdate, SchemaValidator and it is broken.
It is never creates a connection, and this make impossible to run
SchemaExport, SchemaUpdate, SchemaValidator configured by cfg.
I guess the class is going