|
|
Re: XLSXParser::layoutCells() performance issue [message #1848219 is a reply to message #1848218] |
Thu, 25 November 2021 10:47 ![Go to previous message Go to previous message](https://www.eclipse.org/forums/theme/Solstice/images/up.png) ![Go to next message Go to next message](https://www.eclipse.org/forums/theme/Solstice/images/down.png) |
Tristan Faure![Friend of Eclipse Friend](https://www.eclipse.org/donate/web-api/friends_decorator.php?email=faure.tristan%40gmail.com) Messages: 460 Registered: July 2009 |
Senior Member |
|
|
Yes we could investigate on xerces check operations. Nevertheless caching getLentgth() is also relevant
@Denis about overriding XLSparser you can but you have to override the document Factory which can instantiate your XLS Document which will return your xls parser. Doable but not so quick, please ask if you need more details.
About changing the code base I could check or will see with other committers sorry for the potential delay :(
EDIT : BTW thank you very much for the performed analysis @Denis
[Updated on: Thu, 25 November 2021 10:48] Report message to a moderator
|
|
|
|
Re: XLSXParser::layoutCells() performance issue [message #1848224 is a reply to message #1848220] |
Thu, 25 November 2021 12:52 ![Go to previous message Go to previous message](https://www.eclipse.org/forums/theme/Solstice/images/up.png) ![Go to next message Go to next message](https://www.eclipse.org/forums/theme/Solstice/images/down.png) |
Tristan Faure![Friend of Eclipse Friend](https://www.eclipse.org/donate/web-api/friends_decorator.php?email=faure.tristan%40gmail.com) Messages: 460 Registered: July 2009 |
Senior Member |
|
|
Hello yes for the described steps
I made a mistake I thought it was a service and a service extension has a priority and could override another one.
In your case for document factory indeed once an extension is registered you can't override it. Maybe the solution you describe using features works but I'm not sure
A workaround could be to create a factory for a specific extension for instances "xlsx2" ,match this extension in your factory and change the extension of your doc ...
[Updated on: Thu, 25 November 2021 13:13] Report message to a moderator
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Powered by
FUDForum. Page generated in 0.02257 seconds