[ 
https://issues.apache.org/jira/browse/TUSCANY-2300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12596004#action_12596004
 ] 

Stanisław T. Findeisen commented on TUSCANY-2300:
-------------------------------------------------

Adriano, I think those patches are OK. Just take a look into the files. What 
command do you use to apply them?

However, what I said about this being a patch for M3 was not 100% true. I 
attach you tuscany-M3-patch-2008-05-12-do-czytania.tgz, which truly is a patch 
for M3. You can forget about the first 2 files (tuscany-patch1-do-czytania.tgz 
and tuscany-patch2.tgz).

Unfortunately, this new patch is bigger and contains more changes. THIS BIGGER 
PATCH IS ADDRESSING NOT ONLY PERFORMANCE ISSUES, BUT ALSO SEVERAL BUGS. I will 
ask Albert to write more on this.

STF


> performance issues in DataFactoryImpl, TypeImpl, DataObjectImpl, SDOXMLWriter
> -----------------------------------------------------------------------------
>
>                 Key: TUSCANY-2300
>                 URL: https://issues.apache.org/jira/browse/TUSCANY-2300
>             Project: Tuscany
>          Issue Type: Improvement
>          Components: C++ SDO
>            Reporter: Stanisław T. Findeisen
>             Fix For: Cpp-Next
>
>         Attachments: tuscany-patch1-do-czytania.tgz, tuscany-patch2.tgz
>
>
> Hello Tuscany Developers
> Here at IBM Polska we wrote a patch for M3 addressing some performance 
> issues. It is short, so please read it - I hope it is self-explanatory.
> Main areas of our development were:
> (1) DataFactoryImpl::findTypeImpl --- we added new string hash based 
> searching of types.
> (2) TypeImpl --- we added these maps:
>     std::map<const char*, int, TStrCmp> hmPropIdx;   // property index by 
> property name
>     std::map<unsigned int, PropertyImpl*> propByIdx; // property by index
> (3) DataObjectImpl::getPropertyIndex(const Property& p) and 
> DataObjectImpl::getProperty(unsigned int index) (make use of improvements in 
> TypeImpl)
> (4) SDOXMLWriter --- we removed some conversions from std::list to 
> std::vector that we thought were unnecessary.
> I hope you will be able to apply this patch and include it in the next 
> release. Let me know of you have any problems.
> STF

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to