Hi Norman and Eric, I fully agree with simply using OpenJPA annotations.
Concerning the openjpa package I think I found what you mean, Eric. It was confusing because there are two OpenJPA packages: org/apache/james/imap/jpa/mail/model/openjpa org/apache/james/imap/jpa/openjpa The latter is there merely to support the "useStreaming" option, if I don't err. The former is also for streaming, so yes, it would make sense to rename it. On the other hand we could move all OpenJPA stuff to /jpa/openjpa which would basically mean to e.g. put the streaming classes into org/apache/james/imap/jpa/openjpa/mail/model/streaming If everything with proprietary OpenJPA annotations would be in a separate package it would become immediate which classes one needs to implement in order to create a new provider. That's my vote: stick with OpenJPA but still cleanly separate it from standards conforming code. Eric, could you send us a patch of what you've done so far? Then we can finish it (hope you still read this before your trip...) Tim Am Freitag, den 25.06.2010, 19:33 +0200 schrieb Norman Maurer: > Ok so to come to some consequence here.. Let us just use the openjpa > annotation stuff.. If we really want to support other JPA > implementations we could handle it later.. > > Bye, > Norman > > > 2010/6/25 Tim-Christian Mundt <d...@tim-erwin.de>: > > Hey, > > > > > >> I'm also happy with OpenJPA and using its proprietary annotations (not > >> classes) doesn't prohibit a developer/deployer to define another JPA > >> provider. > > Right. > > > >> What about : > >> - @ElementJoinColumn ? > >> - @Index ? > > I'd support those. > > > >> - rename 'openjpa' package to 'streaming' ? > > We already have a streaming package and there is both streaming and > > non-streaming for OpenJPA, so why rename the package? Maybe I haven't > > fully understood your point. > > > >> I will be off for 2 weeks and won't probably be able to continue the > >> conversation. > > Hope I may say "Happy vacations" :) > > > > Best > > Tim > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org > > For additional commands, e-mail: server-dev-h...@james.apache.org > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org > For additional commands, e-mail: server-dev-h...@james.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
