[
https://issues.apache.org/jira/browse/DERBY-2469?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12502152
]
David Van Couvering commented on DERBY-2469:
--------------------------------------------
Hi, Luigi. I was able to apply the patch. A couple of intiial comments.
"Showstopper" means we have to get this resolved prior to checkin. The rest
are desired but IMHO optional.
Showstopper: - The JNLP jar file is not available with JDK 1.4, and also your
code assumes JDK 1.5 (generics, annotations, for-each loops). Does this means
you want this feature to be available only if the compiler (and thus the JRE)
is 1.5 or higher? If so, we need to explicitly prevent this feature from being
loaded at runtime if the user is running with a 1.4 VM. I'm not personally
sure exactly how you do this, but others on the dev list have experience with
this and you can ask about it.
If you want it to be available with a 1.4 VM, then we need to fix the code to
use only 1.4-compliant syntax, and we need to update BUILDING.txt to provide
instructions for how to get jnlp.jar from
http://java.sun.com/products/javawebstart/download-jnlp.html.
- I have to concur with Dan, it would really help to provide instructions (say
in the javadoc for the JNLP storage factory) for how you enable this and how
you use it. I'm not a JNLP expert and I'm kind of scratching my head around
how to get started...
- You might think about checking in a demo Java Web Start application to the
java/demo directory, with instructions, to make it easier for people to figure
out how to use this feature.
I'll probably have more feedback after playing around with this for a bit.
David
> Java Web Start JNLP PersistenceService API storage support
> ----------------------------------------------------------
>
> Key: DERBY-2469
> URL: https://issues.apache.org/jira/browse/DERBY-2469
> Project: Derby
> Issue Type: New Feature
> Components: Store
> Affects Versions: 10.2.2.0
> Environment: Java Web Start
> Reporter: Luigi Lauro
> Priority: Minor
> Attachments: svn-diff-20070329, svn-diff-20070606
>
>
> I would love to have Derby write/read to the storage area provided by the
> JNLP PersistenceService API.
> Since Derby is now bundled with the Java6 JDK as JavaDB, I think this
> integration would go a long way towards making derby more developer- friendly
> in Java Web Start environments, where using the sandbox tools Sun provides us
> it the right way to go, instead of working around it and force the user to
> give the app the authorization to write on the hard drive IMHO.
> I'm investigating the effort needed to provide an implementation of the
> WritableStorageFactory interface around the PersistenceService API, and if
> that's doable in a few days work, I will start working on it and submit a
> patch for testing/approval ASAP.
> Feel free to volounteer and provide pointers/hints/whatever, it's really
> appreciate, especially since I currently know nothing of derby internals.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
