You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ode.apache.org by Matthieu Riou <ma...@offthelip.org> on 2008/09/15 16:23:15 UTC

Re: svn commit: r695391 - /ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java

On Mon, Sep 15, 2008 at 2:31 AM, <va...@apache.org> wrote:

> Author: vanto
> Date: Mon Sep 15 02:31:06 2008
> New Revision: 695391
>
> URL: http://svn.apache.org/viewvc?rev=695391&view=rev
> Log:
> JDK5, not 6..., who volunteers to port our JDBC debug facade to JDK5+6?
>
> Modified:
>
>  ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java
>

Thanks for the fix! Actually I tried to port the JDBC facade but that would
need a few classes (used by the additional methods) that are JDK 6 only. We
could create our own "backport" but I'm not sure it's worth it. Plus there
are some legal issues in implementing classes in the java.* package.

Matthieu


>
> Modified:
> ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java
> URL:
> http://svn.apache.org/viewvc/ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java?rev=695391&r1=695390&r2=695391&view=diff
>
> ==============================================================================
> ---
> ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java
> (original)
> +++
> ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java
> Mon Sep 15 02:31:06 2008
> @@ -129,7 +129,8 @@
>         try {
>             pmodel = (ProcessModel) ois.readObject();
>         } catch (ClassNotFoundException e) {
> -            throw new IOException("DataStream Error", e);
> +            //throw new IOException("DataStream Error", e); // we keep
> this until we've moved to JDK6, okay?
> +            throw new IOException("DataStream Error");
>         }
>
>         return pmodel;
>
>
>

Re: svn commit: r695391 - /ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java

Posted by Matthieu Riou <ma...@offthelip.org>.
On Mon, Sep 15, 2008 at 7:29 AM, Tammo van Lessen <tv...@gmail.com>wrote:

> Matthieu Riou wrote:
> > On Mon, Sep 15, 2008 at 2:31 AM, <va...@apache.org> wrote:
> >
> >> Author: vanto
> >> Date: Mon Sep 15 02:31:06 2008
> >> New Revision: 695391
> >>
> >> URL: http://svn.apache.org/viewvc?rev=695391&view=rev
> >> Log:
> >> JDK5, not 6..., who volunteers to port our JDBC debug facade to JDK5+6?
> >>
> >> Modified:
> >>
> >>
>  ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java
> >>
> >
> > Thanks for the fix! Actually I tried to port the JDBC facade but that
> would
> > need a few classes (used by the additional methods) that are JDK 6 only.
> We
> > could create our own "backport" but I'm not sure it's worth it. Plus
> there
> > are some legal issues in implementing classes in the java.* package.
>
> What about translating it into a dynamic proxy.


Good idea.


> Its then independent of
> the underlying JDBC version. IIRC I have started that a while on a
> another machine, perhaps I can find it again...
>

That'd be sweet :)


>
> Cheers,
>   Tammo
>

Re: svn commit: r695391 - /ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java

Posted by Alex Boisvert <bo...@intalio.com>.
On Mon, Sep 15, 2008 at 7:29 AM, Tammo van Lessen <tv...@gmail.com>wrote:

> >> URL: http://svn.apache.org/viewvc?rev=695391&view=rev
> >> Log:
> >> JDK5, not 6..., who volunteers to port our JDBC debug facade to JDK5+6?
> >>
> >> Modified:
> >>
> >>
>  ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java
> >>
> >
> > Thanks for the fix! Actually I tried to port the JDBC facade but that
> would
> > need a few classes (used by the additional methods) that are JDK 6 only.
> We
> > could create our own "backport" but I'm not sure it's worth it. Plus
> there
> > are some legal issues in implementing classes in the java.* package.
>
> What about translating it into a dynamic proxy. Its then independent of
> the underlying JDBC version. IIRC I have started that a while on a
> another machine, perhaps I can find it again...


Yup, that would be the way to do it.

alex

Re: svn commit: r695391 - /ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java

Posted by Tammo van Lessen <tv...@gmail.com>.
Matthieu Riou wrote:
> On Mon, Sep 15, 2008 at 2:31 AM, <va...@apache.org> wrote:
> 
>> Author: vanto
>> Date: Mon Sep 15 02:31:06 2008
>> New Revision: 695391
>>
>> URL: http://svn.apache.org/viewvc?rev=695391&view=rev
>> Log:
>> JDK5, not 6..., who volunteers to port our JDBC debug facade to JDK5+6?
>>
>> Modified:
>>
>>  ode/trunk/bpel-api/src/main/java/org/apache/ode/bpel/rapi/Serializer.java
>>
> 
> Thanks for the fix! Actually I tried to port the JDBC facade but that would
> need a few classes (used by the additional methods) that are JDK 6 only. We
> could create our own "backport" but I'm not sure it's worth it. Plus there
> are some legal issues in implementing classes in the java.* package.

What about translating it into a dynamic proxy. Its then independent of
the underlying JDBC version. IIRC I have started that a while on a
another machine, perhaps I can find it again...

Cheers,
  Tammo