[ 
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]

Reply via email to