|
From: Marc L. <ma...@lo...> - 2005-06-19 14:26:33
|
Juergen Hoeller wrote: > To make the actual behavior pluggable, we could factor the actual > invocations out to corresponding new JdoDialect methods, by default calling > the standard JDO 2.0 methods accordingly. This could then be overridden for > special vendor-specific attach/detach methods, whether current JDO 2.0 > previews or overloaded vendor-specific versions alongside the standard JDO > 2.0 methods. +1, thats the approach i had in mind in the last days. > Kodo currently has no-arg "attach" and "detach" methods. Versant Open Access > currently has "versantAttachCopy" with one or two boolean args, and > "versantDetachCopy" with a String arg (the fetch group; can be null). Both > could quite easily be used within a Kodo/Versant-specific JdoDialect, > passing in default arguments where necessary. And i know at least one more vendor which could be easily adjusted to that as well. > I'm certainly open to go down that route; I'm just a little undecided > whether it makes sense to do it at this point of time. I'm currently in the > process of gathering some further opinions. If it turns out that the general > agreement is in favor of such hooks, I'll add them for Spring 1.2.2 > (scheduled for coming Thursday/Friday). Yeah and here we will certainly have different oppinions. Most people will think "hey JDO2 is nearly out the door - no need for dialect stuff anymore". I can agree on this only to that point that this argument is true and i dont believe this. I asked Patrick Linskey (or Abe White, dont know anymore) from Solarmetric and he said that JDO2 is not THAT near. Anyway, i am interessted to hear other oppinions on that issue. Perhaps i will ask the Solarmetric guys about that to get some more arguments. Really funny was a statement from Patrick to me where he believed that there is allready an attach/detach abstraction inside JdoDialect, so it seems he likes the idea so much that he thought its allready in there ;-) -- regards Marc Logemann [blog] http://www.logemann.org [busn] http://www.logentis.de |