[GitHub] [jena] georgschmidtdumont commented on issue #651: JENA-1796: Add missing DV for XSD DateTimeStamp data type

[GitBox <gi...@apache.org>]

georgschmidtdumont commented on issue #651: JENA-1796: Add missing DV for XSD DateTimeStamp data type
URL: https://github.com/apache/jena/pull/651#issuecomment-565345090
 
 
   Hi @afs ,
   
   I did not take this from the Xerces2-j code base. I had a look at the Xerces code base and there a one or two small differences in the Xerces implementation and this implementation. I tried to align the code and it is not entirely possible at the moment since the `isXMLSchema11` flag in the `parse` method is not available in Jena implementation.
   Since this flag is added in the xml-schema-1.1-dev branch in Xerces and this branch is not merged or has a PR I would suggest using the implementation in this PR and update it once the xml-schema-1.1-dev branch has been merged.
   I hope this is not a blocker for this PR.
   
   Do you know why the Xerces classes are duplicated in Jena instead of pulling Xerces as a dependency?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services