On the similar lines, I also think the XML support should turn itself OFF gracefully in environments that don't support XML. I think the patch requires JDK 1.4 or higher, so unless Derby has plans to end_of_line JDK 1.3 (not sure if there is a plan), it may be a good idea to turn off XML internally. (Raise error when attempted to use)
I plan to test this part and provide comments along with a few other items. Satheesh >+1 on getting this initial XML support into Derby. Even though the patch >has already in, I will try to review the new version sometime. >At some point the testing does need to be part of the regular derby >suite though I think the current approach is good. I need to understand >the dependency on xerces, and if the default XML parser supplied by the >VM can be used instead. > >This is probably a good example of open source, get some initial working >functionality (incremental development) and then see if it can be >improved by others. > >Dan. > > > > >
