https://bugs.documentfoundation.org/show_bug.cgi?id=120436

Mike Kaganski <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |WORKSFORME
             Status|NEW                         |RESOLVED

--- Comment #8 from Mike Kaganski <[email protected]> ---
Likely, something has changed since then: *both* testformula started to work
much faster, *and* testformulalocal started to work much slower.

And also it seems that I wasn't clear enough in STR. Let me explain.

To check, I opened the file in 6.2.0.3, and made changes according to comment 1
(I use Russian UI, and Use English function names).
Version: 6.2.0.3 (x64)
Build ID: 98c6a8a1c6c7b144ce3cc729e34964b47ce25d62
CPU threads: 12; OS: Windows 10.0; UI render: GL; VCL: win; 
Locale: ru-RU (ru_RU); UI-Language: ru-RU
Calc: CL

First I run testformulalocal (the first in the module). It took less than 10s
on my system.
Then I run testformula, and it took very long. Just as I explained in the
report.

Now I decided to double-check running testformulalocal again, and this time,
testformulalocal took longer than 90 s. Not nearly as log as testformula, but
it made it clear that the repro steps should had a step like "run
testformulalocal once before further testing" to get the numbers reported back
then. However, testformula worked very slowly even with clear column L. So in
the end: the report was valid for version 6.2.

But let me clarify the problematic numbers from comment 0 using v.6.2:

testformulalocal took ~7s the first time on clean column L, and ~100s when run
the second time on already filled column L.
testformula took ~860s the first time on clean column L, and ~1400s when run
the second time on already filled column L.

Now I re-test using 7.3.5.1 (again, Russian UI, and Use English function
names).
Version: 7.3.5.1 (x64) / LibreOffice Community
Build ID: d56c1c78db15939340c3db8ee3b6667832313d23
CPU threads: 12; OS: Windows 10.0 Build 19044; UI render: Skia/Raster; VCL: win
Locale: ru-RU (ru_RU); UI: ru-RU
Calc: CL

testformulalocal took ~13s first time (with clear column L). Re-running it
again, with already filled column L, takes now ~650s. A regression in the
second case.

testformula took ~11s first time (with clear column L). Re-running it again,
with already filled column L, takes now ~24s - great improvement over v.6.2 in
both cases.

I close this one as WORKSFORME (because Formula now works at least no longer
than FormulaLocal now, and much better than in 6.2, and also faster than
FormulaLocal took back then in the worst case). But the regression in
FormulaLocal needs a separate issue, bisection and fixing. Andreas: I assume,
this is what you do in https://gerrit.libreoffice.org/c/core/+/136942 - but I
still see different figures compared to comment 7, so there's some additional
aspect here that I don't see.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to