[
https://issues.apache.org/jira/browse/TUSCANY-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12473128
]
Kelvin Goodson commented on TUSCANY-1091:
-----------------------------------------
Hi Yang,
I'm going to have to push back on a couple of things here, sorry.
First is that I'm getting a test failure when I run in maven (not in eclipse)
the exception stack is ...
Test set: org.apache.tuscany.sdo.test.XMLStreamHelperPerformanceTestCase
-------------------------------------------------------------------------------
Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.06 sec <<<
FAILURE!
testDynamic(org.apache.tuscany.sdo.test.XMLStreamHelperPerformanceTestCase)
Time elapsed: 0.03 sec <<< ERROR!
java.lang.NullPointerException
at
org.apache.tuscany.sdo.util.DataObjectUtil.createDataObject(DataObjectUtil.java:476)
at
org.apache.tuscany.sdo.impl.DataObjectImpl.createDataObject(DataObjectImpl.java:1196)
at
org.apache.tuscany.sdo.test.XMLStreamHelperPerformanceTestCase.testDynamic(XMLStreamHelperPerformanceTestCase.java:64)
at
org.apache.tuscany.sdo.test.XMLStreamHelperPerformanceTestCase.testDynamic(XMLStreamHelperPerformanceTestCase.java:64)
....
I'm not sure why line 64 appears twice in the stack, could you take a look
please?
The second thing is again something I'm not sure we have a policy on but I
think is could be confusing. You have introduced a class which differs in name
from the interface it implements only by case.
NameSpaceContext implements NamespaceContext
I think this is potentially confusing. Would you be able to change this to
perhaps a XXXImpl class please.
> DataObjectXMLStreamReader doesn't have to repeat NameSpace declaration which
> impacts performance
> ------------------------------------------------------------------------------------------------
>
> Key: TUSCANY-1091
> URL: https://issues.apache.org/jira/browse/TUSCANY-1091
> Project: Tuscany
> Issue Type: Improvement
> Components: Java SDO Implementation
> Affects Versions: Java-SDO-Mx
> Reporter: Yang ZHONG
> Fix For: Java-SDO-Mx
>
> Attachments: bindLessNS, bindLessNS, simple3.xsd,
> XMLStreamHelper.xsd, XMLStreamHelperPerformanceTestCase.java
>
>
> <p0:quotes3>
> <p1:symbol3
> xmlns:p1="http://www.example.com/simple3">IBM</p1:symbol3>
> <p1:company3 xmlns:p1="http://www.example.com/simple3">IBM
> Corp.</p1:company3>
> </p0:quotes3>
> can be optimized as
> <p0:quotes3 xmlns:p1="http://www.example.com/simple3">
> <p1:symbol3>IBM</p1:symbol3>
> <p1:company3>IBM Corp.</p1:company3>
> </p0:quotes3>
> Thanks to Fuhwei Lwo for prototyping the Test Case. I've made small changes
> to make use of HelperContext so that it will neither interfere nor be
> interfereed by other build test(s).
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]